Re: [Buildroot] [PATCH 11/21] boot/ti-k3-r5-loader: install tiboot3.bin and sysfw.itb to BINARIES_DIR

[Alexander Sverdlin <alexander.sverdlin@gmail.com>]

Hi Romain,

On Tue, 2024-02-13 at 18:28 +0100, Romain Naour wrote:
> U-boot for k3 r5 core (ti-k3-r5-loader) can install different target
> image (tiboot3*.bin) depending on the k3 SoC variant using different
> boot flow [1]:
> 
> - Split binary with a combined firmware: (eg: AM65)
> - Combined binary with a combined firmware: (eg: AM64)
> - Split binary with a split firmware: (eg: J721E)
> - Combined binary with a split firmware: (eg: AM62)
> 
> K3 SoC using Split Binary Boot Flow also requires sysfw-*.itb file
> to boot (eg: am65, j721e). Only tiboot3*.bin is needed for other
> boot flow.
> 
> From [1]:
> "It's important to rename the generated tiboot3.bin and sysfw.itb
> to match exactly tiboot3.bin and sysfw.itb as ROM and the wakeup
> UBoot SPL will only look for and load the files with these names."
> 
> So boards using HS (High Secure) K3 SoC variant must update the
> tiboot3.bin (and sysfw.itb) in their post-build script.
> 
> ti-k3-r5-loader must install any tiboot3*.bin (and sysfw*.itb)
> to BINARIES_DIR let the user which one should be used in a
> post-build script.
> 
> See the list of all tiboot3*.bin and sysfw*.itb file variant found in meta-ti:


[]

> [1] https://source.denx.de/u-boot/u-boot/-/blob/v2024.01/doc/board/ti/k3.rst
> 
> Signed-off-by: Romain Naour <romain.naour@smile.fr>

Reviewed-by: Alexander Sverdlin <alexander.sverdlin@gmail.com>

> ---
>  boot/ti-k3-r5-loader/ti-k3-r5-loader.mk | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
> index 8f86e0fb07..b3c756dc47 100644
> --- a/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
> +++ b/boot/ti-k3-r5-loader/ti-k3-r5-loader.mk
> @@ -78,8 +78,17 @@ define TI_K3_R5_LOADER_BUILD_CMDS
>  	$(TARGET_CONFIGURE_OPTS) $(TI_K3_R5_LOADER_MAKE) -C $(@D) $(TI_K3_R5_LOADER_MAKE_OPTS)
>  endef
>  
> +# sysfw*.itb are only generated for Split binary based Boot Flow (eg: am65, j721e)
>  define TI_K3_R5_LOADER_INSTALL_IMAGES_CMDS
>  	cp $(@D)/spl/u-boot-spl.bin $(BINARIES_DIR)/r5-u-boot-spl.bin
> +	for f in $(@D)/tiboot3*.bin ; do \
> +		$(INSTALL) -D -m 0644 $$f $(BINARIES_DIR)/`basename $$f` || exit 1; \
> +	done
> +	for f in $(@D)/sysfw*.itb ; do \
> +		if test -e $$f ; then \
> +			$(INSTALL) -D -m 0644 $$f $(BINARIES_DIR)/`basename $$f` || exit 1; \
> +		fi ; \
> +	done
>  endef
>  
>  $(eval $(kconfig-package))

-- 
Alexander Sverdlin.

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot