soc/amd: Add amdfw.rom in coreboot.pre
This change ensures that amdfw.rom binary containing metadata hash anchor is added before any file is added to CBFS. This will allow to verify all the CBFS files that are not excluded from verification. BUG=b:227809919 TEST=Build and boot to OS in Skyrim with CBFS verification enabled using x86 and PSP verstages. Change-Id: Id4d1a2d8b145cbbbf2da27aa73b296c9c8a65209 Signed-off-by: Karthikeyan Ramasubramanian <kramasub@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/66943 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Raul Rangel <rrangel@chromium.org>
This commit is contained in:
parent
da5d0251f5
commit
8d66fb1a70
|
@ -274,11 +274,6 @@ $(obj)/amdfw_b.rom: $(obj)/amdfw.rom
|
||||||
--output $@
|
--output $@
|
||||||
|
|
||||||
|
|
||||||
cbfs-files-y += apu/amdfw
|
|
||||||
apu/amdfw-file := $(obj)/amdfw.rom
|
|
||||||
apu/amdfw-position := $(CEZANNE_FWM_POSITION)
|
|
||||||
apu/amdfw-type := raw
|
|
||||||
|
|
||||||
ifeq ($(CONFIG_VBOOT_SLOTS_RW_A)$(CONFIG_VBOOT_STARTS_BEFORE_BOOTBLOCK),yy)
|
ifeq ($(CONFIG_VBOOT_SLOTS_RW_A)$(CONFIG_VBOOT_STARTS_BEFORE_BOOTBLOCK),yy)
|
||||||
cbfs-files-y += apu/amdfw_a
|
cbfs-files-y += apu/amdfw_a
|
||||||
apu/amdfw_a-file := $(obj)/amdfw_a.rom
|
apu/amdfw_a-file := $(obj)/amdfw_a.rom
|
||||||
|
|
|
@ -14,6 +14,18 @@ DEP_FILES= $(patsubst %,$(FIRMWARE_LOCATION)/%, $(shell sed -e /^$(POUND_SIGN)/d
|
||||||
|
|
||||||
amd_microcode_bins += $(wildcard ${FIRMWARE_LOCATION}/*UcodePatch_*.bin)
|
amd_microcode_bins += $(wildcard ${FIRMWARE_LOCATION}/*UcodePatch_*.bin)
|
||||||
|
|
||||||
|
ifeq ($(CONFIG_RESET_VECTOR_IN_RAM),y)
|
||||||
|
$(objcbfs)/bootblock.bin: $(obj)/amdfw.rom
|
||||||
|
cp $< $@
|
||||||
|
|
||||||
|
add_bootblock = \
|
||||||
|
$(CBFSTOOL) $(1) add -f $(2) -n apu/amdfw -t amdfw \
|
||||||
|
-b $(call int-add, \
|
||||||
|
$(call int-subtract, 0xffffffff \
|
||||||
|
$(call int-shift-left, \
|
||||||
|
0x80000 $(CONFIG_AMD_FWM_POSITION_INDEX))) 0x20000 1)
|
||||||
|
endif # ifeq ($(CONFIG_RESET_VECTOR_IN_RAM),y)
|
||||||
|
|
||||||
ifeq ($(CONFIG_VBOOT_GSCVD),y)
|
ifeq ($(CONFIG_VBOOT_GSCVD),y)
|
||||||
build_complete:: $(AMDFWREAD)
|
build_complete:: $(AMDFWREAD)
|
||||||
endif # ifeq ($(CONFIG_VBOOT_GSCVD),y)
|
endif # ifeq ($(CONFIG_VBOOT_GSCVD),y)
|
||||||
|
|
|
@ -296,11 +296,6 @@ $(obj)/amdfw_b.rom: $(obj)/amdfw.rom
|
||||||
--output $@
|
--output $@
|
||||||
|
|
||||||
|
|
||||||
cbfs-files-y += apu/amdfw
|
|
||||||
apu/amdfw-file := $(obj)/amdfw.rom
|
|
||||||
apu/amdfw-position := $(MENDOCINO_FWM_POSITION)
|
|
||||||
apu/amdfw-type := raw
|
|
||||||
|
|
||||||
ifeq ($(CONFIG_VBOOT_SLOTS_RW_A)$(CONFIG_VBOOT_STARTS_BEFORE_BOOTBLOCK),yy)
|
ifeq ($(CONFIG_VBOOT_SLOTS_RW_A)$(CONFIG_VBOOT_STARTS_BEFORE_BOOTBLOCK),yy)
|
||||||
cbfs-files-y += apu/amdfw_a
|
cbfs-files-y += apu/amdfw_a
|
||||||
apu/amdfw_a-file := $(obj)/amdfw_a.rom
|
apu/amdfw_a-file := $(obj)/amdfw_a.rom
|
||||||
|
|
|
@ -298,11 +298,6 @@ $(obj)/amdfw_b.rom: $(obj)/amdfw.rom
|
||||||
--output $@
|
--output $@
|
||||||
|
|
||||||
|
|
||||||
cbfs-files-y += apu/amdfw
|
|
||||||
apu/amdfw-file := $(obj)/amdfw.rom
|
|
||||||
apu/amdfw-position := $(MORGANA_FWM_POSITION)
|
|
||||||
apu/amdfw-type := raw
|
|
||||||
|
|
||||||
ifeq ($(CONFIG_VBOOT_SLOTS_RW_AB)$(CONFIG_VBOOT_STARTS_BEFORE_BOOTBLOCK),yy)
|
ifeq ($(CONFIG_VBOOT_SLOTS_RW_AB)$(CONFIG_VBOOT_STARTS_BEFORE_BOOTBLOCK),yy)
|
||||||
cbfs-files-y += apu/amdfw_a
|
cbfs-files-y += apu/amdfw_a
|
||||||
apu/amdfw_a-file := $(obj)/amdfw_a.rom
|
apu/amdfw_a-file := $(obj)/amdfw_a.rom
|
||||||
|
|
|
@ -286,11 +286,6 @@ $(obj)/amdfw_b.rom: $(obj)/amdfw.rom
|
||||||
--anywhere \
|
--anywhere \
|
||||||
--output $@
|
--output $@
|
||||||
|
|
||||||
cbfs-files-y += apu/amdfw
|
|
||||||
apu/amdfw-file := $(obj)/amdfw.rom
|
|
||||||
apu/amdfw-position := $(PICASSO_FWM_POSITION)
|
|
||||||
apu/amdfw-type := raw
|
|
||||||
|
|
||||||
ifeq ($(CONFIG_VBOOT_SLOTS_RW_A)$(CONFIG_VBOOT_STARTS_BEFORE_BOOTBLOCK),yy)
|
ifeq ($(CONFIG_VBOOT_SLOTS_RW_A)$(CONFIG_VBOOT_STARTS_BEFORE_BOOTBLOCK),yy)
|
||||||
cbfs-files-y += apu/amdfw_a
|
cbfs-files-y += apu/amdfw_a
|
||||||
apu/amdfw_a-file := $(obj)/amdfw_a.rom
|
apu/amdfw_a-file := $(obj)/amdfw_a.rom
|
||||||
|
|
Loading…
Reference in New Issue