diff options
author | Brian Gerst <bgerst@didntduck.org> | 2005-07-27 14:43:57 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-07-27 19:25:54 -0400 |
commit | 7657e20e46e26b198b24e2aefc696410bbe889c9 (patch) | |
tree | d8305384be42cb95d9e249edaccbea991a144266 /arch/i386/kernel/cpu | |
parent | bbaf364103cee15c895e2086723d0ad9ef47ae99 (diff) |
[PATCH] Fix warning in powernow-k8.c
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>
Cc: Dave Jones <davej@codemonkey.org.uk>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'arch/i386/kernel/cpu')
-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 10cc096c0ade..31f65c8a4c24 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; |