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

[Kever Yang]

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 <jo...@kwiboo.se>

Reviewed-by: Kever Yang <kever.y...@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);