[U-Boot] [PATCH 2/4] mx53loco: Add a comment about local print_cpuinfo method in board file

Otavio Salvador otavio at ossystems.com.br
Sun Jun 24 21:11:07 CEST 2012


On Sun, Jun 24, 2012 at 4:55 AM, Wolfgang Denk <wd at denx.de> wrote:
>> +/*
>> + * Usually this is called too early printing a wrong information about
>> + * the clock. To fix it we need to provide this local method and call
>> + * it later in board_init.
>> + */
>
>
> I don't like this change. It messes with the initialization sequence,
> which is not a good thing. Instead of shift more and more things
> backwards, we should fix any incorrect clock calculations here.

Right however the clock calculation is right but in regular
initialization sequence this information is printed too early so Fabio
has moved it here to change this ordering.

I agree it is not the best solution but a comment here makes it clear
why this has been done.

-- 
Otavio Salvador                             O.S. Systems
E-mail: otavio at ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br


More information about the U-Boot mailing list