[PATCH] ARM: Prefix revision number in /proc/cpuinfo

Alessandro Rubini rubini-list at gnudd.com
Thu Nov 12 09:30:20 EST 2009


> The revision field in /proc/cpuinfo is reported as hex number, so it
> should have a '0x' prefix to make that clear. Otherwise, parsers
> will take it as decimal number unless it contains a letter.

If the problem is real (I assume it is), why don't you fix all
fields?

> -	seq_printf(m, "Revision\t: %04x\n", system_rev);
> +	seq_printf(m, "Revision\t: 0x%04x\n", system_rev);
>  	seq_printf(m, "Serial\t\t: %08x%08x\n",
>  		   system_serial_high, system_serial_low);

"Serial", above, is hex as well.

thanks
/alessandro



More information about the linux-arm-kernel mailing list