[coreboot-gerrit] [L] Change in coreboot[master]: [RFC] drivers/option: Add option list in cbtable

[Benjamin Doron (Code Review)]

Attention is currently required from: Sean Rhodes, Angel Pons, Maximilian Brune, Lean Sheng Tan.

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 3:

(1 comment)

File src/drivers/option/cfr.c:

https://review.coreboot.org/c/coreboot/+/74121/comment/766719a1_5a9c2996 
PS3, Line 103: option_id
> Is this intended to be an index?

I know it's used in EDK2's HII backend as the "question" identifier.

In CFR, I'd rather move it to the backend too, or establish that CFR should be sorted by these IDs. For instance, suppose the developer initialises and adds options in the order 1, 3, 2. At the moment, in EDK2, they're processed in struct-order not ID-order.