[PATCH 19/29] rockchip: Add TPL alternatives

[Jonas Karlman]

Hi Simon,

On 2025-02-05 02:55, Simon Glass wrote:
> The TPL image must be built for each board we need to support. It is the
> only part of the image which is board-specific.

Is this building for multiple targets required for A, B or recovery?,
the main topic for this series? If not it should be added later in the
series or possible in a follow-up series.

> 
> This helps to save space and reduce the size of TPL, so that it can fit
> within the internal 192K IRAM.

This should probably mention something about RK3399 here, Rockchip SoCs
have different amount of usable SRAM.

Regards,
Jonas

> 
> As with other phases, the TPL devicetree is run through fdtgrep to
> remove unwanted nodes and properties and reduce its size.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> 
>  arch/arm/dts/rockchip-u-boot.dtsi | 42 +++++++++++++++++++++----------
>  1 file changed, 29 insertions(+), 13 deletions(-)
> 
> diff --git a/arch/arm/dts/rockchip-u-boot.dtsi b/arch/arm/dts/rockchip-u-boot.dtsi
> index 0611533a310..f6348c532a4 100644
> --- a/arch/arm/dts/rockchip-u-boot.dtsi
> +++ b/arch/arm/dts/rockchip-u-boot.dtsi
> @@ -254,23 +254,23 @@
>  		pad-byte = <0xff>;
>  		skip-at-start = <0x8000>;
>  
> -		mkimage {
> -			filename = "idbloader.img";
> -			args = "-n", CONFIG_SYS_SOC, "-T", "rksd";
> -			multiple-data-files;
> +#ifdef CONFIG_VPL
> +		alternates-fdt {
> +			fdt-list-dir = "dts/upstream/src/arm64/rockchip";
> +			filename-pattern = "alt-NAME.bin";
> +			fdt-phase = "tpl";
>  
> -#ifdef CONFIG_ROCKCHIP_EXTERNAL_TPL
> -			rockchip-tpl {
> -			};
> -#elif defined(CONFIG_TPL)
> -			u-boot-tpl {
> -			};
> -#endif
> -			u-boot-spl {
> +			mkimage {
> +				filename = "idbloader.img";
> +				args = "-n", CONFIG_SYS_SOC, "-T", "rksd";
> +				multiple-data-files;
> +
> +				u-boot-tpl {
> +					symbols-base = <0>;
> +				};
>  			};
>  		};
>  
> -#ifdef CONFIG_VPL
>  		vpl {
>  			type = "fit";
>  			offset = <(CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR * 0x200 + 0x8000)>;
> @@ -361,6 +361,22 @@
>  # endif /* VBE method */
>  #else /* !VPL */
>  
> +		mkimage {
> +			filename = "idbloader.img";
> +			args = "-n", CONFIG_SYS_SOC, "-T", "rksd";
> +			multiple-data-files;
> +
> +#ifdef CONFIG_ROCKCHIP_EXTERNAL_TPL
> +			rockchip-tpl {
> +			};
> +#elif defined(CONFIG_TPL)
> +			u-boot-tpl {
> +			};
> +#endif
> +			u-boot-spl {
> +			};
> +		};
> +
>  #ifdef HAS_FIT
>  		fit {
>  			offset = <CONFIG_SPL_PAD_TO>;