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

[Sean Rhodes (Code Review)]

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

Sean Rhodes 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 4:

(2 comments)

File src/drivers/option/cfr.c:

https://review.coreboot.org/c/coreboot/+/74121/comment/231075d9_235fb69b 
PS3, Line 103: option_id
> It's not an index. This is meant to be used in the future for dynamic dependencies (e.g. […]

Done

https://review.coreboot.org/c/coreboot/+/74121/comment/462aafea_c6529fec 
PS3, Line 103: 	option->option_id = option_id;
             : 	option->flags = flags;
             : 	option->default_value = default_value;
             : 	option->size = sizeof(*option);
             : 
             : 	current += option->size;
             : 	current += sm_write_opt_name(current, opt_name)
> You mean the naming of `option_id` and `opt_name` is inconsistent? Renamed the former to `object_id` […]

Done