[coreboot-gerrit] New patch to review for coreboot: southbridge/intel: Pass platform-specific quirks to ifdtool

[Andrey Petrov (andrey.petrov@intel.com)]

Andrey Petrov (andrey.petrov at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17258

-gerrit

commit 87305fd038acecda541b52cbac77c523207c25cf
Author: Andrey Petrov <andrey.petrov at intel.com>
Date:   Sun Nov 6 23:43:16 2016 -0800

    southbridge/intel: Pass platform-specific quirks to ifdtool
    
    Since IFD format is floating, ifdtool needs to know what specific chipset
    it is dealing with. Add -p <chipset> argument to ifdtool invocations.
    
    Change-Id: I4fd1783b5d994617912aedcf17adc2a98c97227b
    Signed-off-by: Andrey Petrov <andrey.petrov at intel.com>
---
 src/southbridge/intel/common/firmware/Makefile.inc | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/src/southbridge/intel/common/firmware/Makefile.inc b/src/southbridge/intel/common/firmware/Makefile.inc
index fa28ccf..a3f1571 100644
--- a/src/southbridge/intel/common/firmware/Makefile.inc
+++ b/src/southbridge/intel/common/firmware/Makefile.inc
@@ -33,6 +33,7 @@ IFD_SECTIONS := $(addprefix -b ,$(CONFIG_IFD_BIOS_SECTION:"%"=%)) \
 		$(addprefix -p ,$(CONFIG_IFD_PLATFORM_SECTION:"%"=%))
 else
 IFD_BIN_PATH := $(CONFIG_IFD_BIN_PATH)
+IFDTOOL_CHIPSET_QUIRKS := -p $(CONFIG_IFD_USE_CHIPSET_QUIRK)
 endif
 
 add_intel_firmware: $(obj)/coreboot.pre $(IFDTOOL) $(IFDFAKE)
@@ -57,23 +58,27 @@ endif
 ifeq ($(CONFIG_HAVE_GBE_BIN),y)
 	printf "    IFDTOOL    gbe.bin -> coreboot.pre\n"
 	$(objutil)/ifdtool/ifdtool \
+		$(IFDTOOL_CHIPSET_QUIRKS) \
 		-i GbE:$(CONFIG_GBE_BIN_PATH) \
 		$(obj)/coreboot.pre
 	mv $(obj)/coreboot.pre.new $(obj)/coreboot.pre
 endif
 ifeq ($(CONFIG_LOCK_MANAGEMENT_ENGINE),y)
 	printf "    IFDTOOL    Locking Management Engine\n"
-	$(objutil)/ifdtool/ifdtool -l $(obj)/coreboot.pre
+	$(objutil)/ifdtool/ifdtool \
+		$(IFDTOOL_CHIPSET_QUIRKS) -l $(obj)/coreboot.pre
 	mv $(obj)/coreboot.pre.new $(obj)/coreboot.pre
 else ifneq ($(CONFIG_BUILD_WITH_FAKE_IFD),y)
 	printf "    IFDTOOL    Unlocking Management Engine\n"
-	$(objutil)/ifdtool/ifdtool -u $(obj)/coreboot.pre
+	$(objutil)/ifdtool/ifdtool \
+	$(IFDTOOL_CHIPSET_QUIRKS) -u $(obj)/coreboot.pre
 	mv $(obj)/coreboot.pre.new $(obj)/coreboot.pre
 endif
 
 ifeq ($(CONFIG_EM100),y)
 	printf "    IFDTOOL    Setting EM100 mode\n"
-	$(objutil)/ifdtool/ifdtool --em100 $(obj)/coreboot.pre
+	$(objutil)/ifdtool/ifdtool \
+		$(IFDTOOL_CHIPSET_QUIRKS) --em100 $(obj)/coreboot.pre
 	mv $(obj)/coreboot.pre.new $(obj)/coreboot.pre
 endif