[CPUFREQ] powernow-k8: Fix misleading variable naming
authorBorislav Petkov <borislav.petkov@amd.com>
Thu, 8 Jul 2010 16:05:14 +0000 (18:05 +0200)
committerDave Jones <davej@redhat.com>
Tue, 3 Aug 2010 17:47:06 +0000 (13:47 -0400)
rdmsr() takes the lower 32 bits as a second argument and the high 32 as
a third. Fix the names accordingly since they were swapped.

There should be no functionality change resulting from this patch.

Signed-off-by: Borislav Petkov <borislav.petkov@amd.com>
Signed-off-by: Dave Jones <davej@redhat.com>
arch/x86/kernel/cpu/cpufreq/powernow-k8.c

index c48b44b..90cab2d 100644 (file)
@@ -912,8 +912,8 @@ static int fill_powernow_table_pstate(struct powernow_k8_data *data,
 {
        int i;
        u32 hi = 0, lo = 0;
-       rdmsr(MSR_PSTATE_CUR_LIMIT, hi, lo);
-       data->max_hw_pstate = (hi & HW_PSTATE_MAX_MASK) >> HW_PSTATE_MAX_SHIFT;
+       rdmsr(MSR_PSTATE_CUR_LIMIT, lo, hi);
+       data->max_hw_pstate = (lo & HW_PSTATE_MAX_MASK) >> HW_PSTATE_MAX_SHIFT;
 
        for (i = 0; i < data->acpi_data.state_count; i++) {
                u32 index;