arm: stm32mp: Fix board_get_usable_ram_top() again
authorMarek Vasut <marex@denx.de>
Thu, 5 Jan 2023 01:22:22 +0000 (02:22 +0100)
committerPatrick Delaunay <patrick.delaunay@foss.st.com>
Fri, 6 Jan 2023 10:02:17 +0000 (11:02 +0100)
Do not access gd->ram_size and assume this is actual valid RAM size. Since commit
777aaaa706b ("common/memsize.c: Fix get_effective_memsize() to check for overflow")
the RAM size may be less than gd->ram_size , call get_effective_memsize() to get
the limited value instead.

The aforementioned commit makes STM32MP15xx boards with 1 GiB of DRAM
at 0xc0000000 hang on boot, which is a grave defect.

Signed-off-by: Marek Vasut <marex@denx.de>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
arch/arm/mach-stm32mp/dram_init.c

index 9346fa8..80ba5c2 100644 (file)
@@ -51,7 +51,7 @@ phys_size_t board_get_usable_ram_top(phys_size_t total_size)
 
        /* found enough not-reserved memory to relocated U-Boot */
        lmb_init(&lmb);
-       lmb_add(&lmb, gd->ram_base, gd->ram_size);
+       lmb_add(&lmb, gd->ram_base, get_effective_memsize());
        boot_fdt_add_mem_rsv_regions(&lmb, (void *)gd->fdt_blob);
        /* add 8M for reserved memory for display, fdt, gd,... */
        size = ALIGN(SZ_8M + CONFIG_SYS_MALLOC_LEN + total_size, MMU_SECTION_SIZE),