On Mon, Oct 14, 2024 at 07:23:09PM +0200, Quentin Schulz wrote:
> Hi all,
>
> On 10/14/24 5:35 PM, Tom Rini wrote:
> > On Mon, Oct 14, 2024 at 08:59:39PM +0530, Devarsh Thakkar wrote:
> > > Hi,
> > >
> > > On 14/10/24 19:57, Tom Rini wrote:
> > > > On Mon, Oct 14, 2024 at 10:10:43AM +0530, Neha Malcom Francis wrote:
> > > > > Hi Andrew
> > > > >
> > > > > On 11/10/24 01:59, Andrew Davis wrote:
> > > > > > On 10/10/24 1:50 PM, 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.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.
> > > > > >
> > > > >
> > > Knowing how much u-boot is using might help someone who is working on some
> > > memory map related stuff or if they want to stop at u-boot prompt and
> > > tweak
> > > environment variables related to memory map (e.g. load address for kernel,
> > > dtb, load bitmap to DDR etc). Overall I personally don't see much harm on
> > > keeping both the information (DDR available to u-boot and total available
> > > DDR). If the word effective is confusing than the print can be made more
> > > verbose as being intended in this patch-set or by following the
> > > suggestions
> > > given by Simon and Udit.
> >
> > U-Boot isn't "using" 2GiB however. And in fact the command Simon is
> > working on to dump the memory table and what's where, as far as U-Boot
> > knows, would be helpful in that case.
> >
>
> "Available" and "Total" would be matching the terms used by "free" in Linux
> for example.
>
> I still think both are valuable pieces of information (what's populated and
> what is usable by U-Boot AFAIR).Yes, and that won't change the binary size much, thanks. -- Tom
signature.asc
Description: PGP signature
