[coreboot-gerrit] [L] Change in coreboot[main]: [RFC] drivers/option: Add option list in cbtables

[Patrick Rudolph (Code Review)]

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

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74121?usp=email )

Change subject: [RFC] drivers/option: Add option list in cbtables
......................................................................

Patch Set 17:

(3 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/74121/comment/5b5505dc_45770815 :
PS1, Line 7: [RFC] drivers/option: Add option list in cbtable
> Although this is related to coreboot's option API, it's true that this is a different thing. […]

Historical coreboot provided a (CMOS) option list in a cbtable. Will rephrase to match the current implementation.

https://review.coreboot.org/c/coreboot/+/74121/comment/9876cd9c_7b0fa800 :
PS1, Line 9: Introduce a mechanism so that coreboot can provide a list of options to
> How closely to integrate with the option backend is another question. […]

Upstream coreboot always defined the (CMOS) options and passed them to the payload.

Commit Message:

https://review.coreboot.org/c/coreboot/+/74121/comment/cc04501c_3b274f40 :
PS5, Line 9: TODO: rewrite commit msg, add proper documentation. Maybe add tests.
> For tests, some of the code from https://github.com/Th3Fanbus/cfr-tools can certainly be reused. […]

Done