[U-Boot] [PATCH] mx31: Improve the handling of unidentified silicon version

Wolfgang Denk wd at denx.de
Wed Jun 15 14:12:55 CEST 2011


Dear Fabio Estevam,

In message <4DF89C84.7090605 at freescale.com> you wrote:
...
> CPU:   Freescale i.MX31 rev 2.0 unknown at 531 MHz.Reset cause: WDOG
...
> When the chip version is not valid, then we print:
>
> CPU:   Freescale i.MX31 (unknown rev, srev=0x20) at 531 MHz.Reset cause: WDOG

Compare to the current line above - which new information do you
print?  None.  You just print it in a new format, and with more
characters, i. e. with a worse SNR.

> Please let me know what you think about this proposal.

You received 3 messages from 3 different guys who told you all the
same: just leave the code as is.  Your proposed changes are not
considered an improvement.

Thanks.

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
"How is this place run - is it an anarchy?"
"No, I wouldn't say so; it is not that well organised..."


More information about the U-Boot mailing list