summaryrefslogtreecommitdiff
path: root/src/mainboard
diff options
context:
space:
mode:
authorMartin Roth <gaumless@gmail.com>2023-05-11 15:00:40 -0600
committerMartin L Roth <gaumless@gmail.com>2023-05-18 08:16:47 +0000
commitdd4fa8e6038a1481622ee39f02d6cf1a1228ea72 (patch)
treeb5655adccab6d3640bea3e76beca8600aa26ffec /src/mainboard
parentfb0314075721f3ad65148db9900bd51f7cd1b5ab (diff)
Myst: Update Makefile to remove SPD injection
The SPD format in the APCB has changed for Phoenix, and the injection tool 'apcbtool' needs to be updated to match. Until this happens, the APCB will be built containing the correct SPD. BUG=b:281983434 TEST=Build Signed-off-by: Martin Roth <gaumless@gmail.com> Change-Id: If575f98511c796e93c5a12cd450a3a7985e39806 Reviewed-on: https://review.coreboot.org/c/coreboot/+/75143 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Jon Murphy <jpmurphy@google.com>
Diffstat (limited to 'src/mainboard')
-rw-r--r--src/mainboard/google/myst/Makefile.inc39
1 files changed, 23 insertions, 16 deletions
diff --git a/src/mainboard/google/myst/Makefile.inc b/src/mainboard/google/myst/Makefile.inc
index 136966e550..54d4f30d57 100644
--- a/src/mainboard/google/myst/Makefile.inc
+++ b/src/mainboard/google/myst/Makefile.inc
@@ -20,23 +20,30 @@ APCB_NAME=APCB_PHX_D5
ifneq ($(wildcard $(MAINBOARD_BLOBS_DIR)/$(APCB_NAME).bin),)
$(info APCB sources present.)
-ifneq ($(wildcard $(src)/mainboard/$(MAINBOARDDIR)/variants/$(VARIANT_DIR)/memory/Makefile.inc),)
+APCB_SOURCES = $(MAINBOARD_BLOBS_DIR)/$(APCB_NAME).bin
+
+# The SPD is currently built directly into the APCB.
+# Add the below section back in after the apcbtool is updated to handle the
+# Phoenix APCB SPD configuration.
+
+#ifneq ($(wildcard $(src)/mainboard/$(MAINBOARDDIR)/variants/$(VARIANT_DIR)/memory/Makefile.inc),)
+#
+#LIB_SPD_DEPS = $(SPD_SOURCES)
+#
+#APCB_SOURCES = $(obj)/$(APCB_NAME).gen
+#
+#$(obj)/$(APCB_NAME).gen: $(SPD_SOURCES) \
+# $(APCB_V3_EDIT_TOOL) \
+# $(MAINBOARD_BLOBS_DIR)/$(APCB_NAME).bin
+# $(APCB_V3_EDIT_TOOL) $(MAINBOARD_BLOBS_DIR)/$(APCB_NAME).bin \
+# $(obj)/$(APCB_NAME).gen \
+# --spd_sources $(SPD_SOURCES) \
+# --mem_type 'lp5'
+#else
+#$(info SPD sources not found. Skipping APCB.)
+#files_added:: die_no_apcb
+#endif
-LIB_SPD_DEPS = $(SPD_SOURCES)
-
-APCB_SOURCES = $(obj)/$(APCB_NAME).gen
-
-$(obj)/$(APCB_NAME).gen: $(SPD_SOURCES) \
- $(APCB_V3_EDIT_TOOL) \
- $(MAINBOARD_BLOBS_DIR)/$(APCB_NAME).bin
- $(APCB_V3_EDIT_TOOL) $(MAINBOARD_BLOBS_DIR)/$(APCB_NAME).bin \
- $(obj)/$(APCB_NAME).gen \
- --spd_sources $(SPD_SOURCES) \
- --mem_type 'lp5'
-else
-$(info SPD sources not found. Skipping APCB.)
-files_added:: die_no_apcb
-endif
else
$(info APCB sources not found. Skipping APCB.)
files_added:: warn_no_apcb