Hi Simon,

On 2025-02-05 02:55, Simon Glass wrote:
> When VPL is in use, memory init happens in SPL, so there is no need to
> include the DMC device before that. Adjust the tags to save space.
> 
> Signed-off-by: Simon Glass <s...@chromium.org>
> ---
> 
>  arch/arm/dts/rk3399-u-boot.dtsi | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm/dts/rk3399-u-boot.dtsi b/arch/arm/dts/rk3399-u-boot.dtsi
> index 2bec139d833..76efd703902 100644
> --- a/arch/arm/dts/rk3399-u-boot.dtsi
> +++ b/arch/arm/dts/rk3399-u-boot.dtsi
> @@ -71,7 +71,12 @@
>              0x0 0xffa88800 0x0 0x1800
>              0x0 0xffa8a000 0x0 0x2000
>              0x0 0xffa8c000 0x0 0x1000>;
> -     bootph-all;
> +#ifdef CONFIG_VPL
> +             bootph-pre-ram;
> +             bootph-some-ram;
> +#else
> +             bootph-all;
> +#endif
>       status = "okay";
>  };
>  
> @@ -118,8 +123,10 @@
>  };
>  
>  &sdmmc {
> +     bootph-pre-sram;
>       bootph-pre-ram;
>       bootph-some-ram;
> +     bootph-verify;

The addition to include sdmmc in TPL and VPL should also be limited to
VPL=y. All Rockchip AArch64 targets are now using BootROM to load SPL
after DRAM is initialized in TPL, so there is no need for sdmmc node.

The sdmmc node was changed from bootph-all in commit 100f489f58a6
("rockchip: rk3399: Fix loading FIT from SD-card when booting from
eMMC") for similar reason as this patch, remove unneeded/unused nodes
to save space.

Regards,
Jonas

>  
>       /* mmc to sram can't do dma, prevent aborts transferring TF-A parts */
>       u-boot,spl-fifo-mode;

Reply via email to