[coreboot-gerrit] New patch to review for coreboot: fsp1_1: Don't hide build related options behind HAVE_FSP_BIN

[Stefan Reinauer (stefan.reinauer@coreboot.org)]

Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/12465

-gerrit

commit 9fb6abb36eb5e29a865bb12f7a5dec56145b6696
Author: Stefan Reinauer <stefan.reinauer at coreboot.org>
Date:   Wed Nov 18 15:28:19 2015 -0800

    fsp1_1: Don't hide build related options behind HAVE_FSP_BIN
    
    The right thing to do is to hide them behind PLATFORM_USES_FSP1_1.
    The only things that should depend on HAVE_FSP_BIN is the code
    that actually adds the file to CBFS, and the path to the file in Kconfig.
    
    Change-Id: I9b6c3ed0cdfb0e02421d7b98c488a66e39add947
    Signed-off-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
---
 src/drivers/intel/fsp1_1/Kconfig | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/src/drivers/intel/fsp1_1/Kconfig b/src/drivers/intel/fsp1_1/Kconfig
index 4ae3727..ed37820 100644
--- a/src/drivers/intel/fsp1_1/Kconfig
+++ b/src/drivers/intel/fsp1_1/Kconfig
@@ -40,8 +40,6 @@ config HAVE_FSP_BIN
 	  Note: Without this binary, coreboot builds relying on the FSP
 	  will not boot
 
-if HAVE_FSP_BIN
-
 config CPU_MICROCODE_CBFS_LEN
 	hex "Microcode update region length in bytes"
 	default 0
@@ -62,23 +60,19 @@ config FSP_FILE
 
 config FSP_IMAGE_ID_STRING
 	string "8 byte platform string identifying the FSP platform"
+	default "FSPROCKS"
 	help
 	  8 ASCII character byte signature string that will help match the FSP
 	  binary to a supported hardware configuration.
 
-config FSP_INCLUDE_PATH
-	string "Path for FSP specific include files"
-	help
-	  The path and filename of the Intel FSP binary for this platform.
-
 config FSP_LOC
 	hex "Intel FSP Binary location in CBFS"
+	default 0xfe0000
 	help
 	  The location in CBFS that the FSP is located. This must match the
 	  value that is set in the FSP binary.  If the FSP needs to be moved,
 	  rebase the FSP with Intel's BCT (tool).
 
-endif #HAVE_FSP_BIN
 
 config DISPLAY_FAST_BOOT_DATA
 	bool "Display fast boot data"