[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