Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32870 )
Change subject: fit: Refactor config node handling ......................................................................
Patch Set 5:
(1 comment)
https://review.coreboot.org/#/c/32870/5/src/lib/fit_payload.c File src/lib/fit_payload.c:
https://review.coreboot.org/#/c/32870/5/src/lib/fit_payload.c@198 PS5, Line 198: #if defined(CONFIG_LINUX_COMMAND_LINE) : fit_update_chosen(dt, (char *)CONFIG_LINUX_COMMAND_LINE); : #endif
not blocking +2, but will it help if we for example define the config as NULL and check that? […]
Kconfig can't do that, if the string symbol is not visible it will just not be defined at all in config.h. I guess we could hack kconfig further to change that, but I'm not sure if we want that. We use string symbols quite rarely anyway... I think doing this in the few cases that we do is fine.