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

[Werner Zeh (Code Review)]

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

Werner Zeh 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:

(3 comments)

Patchset:

PS29: 
Just two small things. I marked them as resolved right away in order to  not block things here. If you think there is value in my findings feel free to update the patch.

File Documentation/drivers/cfr.md:

https://review.coreboot.org/c/coreboot/+/74121/comment/cf6b0618_10ed885a?usp... :
PS29, Line 211: CFR_OPTFLAG_GRAYOUT
This strongly sounds like adopting UEFI vfr methods. I would rather call it ```CFR_OPTFLAG_DISABLED``` or ```CFR_OPTFLAG_INACTIVE``` as the GRAYOUT strictly associates (at least for me)  with a graphical menu which the user can walk through. This is not the intention you have in mind with this implementation, though. As I do not want to block this it is up to you to decide, hence setting it as resolved.

File src/drivers/option/cfr.c:

https://review.coreboot.org/c/coreboot/+/74121/comment/4e01d13e_9a9643c6?usp... :
PS29, Line 42: char *data
Why redefining? Use ```char *data;`` in line 32 right away.