Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34476 )
Change subject: Rampayload: Attempt to boot coreboot without ramstage ......................................................................
Patch Set 3:
(4 comments)
https://review.coreboot.org/c/coreboot/+/34476/2/Makefile.inc File Makefile.inc:
https://review.coreboot.org/c/coreboot/+/34476/2/Makefile.inc@255 PS2, Line 255: ifeq ($(CONFIG_RAMPAYLOAD),y)
i don't think ENV_PAYLOAD_LOADER macro will able to tell us what is the stage name.
Done
https://review.coreboot.org/c/coreboot/+/34476/2/src/lib/program.ld File src/lib/program.ld:
https://review.coreboot.org/c/coreboot/+/34476/2/src/lib/program.ld@58 PS2, Line 58: #if ENV_PAYLOAD_LOADER
we will try to avoid this. […]
Done
https://review.coreboot.org/c/coreboot/+/34476/2/src/mainboard/google/dragon... File src/mainboard/google/dragonegg/chromeos.c:
https://review.coreboot.org/c/coreboot/+/34476/2/src/mainboard/google/dragon... PS2, Line 24: #if ENV_PAYLOAD_LOADER
if we plan to not support chrome device then we don't need this now
Done
https://review.coreboot.org/c/coreboot/+/34476/2/src/mainboard/intel/icelake... File src/mainboard/intel/icelake_rvp/chromeos.c:
https://review.coreboot.org/c/coreboot/+/34476/2/src/mainboard/intel/icelake... PS2, Line 23: #if ENV_PAYLOAD_LOADER
same question. If you are doing this just to get it all to build, let's take another look. […]
Done