Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33143 )
Change subject: Rampayload: Able to build coreboot without ramstage ......................................................................
Patch Set 7:
(1 comment)
https://review.coreboot.org/#/c/33143/3/toolchain.inc File toolchain.inc:
https://review.coreboot.org/#/c/33143/3/toolchain.inc@50 PS3, Line 50: ifeq ($(CONFIG_RAMPAYLOAD),y)
Today ARCH_RAMSTAGE_X86 is the way to tell if we like to build ramstage or not. At least thats how coreboot soc is structured now.
That's not true. That's what you're trying to turn it into and I'm trying to argue to please not do that. ;)
Today ARCH_RAMSTAGE_X86 tells what architecture the ramstage is and nothing else. There is no way to not build a ramstage today. There is a way to not build two other stages (verstage and postcar), but for both of those the option that controls whether the stage is built is completely orthogonal to the option that controls with architecture it is (and the latter is usually still set even if the former is not... e.g. ARCH_VERSTAGE_X86 is still set even if SEPERATE_VERSTAGE=n).