[U-Boot] [PATCH] mpc8xxx: lc_common_dimm_params: make less verbose

[Wolfgang Denk]

Dear Kumar,

In message <73174EA1-7CBE-4C0F-A426-08F3C0EB7890 at kernel.crashing.org> you wrote:
> 
> > It may have value to you, but for 99.X % of the users it is completey
> > meaningless.
> 
> Its quite helpful for debug of customer issues.  Should we add a CONFIG_ =
> option for this?

I already stated my position:

> >>> Then feel free to add a custom command to display this specific
> >>> information, but keep it off the regular, unconditionally printed boot
> >>> messages.

I don't want to see this in the boot messages.  It can be added to a
command that is used to display such information upon user's request.

For example, detailled output about the memory configuration could be
added to the "bdinfo" command.


I also would like to see to get a lot of the clocks information get
out of the standard boot messages. We don;t need to print 5 or more
lines like on 85xx:

	Clock Configuration:
	       CPU0:833.333 MHz, 
	       CCB:333.333 MHz,
	       DDR:166.667 MHz (333.333 MT/s data rate), LBC:41.667 MHz
	CPM:   333.333 MHz

We have the (optional) "clocks" command that can be used to print
this if anybody is interested in this.

I guess you get the intention: reduce the boot messages to the
essential parts.

Viele Grüße,

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
In Nature there are neither rewards nor punishments, there are conse-
quences.                                            -- R.G. Ingersoll