[coreboot-gerrit] Change in coreboot[master]: mb/google/skyrim: Inject SPDs into APCB

[Karthik Ramasubramanian (Code Review)]

Karthik Ramasubramanian has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/63600 )

Change subject: mb/google/skyrim: Inject SPDs into APCB
......................................................................

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 
Change-Id: I3b3f6f248d54681c6f55c00660d1f2988ae906ba
---
M src/mainboard/google/skyrim/Makefile.inc
1 file changed, 13 insertions(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/00/63600/1

diff --git a/src/mainboard/google/skyrim/Makefile.inc b/src/mainboard/google/skyrim/Makefile.inc
index e66f7c9..bf9c012 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) \
+			     --spd_magic '2311130E'
 else
 $(info APCB sources not found.  Skipping APCB.)
 endif