diff options
author | Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> | 2011-06-16 15:36:40 -0400 |
---|---|---|
committer | Dave Jones <davej@redhat.com> | 2011-06-16 16:31:13 -0400 |
commit | fbb5b89eabea5ae7d621b7861863159560d8faa4 (patch) | |
tree | 51e3d877cb3457bcd801175fb455927fd52f2d2b /drivers/cpufreq | |
parent | a9d3d2068064b7a6395871a49616d3784f802d50 (diff) |
[CPUFREQ] powernow-k8: Don't try to transition if the pstate is incorrect
This patch augments the pstate transition code to error out
(instead of returning 0) when an incorrect pstate is provided.
Suggested-by: Borislav Petkov <bp@alien8.de>
CC: andre.przywara@amd.com
CC: Mark.Langsdorf@amd.com
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Signed-off-by: Dave Jones <davej@redhat.com>
Diffstat (limited to 'drivers/cpufreq')
-rw-r--r-- | drivers/cpufreq/powernow-k8.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/cpufreq/powernow-k8.c b/drivers/cpufreq/powernow-k8.c index 287c56f6749a..bce576d7478e 100644 --- a/drivers/cpufreq/powernow-k8.c +++ b/drivers/cpufreq/powernow-k8.c | |||
@@ -1104,7 +1104,8 @@ static int transition_frequency_pstate(struct powernow_k8_data *data, | |||
1104 | /* get MSR index for hardware pstate transition */ | 1104 | /* get MSR index for hardware pstate transition */ |
1105 | pstate = index & HW_PSTATE_MASK; | 1105 | pstate = index & HW_PSTATE_MASK; |
1106 | if (pstate > data->max_hw_pstate) | 1106 | if (pstate > data->max_hw_pstate) |
1107 | return 0; | 1107 | return -EINVAL; |
1108 | |||
1108 | freqs.old = find_khz_freq_from_pstate(data->powernow_table, | 1109 | freqs.old = find_khz_freq_from_pstate(data->powernow_table, |
1109 | data->currpstate); | 1110 | data->currpstate); |
1110 | freqs.new = find_khz_freq_from_pstate(data->powernow_table, pstate); | 1111 | freqs.new = find_khz_freq_from_pstate(data->powernow_table, pstate); |