[coreboot-gerrit] Change in coreboot[master]: fw_config: Add firmware configuration interface

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41209 )

Change subject: fw_config: Add firmware configuration interface
......................................................................

Patch Set 9:

(1 comment)

I did not really review the work here, I just wanted to leave a general (possibly unrelated) comment that linking static.c into SMM is IMHO a bad idea since the copy of devicetree there does not reflect any modifications ramstage could have made.

https://review.coreboot.org/c/coreboot/+/41209/7/src/lib/fw_config.c 
File src/lib/fw_config.c:

https://review.coreboot.org/c/coreboot/+/41209/7/src/lib/fw_config.c@11 
PS7, Line 11: MAYBE_STATIC_BSS bool fw_config_value_initialized;
> Hm yes this was supposed to be in a function and it isn't really a migrated global but I had not see […]

Leave comments in CB:40535 please.