[U-Boot] [PATCH v2 3/5] davinci: add clocks command
Wolfgang Denk
wd at denx.de
Mon Oct 17 12:53:34 CEST 2011
Dear "=?ISO-8859-1?Q?Andreas_Bie=DFmann?=",
In message <4E9C017D.1080407 at 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: wd at denx.de
Pain is a thing of the mind. The mind can be controlled.
-- Spock, "Operation -- Annihilate!" stardate 3287.2
More information about the U-Boot
mailing list