diff options
| author | Borislav Petkov <borislav.petkov@amd.com> | 2010-07-08 12:05:14 -0400 |
|---|---|---|
| committer | Dave Jones <davej@redhat.com> | 2010-08-03 13:47:06 -0400 |
| commit | b30d3304c9c068ccfe6940232834768af75f8c9a (patch) | |
| tree | e4ca17a9e8cdcac951291b5796ea49ec7c4fb1ba | |
| parent | 55c789bb2bcdcaa8f1f60687b4a9dbd02ffddd88 (diff) | |
[CPUFREQ] powernow-k8: Fix misleading variable naming
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>
| -rw-r--r-- | arch/x86/kernel/cpu/cpufreq/powernow-k8.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/arch/x86/kernel/cpu/cpufreq/powernow-k8.c b/arch/x86/kernel/cpu/cpufreq/powernow-k8.c index c48b44b3b43a..90cab2d4ac0d 100644 --- a/arch/x86/kernel/cpu/cpufreq/powernow-k8.c +++ b/arch/x86/kernel/cpu/cpufreq/powernow-k8.c | |||
| @@ -912,8 +912,8 @@ static int fill_powernow_table_pstate(struct powernow_k8_data *data, | |||
| 912 | { | 912 | { |
| 913 | int i; | 913 | int i; |
| 914 | u32 hi = 0, lo = 0; | 914 | u32 hi = 0, lo = 0; |
| 915 | rdmsr(MSR_PSTATE_CUR_LIMIT, hi, lo); | 915 | rdmsr(MSR_PSTATE_CUR_LIMIT, lo, hi); |
| 916 | data->max_hw_pstate = (hi & HW_PSTATE_MAX_MASK) >> HW_PSTATE_MAX_SHIFT; | 916 | data->max_hw_pstate = (lo & HW_PSTATE_MAX_MASK) >> HW_PSTATE_MAX_SHIFT; |
| 917 | 917 | ||
| 918 | for (i = 0; i < data->acpi_data.state_count; i++) { | 918 | for (i = 0; i < data->acpi_data.state_count; i++) { |
| 919 | u32 index; | 919 | u32 index; |
