
Dear "=?ISO-8859-1?Q?Andreas_Bie=DFmann?=",
In message 4E9C017D.1080407@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