Hi Tom
On 10/10/24 23:28, Tom Rini wrote:
On Thu, Oct 10, 2024 at 12:39:09PM +0530, Neha Malcom Francis 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-francis@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 ");We're growing a constant string in every build here. I would rather see doc/develop/memory.rst start to get updated to spell out that when we print "effective" here it's because we do not have the full 64bit address space available, only 2GiB.
Agreed, maybe we can take both updation of memory.rst as well as Simon's suggestion of s/effective/total to clear things out.