Subject: Re: [PATCH v2] powerpc: Show current speed in /proc/cpuinfo

From: Arnd Bergmann <[email-masked]>
Date: Wed, 2 May 2007 01:48:07 +0200

On Monday 30 April 2007, Olof Johansson wrote:
> On other architectures, the frequency in /proc/cpuinfo moves with cpufreq
> changes. It makes sense to do the same on powerpc to keep users from
> getting confused.
>=20
> Change the format to three decimals, having full Hz granularity is silly
> these days. Also specify in output if it's cpufreq or firmware-reported:
>=20
> clock =A0 =A0 =A0 =A0 =A0 : 1250.000MHz (from firmware)
> or:
> clock =A0 =A0 =A0 =A0 =A0 : 2500.000MHz (actual speed)

Good idea, but I'd be _very_ careful with changing any field in
/proc/cpuinfo. Having the extra text in there can easily confuse
parsers in poorly written user applications, maybe even in good ones ;-)

How about rather adding a new line in there for one of the fields, either

clock             : 1250.000MHz
max-clock         : 2500.000MHz

or

clock             : 2500.000MHz
current-clock     : 1250.000MHz

	Arnd <><