[PATCH v4 10/16] arm: dts: Introduce j784s4 u-boot dts files

[Jerome Forissier]

Hello Apurva,

On 10/1/23 18:55, Apurva Nandan wrote:
> Introduce the base dts files needed for u-boot or to augment the linux
> dtbs for use in the u-boot-spl and u-boot binaries.
> 
> Signed-off-by: Hari Nagalla <hnagalla at ti.com>
> [ add binman and ddr dtsi files ]
> Signed-off-by: Neha Malcom Francis <n-francis at ti.com>
> Signed-off-by: Apurva Nandan <a-nandan at ti.com>
> Signed-off-by: Dasnavis Sabiya <sabiya.d at ti.com>
> ---
>  arch/arm/dts/Makefile                        |    2 +
>  arch/arm/dts/k3-j784s4-binman.dtsi           |  505 +
>  arch/arm/dts/k3-j784s4-ddr-evm-lp4-4266.dtsi | 8757 +++++++++++++++++
>  arch/arm/dts/k3-j784s4-ddr.dtsi              | 8858 ++++++++++++++++++
>  arch/arm/dts/k3-j784s4-evm-u-boot.dtsi       |   25 +
>  arch/arm/dts/k3-j784s4-r5-evm.dts            |  105 +
>  6 files changed, 18252 insertions(+)
>  create mode 100644 arch/arm/dts/k3-j784s4-binman.dtsi
>  create mode 100644 arch/arm/dts/k3-j784s4-ddr-evm-lp4-4266.dtsi
>  create mode 100644 arch/arm/dts/k3-j784s4-ddr.dtsi
>  create mode 100644 arch/arm/dts/k3-j784s4-evm-u-boot.dtsi
>  create mode 100644 arch/arm/dts/k3-j784s4-r5-evm.dts
> 
> diff --git a/arch/arm/dts/Makefile b/arch/arm/dts/Makefile
> index 85fd5b1157..21cfebff58 100644
> --- a/arch/arm/dts/Makefile
> +++ b/arch/arm/dts/Makefile
> @@ -1327,6 +1327,8 @@ dtb-$(CONFIG_SOC_K3_J721S2) += k3-am68-sk-base-board.dtb\
>  			       k3-am68-sk-r5-base-board.dtb\
>  			       k3-j721s2-common-proc-board.dtb\
>  			       k3-j721s2-r5-common-proc-board.dtb
> +dtb-$(CONFIG_SOC_K3_J784S4) += k3-j784s4-evm.dtb\
> +			       k3-j784s4-r5-evm.dtb
>  dtb-$(CONFIG_SOC_K3_AM642) += k3-am642-evm.dtb \
>  			      k3-am642-r5-evm.dtb \
>  			      k3-am642-sk.dtb \
> diff --git a/arch/arm/dts/k3-j784s4-binman.dtsi b/arch/arm/dts/k3-j784s4-binman.dtsi
> new file mode 100644
> index 0000000000..f9b191424c
> --- /dev/null
> +++ b/arch/arm/dts/k3-j784s4-binman.dtsi
> @@ -0,0 +1,505 @@
> +// SPDX-License-Identifier: GPL-2.0-only
> +/*
> + * Copyright (C) 2023 Texas Instruments Incorporated - https://www.ti.com/
> + */

<snip>

> +&binman {
> +	ti-spl_unsigned {
> +		filename = "tispl.bin_unsigned";
> +		pad-byte = <0xff>;
> +
> +		fit {
> +			description = "Configuration to load ATF and SPL";
> +			#address-cells = <1>;
> +
> +			images {
> +
> +				atf {
> +					description = "ARM Trusted Firmware";
> +					type = "firmware";
> +					arch = "arm64";
> +					compression = "none";
> +					os = "arm-trusted-firmware";
> +					load = <CONFIG_K3_ATF_LOAD_ADDR>;
> +					entry = <CONFIG_K3_ATF_LOAD_ADDR>;
> +
> +					atf-bl31 {
> +						filename = "bl31.bin";
> +					};
> +				};
> +
> +				tee {
> +					description = "OPTEE";

The official spelling is "OP-TEE".

> +					type = "tee";
> +					arch = "arm64";
> +					compression = "none";
> +					os = "tee";
> +					load = <0x9e800000>;
> +					entry = <0x9e800000>;
> +
> +					tee-os {
> +						filename = "tee-pager_v2.bin";

Since OP-TEE 3.21.0 this could/should be replaced with tee-raw.bin. Although tee-pager_v2.bin is still produced unconditionally (even when the "pager" feature is disabled in OP-TEE) it might not be the case in the future, and the name is somewhat misleading when pager is disabled anyways (which is clearly the case here since tee-pageable_v2.bin is not used).

Thanks,
-- 
Jerome