[PATCH] boardf: Modify DRAM message

Simon Glass sjg at chromium.org
Thu Oct 10 20:50:07 CEST 2024


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?

Regards,
Simon


More information about the U-Boot mailing list