[U-Boot] [PATCH 2/2] powerpc/85xx: standardize display of address map size (32-bit vs. 36-bit)

Wolfgang Denk wd at denx.de
Thu Sep 1 23:59:31 CEST 2011


Dear Timur Tabi,

In message <4E5F9C8D.3080503 at freescale.com> you wrote:
> Wolfgang Denk wrote:
> > But you are right - I'd much rather see this printed for example as
> > part of the "bdinfo" command than with the regular boot messages.
> 
> Recently, we've been adding boards that have CONFIG_PCI_SCAN_SHOW defined.  Are
> you saying that we should not be doing that?  That adds a lot more text than my
> patch does.

Yes, that's what I'm sayin.  All this "useful" information should be
available easily to everybody who is interested in it - no boubt of
that.  But it shoudld NOT be printed on each and every boot.

> Kumar, are you okay with not displaying the address map size at all during boot
> time?  If we bury this information in the 'bdinfo' command, we're going to have
> even more confusion as to which U-Boot we're booting.

What sort of "confusion" do you have?  I see two situations:  in
99.99% of all cases U-Boot is just a means to boot an OS, and nobody
cares a bit about the actual U-Boot output, as long as the OS is
runnign after a few seconds, and rather sooner than later.  For a few
use cases where developers are working on a board, they might wonder
which state a board is in - then itis very useful to have a command to
display this information any time they are interested in it - even
without having to reboot the system.

It is actually pretty silly to print certain interesting information
only at boot time.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
For every complex problem, there is a solution that is simple,  neat,
and wrong.                                               - Mark Twain


More information about the U-Boot mailing list