Attention is currently required from: Ronak Kanabar, Sergii Dmytruk.
Martin L Roth has posted comments on this change by Sergii Dmytruk. ( https://review.coreboot.org/c/coreboot/+/83738?usp=email )
Change subject: vendorcode/intel: allow selecting EDK headers manually ......................................................................
Patch Set 3:
(1 comment)
File src/vendorcode/intel/Kconfig:
https://review.coreboot.org/c/coreboot/+/83738/comment/c9ccad98_e3b47008?usp... : PS3, Line 34: PAYLOAD_EDK2 Just thinking here, so feel free to ignore this.
I'm not a huge fan of requiring a specific payload be built at the same time as coreboot to enable coreboot functionality. It might make sense to add something to the payloads to say "This is the intended payload, but don't build it and include it in the image right now."
Any feelings about that?