commit 6f023ece0860a7988ef7f1f15bea5b54df0161ff
author Karthikeyan Ramasubramanian <kramasub@google.com> Tue Apr 12 17:06:01 2022 -0600
committer Felix Held <felix-coreboot@felixheld.de> Thu Apr 14 22:25:51 2022 +0000
tree c7dd9081e8ddc17d28d16dfc6ee170fcf283dfe0
parent 44e449b15cecbc774c8ed30f6ca6e0d9917cc913

mb/google/skyrim: Inject SPDs into APCB

Update the build scripts to inject variant specific SPDs into APCB.

BUG=None
TEST=Build and boot to OS in Skyrim boards with all the concerned memory
parts.

Signed-off-by: Karthikeyan Ramasubramanian <kramasub@google.com>
Change-Id: I3b3f6f248d54681c6f55c00660d1f2988ae906ba
Reviewed-on: https://review.coreboot.org/c/coreboot/+/63600
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>

src/mainboard/google/skyrim/Makefile.inc

diff --git a/src/mainboard/google/skyrim/Makefile.inc b/src/mainboard/google/skyrim/Makefile.inc
index e66f7c9..f76a961 100644
--- a/src/mainboard/google/skyrim/Makefile.inc
+++ b/src/mainboard/google/skyrim/Makefile.inc
@@ -16,8 +16,19 @@
 
 ifneq ($(wildcard $(MAINBOARD_BLOBS_DIR)/APCB_SBR_D5.bin),)
 $(info APCB sources present.)
-APCB_SOURCES = $(MAINBOARD_BLOBS_DIR)/APCB_SBR_D5.bin
-APCB_SOURCES_RECOVERY = $(MAINBOARD_BLOBS_DIR)/APCB_SBR_D5.bin
+
+LIB_SPD_DEPS = $(SPD_SOURCES)
+
+APCB_SOURCES = $(obj)/APCB_SBR_D5.gen
+APCB_SOURCES_RECOVERY = $(obj)/APCB_SBR_D5.gen
+
+$(obj)/APCB_SBR_D5.gen: $(SPD_SOURCES) \
+			$(APCB_V3_EDIT_TOOL) \
+			$(MAINBOARD_BLOBS_DIR)/APCB_SBR_D5.bin
+	$(APCB_V3_EDIT_TOOL) $(MAINBOARD_BLOBS_DIR)/APCB_SBR_D5.bin \
+			     $(obj)/APCB_SBR_D5.gen  \
+			     --spd_sources $(SPD_SOURCES) \
+			     --mem_type 'lp5'
 else
 $(info APCB sources not found.  Skipping APCB.)
 endif