[coreboot-gerrit] New patch to review for coreboot: drivers/intel/fsp2_0: Eliminate CBFS hardcoded names for FSP

Andrey Petrov (andrey.petrov@intel.com) gerrit at coreboot.org
Sun May 15 02:22:42 CEST 2016


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

-gerrit

commit 21e327355f49a837b8e65a857325b29372c0c2a1
Author: Andrey Petrov <andrey.petrov at intel.com>
Date:   Sat May 14 16:32:39 2016 -0700

    drivers/intel/fsp2_0: Eliminate CBFS hardcoded names for FSP
    
    Change-Id: I350abeffc4d23e45e339464d036716ecdb2ba83a
    Signed-off-by: Andrey Petrov <andrey.petrov at intel.com>
---
 src/drivers/intel/fsp2_0/Kconfig        | 10 ++++++++++
 src/drivers/intel/fsp2_0/memory_init.c  |  2 +-
 src/drivers/intel/fsp2_0/silicon_init.c |  2 +-
 3 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/src/drivers/intel/fsp2_0/Kconfig b/src/drivers/intel/fsp2_0/Kconfig
index 19e04f7..9289f15 100644
--- a/src/drivers/intel/fsp2_0/Kconfig
+++ b/src/drivers/intel/fsp2_0/Kconfig
@@ -3,4 +3,14 @@ config PLATFORM_USES_FSP2_0
 	help
 	  Include FSP 2.0 wrappers and functionality
 
+if PLATFORM_USES_FSP2_0
 
+config FSP_S_CBFS
+	string "Name of FSP-S in CBFS"
+	default "fsps.bin"
+
+config FSP_M_CBFS
+	string "Name of FSP-M in CBFS"
+	default "fspm.bin"
+
+endif
diff --git a/src/drivers/intel/fsp2_0/memory_init.c b/src/drivers/intel/fsp2_0/memory_init.c
index 8033937..fa6e64e 100644
--- a/src/drivers/intel/fsp2_0/memory_init.c
+++ b/src/drivers/intel/fsp2_0/memory_init.c
@@ -66,7 +66,7 @@ enum fsp_status fsp_memory_init(void **hob_list, struct range_entry *range)
 	struct fsp_header hdr;
 
 	/* TODO: do not hardcode CBFS file names */
-	if (fsp_load_binary(&hdr, "blobs/fspm.bin", range) != CB_SUCCESS)
+	if (fsp_load_binary(&hdr, CONFIG_FSP_M_CBFS, range) != CB_SUCCESS)
 		return FSP_NOT_FOUND;
 
 	return do_fsp_memory_init(hob_list, &hdr);
diff --git a/src/drivers/intel/fsp2_0/silicon_init.c b/src/drivers/intel/fsp2_0/silicon_init.c
index e7d8c54..00d50fc 100644
--- a/src/drivers/intel/fsp2_0/silicon_init.c
+++ b/src/drivers/intel/fsp2_0/silicon_init.c
@@ -55,7 +55,7 @@ enum fsp_status fsp_silicon_init(struct range_entry *range)
 {
 	/* Load FSP-S and save FSP header. We will need it for Notify */
 	/* TODO: do not hardcode CBFS file names */
-	if (fsp_load_binary(&fsps_hdr, "blobs/fsps.bin", range) != CB_SUCCESS)
+	if (fsp_load_binary(&fsps_hdr, CONFIG_FSP_S_CBFS, range) != CB_SUCCESS)
 		return FSP_NOT_FOUND;
 
 	return do_silicon_init(&fsps_hdr);



More information about the coreboot-gerrit mailing list