powernow-k8.c: In function `query_current_values_with_pending_wait':
authorDave Jones <davej@redhat.com>
Thu, 28 Jul 2005 16:45:10 +0000 (09:45 -0700)
committerDave Jones <davej@redhat.com>
Thu, 28 Jul 2005 16:45:10 +0000 (09:45 -0700)
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>
arch/i386/kernel/cpu/cpufreq/powernow-k8.c

index 7fe5b2a..de5deeb 100644 (file)
@@ -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;