On Sat, 2019-03-30 at 15:18 -0600, Simon Glass wrote:
Hi, On Fri, 22 Mar 2019 at 09:32, Dalon Westergreen< dalon.westergreen@linux.intel.com> wrote:
From: Dalon Westergreen dalon.westergreen@intel.com Some architectures, Stratix10, require a hex formatted spl that combinesthe spl image and dtb. This adds a target to create said hex file withand offset of SPL_TEXT_BASE. Signed-off-by: Dalon Westergreen dalon.westergreen@intel.com--- Makefile | 9 +++++++++ 1 file changed, 9 insertions(+)
Reviewed-by: Simon Glass sjg@chromium.org Please see below.
diff --git a/Makefile b/Makefileindex c52a33b403..ecba06ffce 100644--- a/Makefile+++ b/Makefile@@ -1074,6 +1074,11 @@ OBJCOPYFLAGS_u-boot-spl.hex = $(OBJCOPYFLAGS_u-boot.hex) spl/u-boot-spl.hex: spl/u-boot-spl FORCE $(call if_changed,objcopy) +OBJCOPYFLAGS_u-boot-spl-dtb.hex := -I binary -O ihex --change- address=$(CONFIG_SPL_TEXT_BASE)
Can we drop the -dtb part here? u-boot-spl includes the DTB anyway. Itis the -nodtb version which does not.
sure thing.
+spl/u-boot-spl-dtb.hex: spl/u-boot-spl-dtb.bin FORCE
$(call if_changed,objcopy)binary_size_check: u-boot-nodtb.bin FORCE @file_size=$(shell wc -c u-boot-nodtb.bin | awk '{print $$1}') ; \ map_size=$(shell cat u-boot.map | \ @@ -1643,6 +1648,10 @@ u-boot.lds: $(LDSCRIPT) prepare FORCE
spl/u-boot-spl.bin: spl/u-boot-spl @:
+spl/u-boot-spl-dtb.bin: spl/u-boot-spl
@:spl/u-boot-spl: tools prepare \ $(if $(CONFIG_OF_SEPARATE)$(CONFIG_OF_EMBED)$(CONFIG_SPL_OF_PLATDATA),dts/dt.dtb) \ $(if $(CONFIG_OF_SEPARATE)$(CONFIG_OF_EMBED)$(CONFIG_TPL_OF_PLATDATA),dts/dt.dtb) -- 2.20.1
Regards, Simon