[U-Boot] [PATCH v2 1/2] Makefile: Add target to generate hex output for combined spl and dtb
Marek Vasut
marex at denx.de
Tue Jun 4 00:00:28 UTC 2019
On 6/4/19 1:57 AM, Dalon Westergreen wrote:
> From: Dalon Westergreen <dalon.westergreen at intel.com>
>
> Some architectures, Stratix10, require a hex formatted spl that combines
> the spl image and dtb. This adds a target to create said hex file with
> and offset of SPL_TEXT_BASE.
>
> Signed-off-by: Dalon Westergreen <dalon.westergreen at intel.com>
>
[...]
> @@ -363,6 +365,11 @@ endif
> $(obj)/$(SPL_BIN).sfp: $(obj)/$(SPL_BIN).bin FORCE
> $(call if_changed,mkimage)
>
> +OBJCOPYFLAGS_$(SPL_BIN).hex := -I binary -O ihex --change-address=$(CONFIG_SPL_TEXT_BASE)
Do we really need to do it here ? The commit message is not clear why
this is needed ; I think if you link the SPl against the correct
address, this should not be needed.
> +$(obj)/$(SPL_BIN).hex: $(obj)/u-boot-spl.bin FORCE
> + $(call if_changed,objcopy)
> +
> quiet_cmd_mksunxiboot = MKSUNXI $@
> cmd_mksunxiboot = $(objtree)/tools/mksunxiboot \
> --default-dt $(CONFIG_DEFAULT_DEVICE_TREE) $< $@
> @@ -463,3 +470,4 @@ ifdef CONFIG_ARCH_K3
> tispl.bin: $(obj)/u-boot-spl-nodtb.bin $(SHRUNK_ARCH_DTB) $(SPL_ITS) FORCE
> $(call if_changed,mkfitimage)
> endif
> +
Drop this hunk
--
Best regards,
Marek Vasut
More information about the U-Boot
mailing list