[coreboot-gerrit] New patch to review for coreboot: d89c73f Removed unused SOUTHBRIDGE_INTEL_FSP_I89XX expressions

Martin Roth (gaumless@gmail.com) gerrit at coreboot.org
Sat Jun 6 05:55:39 CEST 2015


Martin Roth (gaumless at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10436

-gerrit

commit d89c73fb7cdf8189b8d047e8dfdafc3be099ee65
Author: Martin Roth <gaumless at gmail.com>
Date:   Fri Jun 5 21:00:07 2015 -0600

    Removed unused SOUTHBRIDGE_INTEL_FSP_I89XX expressions
    
    The SOUTHBRIDGE_INTEL_FSP_I89XX symbols are never defined in any Kconfig
    file or used anywhere in the existing coreboot tree.  Removing them as
    unnecessary.  If the southbridge code ever gets uploaded, these can be
    re-added at that point.
    
    Change-Id: I36f9ca8e25e08ce154d10ea9d764a73095590244
    Signed-off-by: Martin Roth <gaumless at gmail.com>
---
 src/northbridge/intel/fsp_sandybridge/fsp/Kconfig | 2 --
 src/vendorcode/intel/Kconfig                      | 1 -
 2 files changed, 3 deletions(-)

diff --git a/src/northbridge/intel/fsp_sandybridge/fsp/Kconfig b/src/northbridge/intel/fsp_sandybridge/fsp/Kconfig
index 2456a42..82e5361 100644
--- a/src/northbridge/intel/fsp_sandybridge/fsp/Kconfig
+++ b/src/northbridge/intel/fsp_sandybridge/fsp/Kconfig
@@ -21,12 +21,10 @@ config SANDYBRIDGE_FSP_SPECIFIC_OPTIONS
 	def_bool y
 	select PLATFORM_USES_FSP1_0
 	select USE_GENERIC_FSP_CAR_INC
-	select FSP_USES_UPD if SOUTHBRIDGE_INTEL_FSP_I89XX
 
 config FSP_FILE
 	string
 	default "../intel/fsp/ivybridge_bd82x6x/FvFsp.bin" if SOUTHBRIDGE_INTEL_FSP_BD82X6X
-	default "../intel/fsp/ivybridge_i89xx/FvFsp.bin" if SOUTHBRIDGE_INTEL_FSP_I89XX
 	help
 	  The path and filename of the Intel FSP binary for this platform.
 
diff --git a/src/vendorcode/intel/Kconfig b/src/vendorcode/intel/Kconfig
index 8e1b743..0da26f0 100644
--- a/src/vendorcode/intel/Kconfig
+++ b/src/vendorcode/intel/Kconfig
@@ -20,6 +20,5 @@
 config FSP_VENDORCODE_HEADER_PATH
 	string
 	default "fsp1_0/ivybridge_bd82x6x" if CPU_INTEL_FSP_MODEL_306AX && SOUTHBRIDGE_INTEL_FSP_BD82X6X
-	default "fsp1_0/ivybridge_i89xx" if CPU_INTEL_FSP_MODEL_306AX && SOUTHBRIDGE_INTEL_FSP_I89XX
 	default "fsp1_0/baytrail" if SOC_INTEL_FSP_BAYTRAIL
 	default "fsp1_0/rangeley" if CPU_INTEL_FSP_MODEL_406DX



More information about the coreboot-gerrit mailing list