mirror of https://gitee.com/openkylin/linux.git
[PATCH] i2c-viapro: Cleanup ifdef usage
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(-)
This commit is contained in:
parent
ed5453e54f
commit
ca68f1193e
|
@ -126,6 +126,8 @@ static void vt596_dump_regs(const char *msg, u8 size)
|
|||
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 */
|
||||
|
@ -135,9 +137,7 @@ static int vt596_transaction(u8 size)
|
|||
int result = 0;
|
||||
int timeout = 0;
|
||||
|
||||
#ifdef DEBUG
|
||||
vt596_dump_regs("Transaction (pre)", size);
|
||||
#endif
|
||||
|
||||
/* 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);
|
||||
|
||||
#ifdef DEBUG
|
||||
vt596_dump_regs("Transaction (post)", size);
|
||||
#endif
|
||||
|
||||
return result;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue