Attention is currently required from: Angel Pons, Maximilian Brune.
Benjamin Doron has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74121 )
Change subject: [RFC] drivers/option: Add option list in cbtable ......................................................................
Patch Set 1:
(1 comment)
File src/drivers/option/cfr.c:
https://review.coreboot.org/c/coreboot/+/74121/comment/8e08ded2_028c3be0 PS1, Line 127: write_body(header); : : menu->size = sm_compute_record_size((char *)menu, header); A thought: if we let the boot-state machine do this, then silicon, platform drivers, board code, etc can all add their options. `sm_write_cfr()` gets split in two and removed from the API for this.
My in-progress implementation for EDK2 should handle cases where the user enabled the driver but nothing adds forms: we process the header and `while (ProcessedLength < CbCfrSetupMenu->size) { .. }` exits early, so no forms are produced, meaning the menu (in theory) is empty.