Update hex values to CBFS binary name types in Makefiles
These binaries were being added to CBFS using hexadecimal values instead of the CBFS binary type names. The same value was being used in different places for different things. For example, the value 0xAB is used for SPDs, MRC & FSP binaries. This patch uses CBFS type names instead of hex values everywhere a hex value was previously used. Change-Id: Id5ac74c3095eb02a2b39d25104a25933304a8389 Signed-off-by: Martin Roth <gaumless@gmail.com> Reviewed-on: http://review.coreboot.org/8978 Tested-by: build bot (Jenkins) Reviewed-by: Marc Jones <marc.jones@se-eng.com> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-by: Aaron Durbin <adurbin@google.com>
This commit is contained in:
parent
dde307c01a
commit
72a8e5e751
|
@ -33,7 +33,7 @@ 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 := 0xab
|
||||
fsp.bin-type := fsp
|
||||
endif
|
||||
|
||||
ifeq ($(CONFIG_ENABLE_MRC_CACHE),y)
|
||||
|
@ -45,5 +45,5 @@ $(obj)/mrc.cache:
|
|||
cbfs-files-y += mrc.cache
|
||||
mrc.cache-file := $(obj)/mrc.cache
|
||||
mrc.cache-position := $(CONFIG_MRC_CACHE_LOC)
|
||||
mrc.cache-type := 0xac
|
||||
mrc.cache-type := mrc_cache
|
||||
endif
|
||||
|
|
|
@ -51,4 +51,4 @@ $(SPD_BIN): $(SPD_DEPS)
|
|||
|
||||
cbfs-files-y += spd.bin
|
||||
spd.bin-file := $(SPD_BIN)
|
||||
spd.bin-type := 0xab
|
||||
spd.bin-type := spd
|
||||
|
|
|
@ -44,4 +44,4 @@ $(SPD_BIN): $(SPD_DEPS)
|
|||
|
||||
cbfs-files-y += spd.bin
|
||||
spd.bin-file := $(SPD_BIN)
|
||||
spd.bin-type := 0xab
|
||||
spd.bin-type := spd
|
||||
|
|
|
@ -42,4 +42,4 @@ $(SPD_BIN): $(SPD_DEPS)
|
|||
|
||||
cbfs-files-y += spd.bin
|
||||
spd.bin-file := $(SPD_BIN)
|
||||
spd.bin-type := 0xab
|
||||
spd.bin-type := spd
|
||||
|
|
|
@ -50,4 +50,4 @@ $(SPD_BIN): $(SPD_DEPS)
|
|||
|
||||
cbfs-files-y += spd.bin
|
||||
spd.bin-file := $(SPD_BIN)
|
||||
spd.bin-type := 0xab
|
||||
spd.bin-type := spd
|
||||
|
|
|
@ -41,4 +41,4 @@ $(SPD_BIN): $(SPD_DEPS)
|
|||
|
||||
cbfs-files-y += spd.bin
|
||||
spd.bin-file := $(SPD_BIN)
|
||||
spd.bin-type := 0xab
|
||||
spd.bin-type := spd
|
||||
|
|
|
@ -49,4 +49,4 @@ $(SPD_BIN): $(SPD_DEPS)
|
|||
|
||||
cbfs-files-y += spd.bin
|
||||
spd.bin-file := $(SPD_BIN)
|
||||
spd.bin-type := 0xab
|
||||
spd.bin-type := spd
|
||||
|
|
|
@ -46,4 +46,4 @@ $(SPD_BIN): $(SPD_DEPS)
|
|||
|
||||
cbfs-files-y += spd.bin
|
||||
spd.bin-file := $(SPD_BIN)
|
||||
spd.bin-type := 0xab
|
||||
spd.bin-type := spd
|
||||
|
|
|
@ -51,4 +51,4 @@ $(SPD_BIN): $(SPD_DEPS)
|
|||
|
||||
cbfs-files-y += spd.bin
|
||||
spd.bin-file := $(SPD_BIN)
|
||||
spd.bin-type := 0xab
|
||||
spd.bin-type := spd
|
||||
|
|
|
@ -43,4 +43,4 @@ $(SPD_BIN): $(SPD_DEPS)
|
|||
|
||||
cbfs-files-y += spd.bin
|
||||
spd.bin-file := $(SPD_BIN)
|
||||
spd.bin-type := 0xab
|
||||
spd.bin-type := spd
|
||||
|
|
|
@ -53,4 +53,4 @@ $(SPD_BIN): $(SPD_DEPS)
|
|||
|
||||
cbfs-files-y += spd.bin
|
||||
spd.bin-file := $(SPD_BIN)
|
||||
spd.bin-type := 0xab
|
||||
spd.bin-type := spd
|
||||
|
|
|
@ -30,4 +30,4 @@ $(SPD_BIN):
|
|||
|
||||
cbfs-files-y += spd.bin
|
||||
spd.bin-file := $(SPD_BIN)
|
||||
spd.bin-type := 0xab
|
||||
spd.bin-type := spd
|
||||
|
|
|
@ -27,25 +27,25 @@ ramstage-y += ptn3460.c
|
|||
|
||||
cbfs-files-y += hwinfo.hex
|
||||
hwinfo.hex-file := hwinfo.hex
|
||||
hwinfo.hex-type := 0x50
|
||||
hwinfo.hex-type := raw
|
||||
hwinfo.hex-align := 0x1000
|
||||
|
||||
cbfs-files-y += version.hex
|
||||
version.hex-file := version.hex
|
||||
version.hex-type := 0x50
|
||||
version.hex-type := raw
|
||||
|
||||
cbfs-files-y += hwinfo10.hex
|
||||
hwinfo10.hex-file := hwinfo10.hex
|
||||
hwinfo10.hex-type := 0x50
|
||||
hwinfo10.hex-type := raw
|
||||
|
||||
cbfs-files-y += hwinfo12.hex
|
||||
hwinfo12.hex-file := hwinfo12.hex
|
||||
hwinfo12.hex-type := 0x50
|
||||
hwinfo12.hex-type := raw
|
||||
|
||||
cbfs-files-y += hwinfo15.hex
|
||||
hwinfo15.hex-file := hwinfo15.hex
|
||||
hwinfo15.hex-type := 0x50
|
||||
hwinfo15.hex-type := raw
|
||||
|
||||
cbfs-files-y += hwinfo19.hex
|
||||
hwinfo19.hex-file := hwinfo19.hex
|
||||
hwinfo19.hex-type := 0x50
|
||||
hwinfo19.hex-type := raw
|
||||
|
|
|
@ -38,7 +38,7 @@ smm-$(CONFIG_HAVE_SMI_HANDLER) += finalize.c
|
|||
cbfs-files-$(CONFIG_HAVE_MRC) += mrc.bin
|
||||
mrc.bin-file := $(call strip_quotes,$(CONFIG_MRC_FILE))
|
||||
mrc.bin-position := 0xfffa0000
|
||||
mrc.bin-type := 0xab
|
||||
mrc.bin-type := mrc
|
||||
|
||||
ifneq ($(CONFIG_CHROMEOS),y)
|
||||
$(obj)/mrc.cache: $(obj)/config.h
|
||||
|
@ -49,6 +49,6 @@ $(obj)/mrc.cache: $(obj)/config.h
|
|||
cbfs-files-y += mrc.cache
|
||||
mrc.cache-file := $(obj)/mrc.cache
|
||||
mrc.cache-position := 0xfffe0000
|
||||
mrc.cache-type := 0xac
|
||||
mrc.cache-type := mrc_cache
|
||||
endif
|
||||
|
||||
|
|
|
@ -40,5 +40,5 @@ $(obj)/mrc.cache:
|
|||
cbfs-files-y += mrc.cache
|
||||
mrc.cache-file := $(obj)/mrc.cache
|
||||
mrc.cache-position := 0xfffe0000
|
||||
mrc.cache-type := 0xac
|
||||
mrc.cache-type := mrc_cache
|
||||
|
||||
|
|
|
@ -47,7 +47,7 @@ smm-$(CONFIG_HAVE_SMI_HANDLER) += finalize.c
|
|||
cbfs-files-$(CONFIG_HAVE_MRC) += mrc.bin
|
||||
mrc.bin-file := $(call strip_quotes,$(CONFIG_MRC_FILE))
|
||||
mrc.bin-position := 0xfffa0000
|
||||
mrc.bin-type := 0xab
|
||||
mrc.bin-type := mrc
|
||||
|
||||
ifneq ($(CONFIG_CHROMEOS),y)
|
||||
$(obj)/mrc.cache: $(obj)/config.h
|
||||
|
@ -62,6 +62,6 @@ mrc-cache-position-$(CONFIG_NORTHBRIDGE_INTEL_SANDYBRIDGE) := 0xfffd0000
|
|||
mrc-cache-position-$(CONFIG_NORTHBRIDGE_INTEL_IVYBRIDGE_NATIVE) := 0xfffe0000
|
||||
mrc-cache-position-$(CONFIG_NORTHBRIDGE_INTEL_SANDYBRIDGE_NATIVE) := 0xfffe0000
|
||||
mrc.cache-position := $(mrc-cache-position-y)
|
||||
mrc.cache-type := 0xac
|
||||
mrc.cache-type := mrc_cache
|
||||
endif
|
||||
|
||||
|
|
|
@ -98,6 +98,6 @@ mrcelfentry = $(shell $(READELF_x86_32) -h -W $(CONFIG_MRC_FILE) | grep 'Entry p
|
|||
cbfs-files-$(CONFIG_HAVE_MRC) += mrc.bin
|
||||
mrc.bin-file := $(call strip_quotes,$(CONFIG_MRC_FILE))
|
||||
mrc.bin-position := $(if $(findstring elf,$(CONFIG_MRC_FILE)),$(shell printf "0x%x" $$(( $(mrcelfentry) - $(mrcelfoffset) )) ),$(CONFIG_MRC_BIN_ADDRESS))
|
||||
mrc.bin-type := 0xab
|
||||
mrc.bin-type := mrc
|
||||
|
||||
PHONY += baytrail_add_me
|
||||
|
|
|
@ -127,4 +127,4 @@ mrcelfentry = $(shell $(READELF_x86_32) -h -W $(CONFIG_MRC_FILE) | grep 'Entry p
|
|||
cbfs-files-$(CONFIG_HAVE_MRC) += mrc.bin
|
||||
mrc.bin-file := $(call strip_quotes,$(CONFIG_MRC_FILE))
|
||||
mrc.bin-position := $(if $(findstring elf,$(CONFIG_MRC_FILE)),$(shell printf "0x%x" $$(( $(mrcelfentry) - $(mrcelfoffset) )) ),$(CONFIG_MRC_BIN_ADDRESS))
|
||||
mrc.bin-type := 0xab
|
||||
mrc.bin-type := mrc
|
||||
|
|
|
@ -123,4 +123,4 @@ MTS_FILE = $(MTS_DIR)/mts_cr.bin
|
|||
MTS_FILE_CBFS = mts
|
||||
cbfs-files-y += $(MTS_FILE_CBFS)
|
||||
$(MTS_FILE_CBFS)-file := $(MTS_FILE)
|
||||
$(MTS_FILE_CBFS)-type := 0x50
|
||||
$(MTS_FILE_CBFS)-type := raw
|
||||
|
|
|
@ -37,5 +37,5 @@ smm-$(CONFIG_HAVE_SMI_HANDLER) += smihandler.c
|
|||
# We don't ship that, but booting without it is bound to fail
|
||||
cbfs-files-$(CONFIG_HAVE_CMC) += cmc.bin
|
||||
cmc.bin-file := $(call strip_quotes,$(CONFIG_CMC_FILE))
|
||||
cmc.bin-type := 0xaa
|
||||
cmc.bin-type := raw
|
||||
cmc.bin-position := 0xfffd0000
|
||||
|
|
Loading…
Reference in New Issue