[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 c48b44b3b43ab1e08c0c5c250f3a7f2fcbc5caf6..90cab2d4ac0dd7068cacb1dfdac0664a313cfcd9 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;