Dear "=?ISO-8859-1?Q?Andreas_Bie=DFmann?=",

In message <4e9c017d.1080...@gmail.com> you wrote:
>
> > add 'clocks' command to print various clock frequency info found
> > in SOC such as ARM core frequency, DSP core frequency and DDR
> > frequency.
> 
> Isn't that the job of command 'bdi'? Why introduce a new, proprietary
> command instead of changing still available ones?

"bdinfo" indeed should include major clock information, but does not
necissarily print all relevant data on a specific SoC.

> Well maybe is a 'clocks' command really useful, but then it should be
> provided to all with a clear interface and not hidden in some
> architecture specific directory.

Correct.  There are already several such implementations (in
arch/powerpc/cpu/mpc512x/speed.c and
arch/powerpc/cpu/mpc83xx/speed.c), now it is high time to turn this
into a generic command that can be used by everyone.

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: w...@denx.de
Pain is a thing of the mind.  The mind can be controlled.
        -- Spock, "Operation -- Annihilate!" stardate 3287.2
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to