2007-11-21 03:35:59

by Yinghai Lu

[permalink] [raw]
Subject: [PATCH] CPUFREQ: powernow-k8 print pstate instead of fid/did for family 10h

[PATCH] CPUFREQ: powernow-k8 print pstate instead of fid/did for family 10h

powernow-k8: Found 1 Quad-Core AMD Opteron(tm) Processor 8354 processors (4 cpu cores) (version 2.20.00)
powernow-k8: 0 : fid 0x0 did 0x0 (2200 MHz)
powernow-k8: 1 : fid 0x0 did 0x0 (2000 MHz)
powernow-k8: 2 : fid 0x0 did 0x0 (1700 MHz)
powernow-k8: 3 : fid 0x0 did 0x0 (1400 MHz)
powernow-k8: 4 : fid 0x0 did 0x0 (1100 MHz)

actually index for CPU_HW_PSTATE is pstate instead of fid/vid
So print it out as pstate.

powernow-k8: Found 1 Quad-Core AMD Opteron(tm) Processor 8354 processors (4 cpu cores) (version 2.20.00)
powernow-k8: 0 : pstate 0 (2200 MHz)
powernow-k8: 1 : pstate 1 (2000 MHz)
powernow-k8: 2 : pstate 2 (1700 MHz)
powernow-k8: 3 : pstate 3 (1400 MHz)
powernow-k8: 4 : pstate 4 (1100 MHz)

Signed-off-by: Yinghai Lu <[email protected]>

Index: linux-2.6/arch/x86/kernel/cpu/cpufreq/powernow-k8.c
===================================================================
--- linux-2.6.orig/arch/x86/kernel/cpu/cpufreq/powernow-k8.c
+++ linux-2.6/arch/x86/kernel/cpu/cpufreq/powernow-k8.c
@@ -578,10 +578,9 @@ static void print_basics(struct powernow
for (j = 0; j < data->numps; j++) {
if (data->powernow_table[j].frequency != CPUFREQ_ENTRY_INVALID) {
if (cpu_family == CPU_HW_PSTATE) {
- printk(KERN_INFO PFX " %d : fid 0x%x did 0x%x (%d MHz)\n",
+ printk(KERN_INFO PFX " %d : pstate %d (%d MHz)\n",
j,
- (data->powernow_table[j].index & 0xff00) >> 8,
- (data->powernow_table[j].index & 0xff0000) >> 16,
+ data->powernow_table[j].index,
data->powernow_table[j].frequency/1000);
} else {
printk(KERN_INFO PFX " %d : fid 0x%x (%d MHz), vid 0x%x\n",