[PATCH v3 3/3] rockchip: sdram: Ensure ram_base is correct in SPL

Kever Yang kever.yang at rock-chips.com
Wed Feb 5 04:24:34 CET 2025


On 2025/1/31 06:07, Jonas Karlman wrote:
> Most Rockchip SoCs use 0x0 as DRAM base address, however some SoCs use
> 0x60000000 and RK3576 use 0x40000000 as DRAM base address.
>
> CFG_SYS_SDRAM_BASE is defined with correct address for each SoC and
> U-Boot proper use this to set correct gd->ram_base in setup_dest_addr().
>
> SPL never assign any value to gd->ram_base and instead use the default,
> 0x0. Set correct gd->ram_base in dram_init() to ensure its correctness
> in SPL.
>
> Signed-off-by: Jonas Karlman <jonas at kwiboo.se>
Reviewed-by: Kever Yang <kever.yang at rock-chips.com>

Thanks,
- Kever
> ---
> v3: No change
> v2: New patch
> ---
>   arch/arm/mach-rockchip/sdram.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/mach-rockchip/sdram.c b/arch/arm/mach-rockchip/sdram.c
> index 4b8b6b9da7c5..f7d32829295c 100644
> --- a/arch/arm/mach-rockchip/sdram.c
> +++ b/arch/arm/mach-rockchip/sdram.c
> @@ -478,6 +478,7 @@ int dram_init(void)
>   		debug("Cannot get DRAM size: %d\n", ret);
>   		return ret;
>   	}
> +	gd->ram_base = ram.base;
>   	gd->ram_size = ram.size;
>   	debug("SDRAM base=%lx, size=%lx\n",
>   	      (unsigned long)ram.base, (unsigned long)ram.size);


More information about the U-Boot mailing list