[coreboot-gerrit] Change in coreboot[master]: [RFC] ChromeOS: Declare CHROMEOS_NO_GPIOS

[Yu-Ping Wu (Code Review)]

Attention is currently required from: Jason Glenesk, Marshall Dawson, Julius Werner, Kyösti Mälkki, Felix Held.
Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58956 )

Change subject: [RFC] ChromeOS: Declare CHROMEOS_NO_GPIOS
......................................................................

Patch Set 1:

(1 comment)

Commit Message:

https://review.coreboot.org/c/coreboot/+/58956/comment/fb1911b6_3382ce4c 
PS1, Line 9: Implementation of fill_lb_gpios() is not about VBOOT
I agree that bootmode.c isn't the best place to put fill_lb_gpios(). However, that doesn't justify this patch, which replaces the weak function with a Kconfig option. Please explain why we can't just move fill_lb_gpios() to somewhere else, for example, coreboot_table.c. We use weak functions a lot in coreboot. I don't see there's a benefit changing them to Kconfig options.