Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32618 )
Change subject: Kconfig: Enable coreboot lite firmware flow ......................................................................
Patch Set 3:
(3 comments)
https://review.coreboot.org/#/c/32618/3/src/Kconfig File src/Kconfig:
https://review.coreboot.org/#/c/32618/3/src/Kconfig@280 PS3, Line 280: config COREBOOT_LITE
Should that be the marketing name? […]
I think CONFIG_NO_RAMSTAGE or CONFIG_SKIP_RAMSTAGE would be the most reasonable names for this.
https://review.coreboot.org/#/c/32618/3/src/Kconfig@281 PS3, Line 281: bool "Enable coreboot lite flow (remove ramstage)" Does this need a depends on ARCH_X86 (at least for now)?
https://review.coreboot.org/#/c/32618/3/src/Kconfig@283 PS3, Line 283: help This should have a lot more explanation what this choice entails (e.g. no PCI enumeration or whatever), or refer to a Documentation/ file with more details.