i2c-i801: fix all coding style issues found by checkpatch.pl
PATCH 1/1 i2c-i801: fix all coding style issues found by checkpatch.pl
This patch fixes up all coding style issues found by the checkpatch.pl tool. After applying this patch the binary object is still exactly the same.
Signed-off-by Farid Hammane <[email protected]>
--- drivers/i2c/busses/i2c-i801.c | 22 +++++++++++----------- 1 files changed, 11 insertions(+), 11 deletions(-)
drivers/i2c/busses/i2c-i801.c +11 -11
@@ -22,8 +22,8 @@/* Supports the following Intel I/O Controller Hubs (ICH): -I/O Block I2C -region SMBus Block proc. block +I/O Block I2C +region SMBus Block proc. block Chip name PCI ID size PEC buffer call read ---------------------------------------------------------------------- 82801AA (ICH) 0x2413 16 no no no no
@@ -341,8 +341,7 @@ static int i801_block_transaction_byte_by_byte(union i2c_smbus_data *data,do {msleep (1 ); status =inb_p (SMBHSTSTS ); - } -while ((!(status &SMBHSTSTS_BYTE_DONE )) + }while ((!(status &SMBHSTSTS_BYTE_DONE )) && (timeout++ <MAX_TIMEOUT )); result =i801_check_post (status, timeout >MAX_TIMEOUT );
@@ -440,9 +439,9 @@ static int i801_block_transaction(union i2c_smbus_data *data, char read_write, }/* Return negative errno on error. */ -static s32 i801_access (struct i2c_adapter * adap,u16 addr, +static s32 i801_access (struct i2c_adapter *adap,u16 addr,unsigned short flags,char read_write,u8 command, -int size,union i2c_smbus_data * data) +int size,union i2c_smbus_data *data) {int hwpec;int block =0 ;
@@ -511,7 +510,7 @@ static s32 i801_access(struct i2c_adapter * adap, u16 addr,else outb_p (inb_p (SMBAUXCTL ) & (~SMBAUXCTL_CRC ),SMBAUXCTL ); -if (block) +if (block) ret =i801_block_transaction (data, read_write, size, hwpec);else ret =i801_transaction (xact |ENABLE_INT9 );
@@ -523,9 +522,9 @@ static s32 i801_access(struct i2c_adapter * adap, u16 addr,outb_p (inb_p (SMBAUXCTL ) & ~(SMBAUXCTL_CRC |SMBAUXCTL_E32B ),SMBAUXCTL ); -if (block) +if (block)return ret; -if (ret) +if (ret)return ret;if ((read_write ==I2C_SMBUS_WRITE ) || (xact ==I801_QUICK ))return 0 ;
@@ -585,7 +584,7 @@ static const struct pci_device_id i801_ids[] = { { 0, } }; -MODULE_DEVICE_TABLE (pci, i801_ids); +MODULE_DEVICE_TABLE (pci, i801_ids);#if definedCONFIG_INPUT_APANEL || definedCONFIG_INPUT_APANEL_MODULE static unsigned char apanel_addr;
@@ -689,7 +688,8 @@ static void __devinit dmi_check_onboard_devices(const struct dmi_header *dm, }#endif -static int __devinit i801_probe (struct pci_dev *dev,const struct pci_device_id *id) +static int __devinit i801_probe (struct pci_dev *dev, +const struct pci_device_id *id) {unsigned char temp;int err;
Signature · 1 line
1.7.0
Hi Farid,
Quoted 7 lines · Farid Hammane
This patch fixes up all coding style issues found by the checkpatch.pl tool. After applying this patch the binary object is still exactly the same.
Signed-off-by Farid Hammane <[email protected]>--- drivers/i2c/busses/i2c-i801.c | 22 +++++++++++----------- 1 files changed, 11 insertions(+), 11 deletions(-)
I already have an equivalent patch in my tree:
ftp://ftp.kernel.org/pub/linux/kernel/people/jdelvare/linux-2.6/jdelvare-i2c/i2c-i801-04-coding-style-fixes.patch
Signature · 1 line
Jean Delvare