Hi Simon,
On Thu, 10 Oct 2024 at 18:10, Simon Glass <s...@chromium.org> wrote: > > Hi, > > On Thu, 10 Oct 2024 at 06:02, Ilias Apalodimas > <ilias.apalodi...@linaro.org> wrote: > > > > On Thu, 10 Oct 2024 at 01:39, Tom Rini <tr...@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.apalodi...@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? Thanks /Ilias > > Regards, > Simon