[PATCH 6/6] cmd: Add a command to show the memory map

[Tom Rini]

On Mon, Oct 14, 2024 at 01:13:25PM -0600, Simon Glass wrote:
> Hi Ilias,
> 
> On Thu, 10 Oct 2024 at 09:27, Ilias Apalodimas
> <ilias.apalodimas at linaro.org> wrote:
> >
> > Hi Simon,
> >
> >
> > On Thu, 10 Oct 2024 at 18:10, Simon Glass <sjg at chromium.org> wrote:
> > >
> > > Hi,
> > >
> > > On Thu, 10 Oct 2024 at 06:02, Ilias Apalodimas
> > > <ilias.apalodimas at linaro.org> wrote:
> > > >
> > > > On Thu, 10 Oct 2024 at 01:39, Tom Rini <trini at konsulko.com> wrote:
> > > > >
> > > > > On Wed, Oct 09, 2024 at 03:14:25PM -0600, Simon Glass wrote:
> > > > > > Hi Ilias,
> > > > > >
> > > > > > On Wed, 9 Oct 2024 at 04:41, Ilias Apalodimas
> > > > > > <ilias.apalodimas at linaro.org> wrote:
> > > > > > >
> > > > > > > Hi Simon,
> > > > > > >
> > > > > > > We already have a mem info command, which is pretty useless. Can't we
> > > > > > > reuse that ?
> > > > > >
> > > > > > I am not too keen on using that as 'info' is pretty vague. We could
> > > > > > perhaps create a 'mem' command with 'info' as one subcommand and 'map'
> > > > > > as another?
> > > > >
> > > > > Are we talking about "meminfo" which literally only prints gd->ram_size?
> > > > > If so, I don't think it's worth worrying about enhancing that.
> > > >
> > > > Yes, that's the one
> > >
> > > So, you mean to just delete that command and make it show the memory
> > > map instead? Or show the memory size at the top, before the memory
> > > map?
> >
> > I meant the latter, but if you and Tom agree that we should leave it
> > as is, I don't mind adding a new command. I assume that the list above
> > is non exhaustive, so in the future we could expand it with lmb
> > reservations etc?
> 
> I sent v2 with that done. The lmb reservations aren't that useful as
> there are no names attached.

I disagree with names being needed to be useful in this dump.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20241014/6daa8c71/attachment.sig>