Hi Simon + Vignesh
On 11/10/24 00:20, Simon Glass wrote:
Hi Neha, On Thu, 10 Oct 2024 at 01:09, Neha Malcom Francis <n-fran...@ti.com> wrote:The message "DRAM: 2 GiB (effective 32 GiB)" can be a little confusing, modify the message to show exactly what is meant: "DRAM: 2 GiB (available for U-Boot out of total 32 GiB)" Signed-off-by: Neha Malcom Francis <n-fran...@ti.com> --- common/board_f.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/board_f.c b/common/board_f.c index 154675d0e40..b7add8f7d3d 100644 --- a/common/board_f.c +++ b/common/board_f.c @@ -254,7 +254,7 @@ static int show_dram_config(void) print_size(gd->ram_size, ""); if (!sizes_near(gd->ram_size, size)) { - printf(" (effective "); + printf(" (available for U-Boot out of total "); print_size(size, ")"); } board_add_ram_info(0); -- 2.34.1I've forgotten why the first number can be so much lower than the second. What is causing that?
AFAIK, U-Boot supports up to 39-bit addressing and possibly since there are systems that have RAM sizes exceeding 39-bit, addressing has been capped to the first bank to avoid any possible problems. So U-Boot sets gd->ram_size to the first bank while it acknowledges that the system RAM is more than that. Tom can probably confirm if this is right.
In an offline discussion with Vignesh, he mentioned that a cleaner solution would be to extend out fdtdec_* functions to support all memory within a 39-bit address space.
This is quite a long string...could we say "(total 32 GiB)" ? Then perhaps add something to doc/ to talk about the banner that U-Boot prints? Regards, Simon
-- Thanking You Neha Malcom Francis
