[PATCH] boardf: Modify DRAM message

[Andrew Davis]

On 10/10/24 1:50 PM, Simon Glass wrote:
> Hi Neha,
> 
> On Thu, 10 Oct 2024 at 01:09, Neha Malcom Francis <n-francis at 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-francis at 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.1
>>
> 
> I've forgotten why the first number can be so much lower than the
> second. What is causing that?
> 
> 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?
> 

Why not just only print the total number? The amount available to
U-Boot seems like an internal detail, I'd guess most folks just what
to see at a glance how much DDR was detected on their system.

Andrew

> Regards,
> Simon