Attention is currently required from: Jason Glenesk, Raul Rangel, Matt DeVillier, Fred Reitberger, Felix Held.
Grzegorz Bernacki has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/74268 )
Change subject: mendocino: Generate FSP version file ......................................................................
mendocino: Generate FSP version file
This commits adds generation of FSP version file and addint it into CBFS when correct config is enabled.
BUG=b:224780134 TEST=Tested on Skyrim device
Change-Id: Id1a27261e1b88a26732fbb4efaf1a8a0a7158140 Signed-off-by: Grzegorz Bernacki bernacki@google.com --- M src/soc/amd/mendocino/Makefile.inc 1 file changed, 27 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/68/74268/1
diff --git a/src/soc/amd/mendocino/Makefile.inc b/src/soc/amd/mendocino/Makefile.inc index 9d9adce..665eb9e 100644 --- a/src/soc/amd/mendocino/Makefile.inc +++ b/src/soc/amd/mendocino/Makefile.inc @@ -216,6 +216,9 @@ # If vboot uses 2 RW slots, then 2 copies of PSP binaries are redundant OPT_RECOVERY_AB_SINGLE_COPY=$(if $(CONFIG_VBOOT_SLOTS_RW_AB), --recovery-ab-single-copy)
+FSP_VERSION_FILE=$(obj)/fsp_version +OPT_FSP_VERSION=$(if $(CONFIG_FSP_VERSION_FILE), --fsp-version $(FSP_VERSION_FILE)) + AMDFW_COMMON_ARGS=$(OPT_PSP_APCB_FILES) \ $(OPT_APOB_ADDR) \ $(OPT_DEBUG_AMDFWTOOL) \ @@ -259,6 +262,7 @@ $(OPT_VERSTAGE_FILE) \ $(OPT_VERSTAGE_SIG_FILE) \ $(OPT_SPL_TABLE_FILE) \ + $(OPT_FSP_VERSION) \ --location $(shell printf "%#x" $(MENDOCINO_FWM_POSITION)) \ --output $@
@@ -309,6 +313,13 @@ $(obj)/amdfw_a.rom.body: $(obj)/amdfw_a.rom $(obj)/amdfw_b.rom.body: $(obj)/amdfw_b.rom
+ifeq ($(CONFIG_FSP_VERSION_FILE),y) +$(FSP_VERSION_FILE): $(obj)/amdfw.rom +cbfs-files-y += fsp_version +fsp_version-file := $(FSP_VERSION_FILE) +fsp_version-type := raw +endif + ifeq ($(CONFIG_VBOOT_SLOTS_RW_A)$(CONFIG_VBOOT_STARTS_BEFORE_BOOTBLOCK),yy) cbfs-files-y += apu/amdfw_a apu/amdfw_a-file := $(obj)/amdfw_a.rom
-
Grzegorz Bernacki (Code Review)