[U-Boot] [PATCH] Add "cpu_config" command for PPC4xx bootstrap configuration

Wolfgang Denk wd at denx.de
Fri Jul 17 13:20:17 CEST 2009


Dear Stefan Roese,

In message <200907171313.37259.sr at denx.de> you wrote:
> 
> > Why are the lines indented by one space?
> 
> It was this way in the original bootstrap version. I can remove it if you 
> prefer.

Please do.

> > Would it be possible to also mark the current setting in this output?
> > Like printing an asterisk befor or after it?
> 
> No. Currently the implementation only checks strings and has no idea of 
> frequencies. Also, not only PLL frequencies are configured in this I2C 
> bootstrap EEPROM but also boot-location (NOR, NAND, PCI) and other things. So 
> it's really not trivial to find/search the current configuration in this 
> table.

Umm.. I think it should be fairly trivial to read the current  EEPROM
settings and compare these against the entries in the table above?

> Since 4xx prints the speed's in the bootlog, it shouldn't be too hard to find 
> the current setting there (if necessary).

But would it not be  useful  to  be  able  to  detect  any  "illegal"
settings,  for  example resulting from accidentyial corruption of the
EEPROM data?

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
If you're not part of the solution, then you're part of the  precipi-
tate.


More information about the U-Boot mailing list