Hello Aaron Durbin, ron minnich, Julius Werner, build bot (Jenkins), Martin Roth, Furquan Shaikh, Patrick Georgi,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/33142
to look at the new patch set (#7).
Change subject: Rampayload: Able to build coreboot without ramstage ......................................................................
Rampayload: Able to build coreboot without ramstage
This patch removes all possible dependencies in order to build platform with CONFIG_RAMPAYLOAD enable(without ramstage).
A. Create coreboot separate stage kconfigs
This patch creates seperate stage configs as below 1. ENABLE_STAGE_BOOTBLOCK 2. ENABLE_STAGE_VERSTAGE 3. ENABLE_STAGE_ROMSTAGE 4. ENABLE_STAGE_POSTCAR 5. ENABLE_STAGE_RAMSTAGE
B. Also ensures below kconfigs are aligned with correct stage configs
1. COMPRESS_RAMSTAGE and RELOCATABLE_RAMSTAGE are now enable if CONFIG_ENABLE_STAGE_RAMSTAGE is selected. 2. COMPRESS_BOOTBLOCK will enable if CONFIG_STAGE_BOOTBLOCK is set 3. COMPRESS_PRERAM_STAGES will enable if CONFIG_ENABLE_STAGE_VERSTAGE || CONFIG_ENABLE_STAGE_ROMSTAGE is selected.
C. Also fix compilation issue with !CONFIG_ENABLE_STAGE_RAMSTAGE
On x86 platform: Case 1: ramstage do exist: CONFIG_STAGE_RAMSTAGE=1
rmodules_$(ARCH-ramstage-y) will evaluate as rmodules_x86_32
Case 2: ramstage doesn't exist: CONFIG_STAGE_RAMSTAGE=0
rmodules_$(ARCH-ramstage-y) will evaluate as rmodules_
This patch fixes Case 2 usecase where platform doesn't select CONFIG_ENABLE_STAGE_RAMSTAGE.
Also add option to create sipi_vector.manual based on $(TARGET_STAGE) variable.
$(TARGET_STAGE)=ramstage if user selects CONFIG_ENABLE_STAGE_RAMSTAGE $(TARGET_STAGE)=postcar if user selects CONFIG_RAMPAYLOAD
Change-Id: I0f7e4174619016c5a54c28bedd52699df417a5b7 Signed-off-by: Subrata Banik subrata.banik@intel.com --- M Makefile.inc M src/Kconfig M src/cpu/x86/Makefile.inc 3 files changed, 66 insertions(+), 19 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/42/33142/7