Changeset 5816

Timestamp:

01/17/10 12:32:11 (3 years ago)

Author:

khali

Message:

i2c-ali1563: Remove sparse warnings.
Backported from Linux 2.6, original patch from Marton Nemeth:
Remove the following sparse warnings (see "make C=1"):

• drivers/i2c/busses/i2c-ali1563.c:91:3: warning: do-while statement is not a compound statement
• drivers/i2c/busses/i2c-ali1563.c:161:3: warning: do-while statement is not a compound statement

Location:

lm-sensors/branches/lm-sensors-2.10

Files:

• CHANGES (modified) (1 diff)
• kernel/busses/i2c-ali1563.c (modified) (2 diffs)

lm-sensors/branches/lm-sensors-2.10/CHANGES

@@ -5 +5 @@
   File lm_sensors.lsm: Delete, don't advertise a legacy branch
   File doc/developers/release_checklist: Skip mail to lsm
+  Module i2c-ali1563: Remove sparse warnings (2.6 backport)
   Module i2c-amd8111: Fix timeout handling (2.6 backport)
   Module i2c-piix4: ATI SB800 is not actually supported

lm-sensors/branches/lm-sensors-2.10/kernel/busses/i2c-ali1563.c

@@ -93 +93 @@
 
         timeout = ALI1563_MAX_TIMEOUT;
-        do
+        do {
                 i2c_delay(1);
-        while (((data = inb_p(SMB_HST_STS)) & HST_STS_BUSY) && --timeout);
+        } while (((data = inb_p(SMB_HST_STS)) & HST_STS_BUSY) && --timeout);
 
         printk(KERN_DEBUG "ali1563: Transaction (post): STS=%02x, CNTL1=%02x, "
@@ -163 +163 @@
 
         timeout = ALI1563_MAX_TIMEOUT;
-        do
+        do {
                 i2c_delay(1);
-        while (!((data = inb_p(SMB_HST_STS)) & HST_STS_DONE) && --timeout);
+        } while (!((data = inb_p(SMB_HST_STS)) & HST_STS_DONE) && --timeout);
 
         printk(KERN_DEBUG "ali1563: Block (post): STS=%02x, CNTL1=%02x, "