diff options
| author | Dave Jones <davej@redhat.com> | 2005-07-28 12:45:10 -0400 |
|---|---|---|
| committer | Dave Jones <davej@redhat.com> | 2005-07-28 12:45:10 -0400 |
| commit | 7153d9612fe5cefc29f9f17d7a6b9f0dcd07b20e (patch) | |
| tree | 571f533262cd79dd1f2d990e3643a33eb0d18ef0 | |
| parent | cc993cab0239cb07af329d2e18faac7888821075 (diff) | |
powernow-k8.c: In function `query_current_values_with_pending_wait':
powernow-k8.c:110: warning: `hi' may be used uninitialized in this function
Signed-off-by: Brian Gerst <bgerst@didntduck.org>
Signed-off-by: Dave Jones <davej@redhat.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
| -rw-r--r-- | arch/i386/kernel/cpu/cpufreq/powernow-k8.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/arch/i386/kernel/cpu/cpufreq/powernow-k8.c b/arch/i386/kernel/cpu/cpufreq/powernow-k8.c index 7fe5b2aaa209..de5deebc0154 100644 --- a/arch/i386/kernel/cpu/cpufreq/powernow-k8.c +++ b/arch/i386/kernel/cpu/cpufreq/powernow-k8.c | |||
| @@ -110,14 +110,13 @@ static int query_current_values_with_pending_wait(struct powernow_k8_data *data) | |||
| 110 | u32 lo, hi; | 110 | u32 lo, hi; |
| 111 | u32 i = 0; | 111 | u32 i = 0; |
| 112 | 112 | ||
| 113 | lo = MSR_S_LO_CHANGE_PENDING; | 113 | do { |
| 114 | while (lo & MSR_S_LO_CHANGE_PENDING) { | ||
| 115 | if (i++ > 0x1000000) { | 114 | if (i++ > 0x1000000) { |
| 116 | printk(KERN_ERR PFX "detected change pending stuck\n"); | 115 | printk(KERN_ERR PFX "detected change pending stuck\n"); |
| 117 | return 1; | 116 | return 1; |
| 118 | } | 117 | } |
| 119 | rdmsr(MSR_FIDVID_STATUS, lo, hi); | 118 | rdmsr(MSR_FIDVID_STATUS, lo, hi); |
| 120 | } | 119 | } while (lo & MSR_S_LO_CHANGE_PENDING); |
| 121 | 120 | ||
| 122 | data->currvid = hi & MSR_S_HI_CURRENT_VID; | 121 | data->currvid = hi & MSR_S_HI_CURRENT_VID; |
| 123 | data->currfid = lo & MSR_S_LO_CURRENT_FID; | 122 | data->currfid = lo & MSR_S_LO_CURRENT_FID; |
