[coreboot-gerrit] New patch to review for coreboot: drivers/intel/fsp1_1: only set a base address for FSP in COREBOOT CBFS

Aaron Durbin (adurbin@chromium.org) gerrit at coreboot.org
Fri May 6 23:07:59 CEST 2016


Aaron Durbin (adurbin at chromium.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14641

-gerrit

commit 4a58cf7583fed42cf25ed4bb3f74ec2807e53760
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Fri May 6 16:06:36 2016 -0500

    drivers/intel/fsp1_1: only set a base address for FSP in COREBOOT CBFS
    
    The -b FSP_LOC argument to cbfstool is only valid for the COREBOOT
    CBFS. Don't pass that value for all other CBFS regions.
    
    Change-Id: Ib5321e7a7dbee8d26eb558933c8ce3fea50b11fe
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
---
 src/drivers/intel/fsp1_1/Makefile.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/drivers/intel/fsp1_1/Makefile.inc b/src/drivers/intel/fsp1_1/Makefile.inc
index 3f214cb..5c84d79 100644
--- a/src/drivers/intel/fsp1_1/Makefile.inc
+++ b/src/drivers/intel/fsp1_1/Makefile.inc
@@ -52,8 +52,8 @@ vbt.bin-type := optionrom
 ifeq ($(CONFIG_HAVE_FSP_BIN),y)
 cbfs-files-y += fsp.bin
 fsp.bin-file := $(call strip_quotes,$(CONFIG_FSP_FILE))
-fsp.bin-position := $(CONFIG_FSP_LOC)
 fsp.bin-type := fsp
+fsp.bin-COREBOOT-cbfstool-opts := -b $(CONFIG_FSP_LOC)
 endif
 
 endif



More information about the coreboot-gerrit mailing list