drivers/intel/fsp2_0: Use strip_quotes for cbfs filenames
The quotes were not stripped for the cbfs filenames of the FSP components. This is causing problems when the regions-for-file macro is executed (when VBOOT is enabled and the files should be filtered). BUG=N/A TEST=build Change-Id: I14267502cfab5308d3874a0c0fd18a71b08bb9f8 Signed-off-by: Wim Vervoorn <wvervoorn@eltan.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/36548 Reviewed-by: Frans Hendriks <fhendriks@eltan.com> Reviewed-by: Nico Huber <nico.h@gmx.de> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
parent
1058dd84f0
commit
8fc523e313
|
@ -42,25 +42,29 @@ postcar-y += hand_off_block.c
|
|||
|
||||
CPPFLAGS_common += -I$(src)/drivers/intel/fsp2_0/include
|
||||
|
||||
FSP_T_CBFS = $(call strip_quotes,$(CONFIG_FSP_T_CBFS))
|
||||
FSP_M_CBFS = $(call strip_quotes,$(CONFIG_FSP_M_CBFS))
|
||||
FSP_S_CBFS = $(call strip_quotes,$(CONFIG_FSP_S_CBFS))
|
||||
|
||||
# Add FSP blobs into cbfs. SoC code may supply additional options with
|
||||
# -options, e.g --xip or -b
|
||||
cbfs-files-$(CONFIG_FSP_CAR) += $(CONFIG_FSP_T_CBFS)
|
||||
$(CONFIG_FSP_T_CBFS)-file := $(call strip_quotes,$(CONFIG_FSP_T_FILE))
|
||||
$(CONFIG_FSP_T_CBFS)-type := fsp
|
||||
cbfs-files-$(CONFIG_FSP_CAR) += $(FSP_T_CBFS)
|
||||
$(FSP_T_CBFS)-file := $(call strip_quotes,$(CONFIG_FSP_T_FILE))
|
||||
$(FSP_T_CBFS)-type := fsp
|
||||
ifeq ($(CONFIG_FSP_T_XIP),y)
|
||||
$(CONFIG_FSP_T_CBFS)-options := --xip $(TXTIBB)
|
||||
$(FSP_T_CBFS)-options := --xip $(TXTIBB)
|
||||
endif
|
||||
|
||||
cbfs-files-$(CONFIG_ADD_FSP_BINARIES) += $(CONFIG_FSP_M_CBFS)
|
||||
$(CONFIG_FSP_M_CBFS)-file := $(call strip_quotes,$(CONFIG_FSP_M_FILE))
|
||||
$(CONFIG_FSP_M_CBFS)-type := fsp
|
||||
cbfs-files-$(CONFIG_ADD_FSP_BINARIES) += $(FSP_M_CBFS)
|
||||
$(FSP_M_CBFS)-file := $(call strip_quotes,$(CONFIG_FSP_M_FILE))
|
||||
$(FSP_M_CBFS)-type := fsp
|
||||
ifeq ($(CONFIG_FSP_M_XIP),y)
|
||||
$(CONFIG_FSP_M_CBFS)-options := --xip $(TXTIBB)
|
||||
$(FSP_M_CBFS)-options := --xip $(TXTIBB)
|
||||
endif
|
||||
|
||||
cbfs-files-$(CONFIG_ADD_FSP_BINARIES) += $(CONFIG_FSP_S_CBFS)
|
||||
$(CONFIG_FSP_S_CBFS)-file := $(call strip_quotes,$(CONFIG_FSP_S_FILE))
|
||||
$(CONFIG_FSP_S_CBFS)-type := fsp
|
||||
cbfs-files-$(CONFIG_ADD_FSP_BINARIES) += $(FSP_S_CBFS)
|
||||
$(FSP_S_CBFS)-file := $(call strip_quotes,$(CONFIG_FSP_S_FILE))
|
||||
$(FSP_S_CBFS)-type := fsp
|
||||
|
||||
ifeq ($(CONFIG_FSP_USE_REPO),y)
|
||||
$(obj)/Fsp_M.fd: $(call strip_quotes,$(CONFIG_FSP_FD_PATH))
|
||||
|
|
|
@ -108,7 +108,7 @@ endif
|
|||
CPPFLAGS_common += -I$(src)/soc/intel/apollolake/include
|
||||
|
||||
# Since FSP-M runs in CAR we need to relocate it to a specific address
|
||||
$(CONFIG_FSP_M_CBFS)-options := -b $(CONFIG_FSP_M_ADDR)
|
||||
$(FSP_M_CBFS)-options := -b $(CONFIG_FSP_M_ADDR)
|
||||
|
||||
# Handle GLK paging requirements
|
||||
ifeq ($(CONFIG_PAGING_IN_CACHE_AS_RAM),y)
|
||||
|
|
|
@ -90,8 +90,8 @@ CPPFLAGS_common += -I$(src)/vendorcode/intel/fsp/fsp2_0/denverton_ns
|
|||
|
||||
##Set FSP binary blobs memory location
|
||||
|
||||
$(CONFIG_FSP_T_CBFS)-options := -b $(CONFIG_FSP_T_ADDR) --xip
|
||||
$(CONFIG_FSP_M_CBFS)-options := -b $(CONFIG_FSP_M_ADDR) --xip
|
||||
$(CONFIG_FSP_S_CBFS)-options := -b $(CONFIG_FSP_S_ADDR) --xip
|
||||
$(FSP_T_CBFS)-options := -b $(CONFIG_FSP_T_ADDR) --xip
|
||||
$(FSP_M_CBFS)-options := -b $(CONFIG_FSP_M_ADDR) --xip
|
||||
$(FSP_S_CBFS)-options := -b $(CONFIG_FSP_S_ADDR) --xip
|
||||
|
||||
endif ## CONFIG_SOC_INTEL_DENVERTON_NS
|
||||
|
|
|
@ -71,7 +71,7 @@ CPPFLAGS_common += -I$(src)/soc/intel/quark/include/soc/fsp
|
|||
CPPFLAGS_common += -I3rdparty/blobs/soc/intel/quark
|
||||
|
||||
# Since FSP-M runs in CAR we need to relocate it to a specific address
|
||||
$(CONFIG_FSP_M_CBFS)-options := -b $(CONFIG_FSP_ESRAM_LOC)
|
||||
$(FSP_M_CBFS)-options := -b $(CONFIG_FSP_ESRAM_LOC)
|
||||
|
||||
# Add the FSP binary to the CBFS image
|
||||
cbfs-files-$(CONFIG_ADD_FSP_RAW_BIN) += fsp.bin
|
||||
|
|
Loading…
Reference in New Issue