Subrata Banik 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)
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).
So you are telling to have ENABLE_STAGE_RAMSTAGE=n and ARCH_RAMSTAGE_X86 can still be yes ? is that what you wish to conclude then i can make code changes accordingly.
I guess Aaron has recommended to have ENABLE_STAGE_<bootblock/romstage/ramsatge> selection from ARCH_ config. And looks like you don't wnat that to be interdependent ?
I was saying that's the path of least resistance w.r.t. selecting the high level configs. Julius wants to put that selection somewhere else. In either case, getting the Makefile root dependencies decoupled from the implicit assumptions and base them on high level configs needs to be done *regardless* of the point of selection and steering the high level configs.
Oh yes, looks like root dependency is almost sorted now except CL: https://review.coreboot.org/c/coreboot/+/33114 which you might need to take a look.
Now stage selection is holding this ramstage independent coreboot build :(