]> nv-tegra.nvidia Code Review - linux-2.6.git/commitdiff
[PATCH] i2c-viapro: Cleanup ifdef usage
authorGreg KH <greg@kroah.com>
Thu, 22 Sep 2005 20:23:32 +0000 (22:23 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 28 Oct 2005 21:02:08 +0000 (14:02 -0700)
It's not nice to put #ifdef in the middle of functions.

CC: Jean Delvare <khali@linux-fr.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
 drivers/i2c/busses/i2c-viapro.c |    6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

drivers/i2c/busses/i2c-viapro.c

index c1c9eb1d714f3a283d5d4e9a3d3c8ceccff2538c..566342d50c5f189960c10b3286ef9b8c90b29417 100644 (file)
@@ -126,6 +126,8 @@ static void vt596_dump_regs(const char *msg, u8 size)
                printk("%02x\n", inb_p(SMBBLKDAT));
        }
 }
                printk("%02x\n", inb_p(SMBBLKDAT));
        }
 }
+#else
+static inline void vt596_dump_regs(const char *msg, u8 size) { }
 #endif
 
 /* Return -1 on error, 0 on success */
 #endif
 
 /* Return -1 on error, 0 on success */
@@ -135,9 +137,7 @@ static int vt596_transaction(u8 size)
        int result = 0;
        int timeout = 0;
 
        int result = 0;
        int timeout = 0;
 
-#ifdef DEBUG
        vt596_dump_regs("Transaction (pre)", size);
        vt596_dump_regs("Transaction (pre)", size);
-#endif
 
        /* Make sure the SMBus host is ready to start transmitting */
        if ((temp = inb_p(SMBHSTSTS)) & 0x1F) {
 
        /* Make sure the SMBus host is ready to start transmitting */
        if ((temp = inb_p(SMBHSTSTS)) & 0x1F) {
@@ -192,9 +192,7 @@ static int vt596_transaction(u8 size)
        if (temp & 0x1F)
                outb_p(temp, SMBHSTSTS);
 
        if (temp & 0x1F)
                outb_p(temp, SMBHSTSTS);
 
-#ifdef DEBUG
        vt596_dump_regs("Transaction (post)", size);
        vt596_dump_regs("Transaction (post)", size);
-#endif
 
        return result;
 }
 
        return result;
 }