diff options
| author |   Dave Jones <davej@redhat.com> | 2005-07-28 09:45:10 -0700 |
|---|---|---|
| committer | Dave Jones <davej@redhat.com> | 2005-07-28 09:45:10 -0700 |
| commit | 7153d9612fe5cefc29f9f17d7a6b9f0dcd07b20e (patch) | |
| tree | 571f533262cd79dd1f2d990e3643a33eb0d18ef0 /arch/i386 | |
| parent | Here are two possible cleanups in cpufreq.c: (diff) | |
| download | linux-dev-7153d9612fe5cefc29f9f17d7a6b9f0dcd07b20e.tar.xz linux-dev-7153d9612fe5cefc29f9f17d7a6b9f0dcd07b20e.zip | |
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>
Diffstat (limited to 'arch/i386')
| -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) u32 lo, hi; u32 i = 0; - lo = MSR_S_LO_CHANGE_PENDING; - while (lo & MSR_S_LO_CHANGE_PENDING) { + do { if (i++ > 0x1000000) { printk(KERN_ERR PFX "detected change pending stuck\n"); return 1; } rdmsr(MSR_FIDVID_STATUS, lo, hi); - } + } while (lo & MSR_S_LO_CHANGE_PENDING); data->currvid = hi & MSR_S_HI_CURRENT_VID; data->currfid = lo & MSR_S_LO_CURRENT_FID; |