[U-Boot] [PATCH v5] ARM: mx31: Print the silicon version
Detlev Zundel
dzu at denx.de
Wed Apr 27 11:11:54 CEST 2011
Hi Stefano,
> On 04/15/2011 08:07 PM, Stefano Babic wrote:
>> On 04/12/2011 04:18 AM, Fabio Estevam wrote:
>>> Use the same method of the Linux kernel to print the MX31 silicon version on
>>> boot.
>>>
>>> Tested on a MX31PDK with a 2.0 silicon, where it shows:
>>>
>>> CPU: Freescale i.MX31 rev 2.0 at 531 MHz
>>>
>>> Signed-off-by: Fabio Estevam <fabio.estevam at freescale.com>
>>>
>>
>> Applied to u-boot-imx, thanks.
>>
>
> Fabio,
>
> I have not noted before that your patch introduce a warning:
>
> generic.c: In function 'get_cpu_rev':
> generic.c:131: warning: return discards qualifiers from pointer target type
>
> This is due to the usage of the const in the mx3_cpu_type:
>
> struct mx3_cpu_type {
> u8 srev;
> const char *v;
>
> Do you agree if I drop myself the const attribute on u-boot-imx before
> pulling your patch to the arm tree ?
Sorry to jump in late, but why not change get_cpu_rev to 'const char *'
which it actually is?
Cheers
Detlev
--
Emacs is the way to purify your soul using garbage collection.
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de
More information about the U-Boot
mailing list