[coreboot-gerrit] [XL] Change in coreboot[main]: drivers/option: Add forms in cbtables

[Angel Pons (Code Review)]

Attention is currently required from: Benjamin Doron, Lean Sheng Tan, Martin L Roth, Maximilian Brune, Nico Huber, Patrick Rudolph, Sean Rhodes, Werner Zeh.

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

Change subject: drivers/option: Add forms in cbtables
......................................................................

Patch Set 29:

(1 comment)

File src/commonlib/include/commonlib/cfr.h:

https://review.coreboot.org/c/coreboot/+/74121/comment/223f540b_75abbe6b?usp... :
PS27, Line 143: uint32_t tag;		/* CFR_TAG_OPTION_VARCHAR */
              : 	uint32_t size;
              : 	uint64_t object_id;	/* Uniqueue ID */
              : 	uint64_t dependency_id;	/* Grayout if value of lb_cfr_numeric_option with given ID is 0.
              : 				 * Ignore if field is 0.
              : 				 */
              : 	uint32_t flags;		/* enum cfr_option_flags */
> Is this a nit? I am not sure how to do it, and i think current implementation is fine?

It's a "I will take care of it myself in a follow-up"