Dear Jason Cooper,

In message <[email protected]> you wrote:
> If CONFIG_DISPLAY_CPUINFO is enabled on kirkwood SoCs, this will print the
> speeds of the various components.
> 
> Signed-off-by: Jason Cooper <[email protected]>
> ---
> Changes since v1:
>       - optimized macros to remove one-timers
>       - reduced switch/case to if/else since it operates on one bit.
>       - remove RFC
> 
>  arch/arm/cpu/arm926ejs/kirkwood/cpu.c    |   41 
> ++++++++++++++++++++++++++++++
>  arch/arm/include/asm/arch-kirkwood/cpu.h |    1 +
>  2 files changed, 42 insertions(+), 0 deletions(-)
...
> +     printf("CPU running @ %dMHz L2 running @ %dMHz\n",
> +             cpu_clk, l2_clk);
> +     printf("SysClock = %dMHz, TClock = %dMHz\n",
> +             sys_clk, t_clk / 1000000);

Please do NOT print such information as part of the regular,
unconditionally printed boot messages.  We want to keep thes
einformatiove, but otherwise as short as possible.

I recommend to use the "clock" command to print such information  then
the user is free to decide if he wants to see this, or not.

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: [email protected]
panic: kernel trap (ignored)
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to