Skip to content

Conversation

@wiktormowinski
Copy link
Contributor

@wiktormowinski wiktormowinski commented Oct 29, 2025

ref: prot-1859

@macpijan
Copy link
Contributor

@wiktormowinski Please follow: https://docs.dasharo.com/dev-proc/source-code-structure/#commit-quality when contributing to coreboot/edk2 repositories.

@wiktormowinski wiktormowinski force-pushed the vp66xx-parity branch 2 times, most recently from cf7b40c to caaa6f4 Compare December 9, 2025 17:33
Upstream-Status: Inappropriate [Dasharo downstream]
Signed-off-by: Wiktor Mowinski <wiktor.mowinski@3mdeb.com>
Upstream-Status: Inappropriate [Dasharo downstream]
Signed-off-by: Wiktor Mowinski <wiktor.mowinski@3mdeb.com>
@wiktormowinski wiktormowinski force-pushed the vp66xx-parity branch 3 times, most recently from fc88f77 to 1f0d07c Compare December 10, 2025 09:40
@wiktormowinski
Copy link
Contributor Author

wiktormowinski commented Dec 10, 2025

@mkopec i need your help. This approach is being constantly marked as syntax error by runner.
1f0d07c
is this even a correct way to enable wol?

@mkopec
Copy link
Member

mkopec commented Dec 10, 2025

@wiktormowinski doesn't look correct. Try doing what this patch does: https://review.coreboot.org/c/coreboot/+/89783

@wiktormowinski wiktormowinski force-pushed the vp66xx-parity branch 2 times, most recently from 10c84e0 to 5ba4106 Compare December 10, 2025 10:48
Upstream-Status: Inappropriate [Dasharo downstream]
Signed-off-by: Wiktor Mowinski <wiktor.mowinski@3mdeb.com>
@filipleple filipleple changed the title configs/config.protectli_vp66xx: enable me protectli_vp66xx: feature parity Dec 17, 2025
@filipleple
Copy link
Member

Renamed to reflect what's actually being done

Upstream-Status: Inappropriate [Dasharo downstream]
Signed-off-by: Wiktor Mowinski <wiktor.mowinski@3mdeb.com>
filipleple
filipleple previously approved these changes Dec 19, 2025
Upstream-Status: Inappropriate [Dasharo downstream]
Signed-off-by: Wiktor Mowinski <wiktor.mowinski@3mdeb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants