[coreboot-gerrit] Change in ...coreboot[master]: fit: Refactor config node handling

[Hung-Te Lin (Code Review)]

Hung-Te Lin 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
> No, because this is a string Kconfig, CONFIG() is only for boolean Kconfigs.

not blocking +2, but will it help if we for example define the config as NULL and check that?

simply trying to eliminate conditional compile.