soc/amd/*/Makefile.inc: Do some cosmetics
The first target for the add_intermediate targets is always $(obj)/coreboot.pre. Change-Id: Iea2322ca1abd43900f3631b7965f07fed4235ca0 Signed-off-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-on: https://review.coreboot.org/c/coreboot/+/56117 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Raul Rangel <rrangel@chromium.org> Reviewed-by: Martin L Roth <gaumless@tutanota.com>
This commit is contained in:
@@ -13,7 +13,7 @@ APCB_SOURCES_RECOVERY = $(MAINBOARD_BLOBS_DIR)/APCB_CZN_D4_DefaultRecovery.bin
|
|||||||
|
|
||||||
ifeq ($(CONFIG_MAJOLICA_HAVE_MCHP_FW),y)
|
ifeq ($(CONFIG_MAJOLICA_HAVE_MCHP_FW),y)
|
||||||
$(call add_intermediate, add_mchp_fw)
|
$(call add_intermediate, add_mchp_fw)
|
||||||
$(CBFSTOOL) $(obj)/coreboot.pre write -r EC -f $(CONFIG_MAJOLICA_MCHP_FW_FILE) --fill-upward
|
$(CBFSTOOL) $< write -r EC -f $(CONFIG_MAJOLICA_MCHP_FW_FILE) --fill-upward
|
||||||
else
|
else
|
||||||
files_added:: warn_no_mchp
|
files_added:: warn_no_mchp
|
||||||
endif # CONFIG_MAJOLICA_HAVE_MCHP_FW
|
endif # CONFIG_MAJOLICA_HAVE_MCHP_FW
|
||||||
|
@@ -19,7 +19,7 @@ endif
|
|||||||
ifeq ($(CONFIG_MANDOLIN_HAVE_MCHP_FW),y)
|
ifeq ($(CONFIG_MANDOLIN_HAVE_MCHP_FW),y)
|
||||||
|
|
||||||
$(call add_intermediate, add_mchp_fw)
|
$(call add_intermediate, add_mchp_fw)
|
||||||
$(CBFSTOOL) $(obj)/coreboot.pre write -r EC -f $(CONFIG_MANDOLIN_MCHP_FW_FILE) --fill-upward
|
$(CBFSTOOL) $< write -r EC -f $(CONFIG_MANDOLIN_MCHP_FW_FILE) --fill-upward
|
||||||
|
|
||||||
else
|
else
|
||||||
files_added:: warn_no_mchp
|
files_added:: warn_no_mchp
|
||||||
|
@@ -171,7 +171,7 @@ $(call add_intermediate, add_amdfw, $(obj)/amdfw.rom)
|
|||||||
printf " DD Adding AMD Firmware at ROM offset 0x%x\n" \
|
printf " DD Adding AMD Firmware at ROM offset 0x%x\n" \
|
||||||
"$(STONEYRIDGE_FWM_ROM_POSITION)"
|
"$(STONEYRIDGE_FWM_ROM_POSITION)"
|
||||||
dd if=$(obj)/amdfw.rom \
|
dd if=$(obj)/amdfw.rom \
|
||||||
of=$(obj)/coreboot.pre conv=notrunc bs=1 \
|
of=$< conv=notrunc bs=1 \
|
||||||
seek=$(STONEYRIDGE_FWM_ROM_POSITION) >/dev/null 2>&1
|
seek=$(STONEYRIDGE_FWM_ROM_POSITION) >/dev/null 2>&1
|
||||||
|
|
||||||
else # ifeq ($(CONFIG_AMDFW_OUTSIDE_CBFS),y)
|
else # ifeq ($(CONFIG_AMDFW_OUTSIDE_CBFS),y)
|
||||||
|
@@ -157,7 +157,7 @@ ifeq ($(CONFIG_AMDFW_OUTSIDE_CBFS),y)
|
|||||||
$(call add_intermediate, add_amdfw, $(obj)/amdfw.rom)
|
$(call add_intermediate, add_amdfw, $(obj)/amdfw.rom)
|
||||||
printf " DD Adding AMD Firmware\n"
|
printf " DD Adding AMD Firmware\n"
|
||||||
dd if=$(obj)/amdfw.rom \
|
dd if=$(obj)/amdfw.rom \
|
||||||
of=$(obj)/coreboot.pre conv=notrunc bs=1 seek=131072 >/dev/null 2>&1
|
of=$< conv=notrunc bs=1 seek=131072 >/dev/null 2>&1
|
||||||
|
|
||||||
else # ifeq ($(CONFIG_AMDFW_OUTSIDE_CBFS),y)
|
else # ifeq ($(CONFIG_AMDFW_OUTSIDE_CBFS),y)
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user