diff options
author | Jean Delvare <khali@linux-fr.org> | 2005-09-22 15:50:47 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2005-10-28 17:02:07 -0400 |
commit | 5f49ef8e8cefe0a95948b4270db28507c1c287d4 (patch) | |
tree | ea50363d9480a9ccae000bd46633866561a62cea /drivers | |
parent | 32c0a520f506c046f241de2a3a9b02a395ef76d2 (diff) |
[PATCH] i2c-viapro: Coding style fixes
Before I go on cleaning up and improving the i2c-viapro driver, let's
fix all the coding style issues: mostly trailing white space, and
spaces used where tabs should be.
Signed-off-by: Jean Delvare <khali@linux-fr>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Documentation/i2c/busses/i2c-viapro | 12 ++---
drivers/i2c/busses/i2c-viapro.c | 76 ++++++++++++++++++------------------
2 files changed, 43 insertions(+), 45 deletions(-)
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/i2c/busses/i2c-viapro.c | 76 |
1 files changed, 38 insertions, 38 deletions
diff --git a/drivers/i2c/busses/i2c-viapro.c b/drivers/i2c/busses/i2c-viapro.c index 99d209e0485a..262755e1ae02 100644 --- a/drivers/i2c/busses/i2c-viapro.c +++ b/drivers/i2c/busses/i2c-viapro.c | |||
@@ -1,7 +1,7 @@ | |||
1 | /* | 1 | /* |
2 | i2c-viapro.c - Part of lm_sensors, Linux kernel modules for hardware | 2 | i2c-viapro.c - Part of lm_sensors, Linux kernel modules for hardware |
3 | monitoring | 3 | monitoring |
4 | Copyright (c) 1998 - 2002 Frodo Looijaard <frodol@dds.nl>, | 4 | Copyright (c) 1998 - 2002 Frodo Looijaard <frodol@dds.nl>, |
5 | Philip Edelbrock <phil@netroedge.com>, Kyösti Mälkki <kmalkki@cc.hut.fi>, | 5 | Philip Edelbrock <phil@netroedge.com>, Kyösti Mälkki <kmalkki@cc.hut.fi>, |
6 | Mark D. Studebaker <mdsxyz123@yahoo.com> | 6 | Mark D. Studebaker <mdsxyz123@yahoo.com> |
7 | 7 | ||
@@ -46,9 +46,9 @@ | |||
46 | 46 | ||
47 | static struct pci_dev *vt596_pdev; | 47 | static struct pci_dev *vt596_pdev; |
48 | 48 | ||
49 | #define SMBBA1 0x90 | 49 | #define SMBBA1 0x90 |
50 | #define SMBBA2 0x80 | 50 | #define SMBBA2 0x80 |
51 | #define SMBBA3 0xD0 | 51 | #define SMBBA3 0xD0 |
52 | 52 | ||
53 | /* SMBus address offsets */ | 53 | /* SMBus address offsets */ |
54 | static unsigned short vt596_smba; | 54 | static unsigned short vt596_smba; |
@@ -68,26 +68,26 @@ static unsigned short vt596_smba; | |||
68 | /* PCI Address Constants */ | 68 | /* PCI Address Constants */ |
69 | 69 | ||
70 | /* SMBus data in configuration space can be found in two places, | 70 | /* SMBus data in configuration space can be found in two places, |
71 | We try to select the better one*/ | 71 | We try to select the better one */ |
72 | 72 | ||
73 | static unsigned short smb_cf_hstcfg = 0xD2; | 73 | static unsigned short smb_cf_hstcfg = 0xD2; |
74 | 74 | ||
75 | #define SMBHSTCFG (smb_cf_hstcfg) | 75 | #define SMBHSTCFG (smb_cf_hstcfg) |
76 | #define SMBSLVC (smb_cf_hstcfg + 1) | 76 | #define SMBSLVC (smb_cf_hstcfg + 1) |
77 | #define SMBSHDW1 (smb_cf_hstcfg + 2) | 77 | #define SMBSHDW1 (smb_cf_hstcfg + 2) |
78 | #define SMBSHDW2 (smb_cf_hstcfg + 3) | 78 | #define SMBSHDW2 (smb_cf_hstcfg + 3) |
79 | #define SMBREV (smb_cf_hstcfg + 4) | 79 | #define SMBREV (smb_cf_hstcfg + 4) |
80 | 80 | ||
81 | /* Other settings */ | 81 | /* Other settings */ |
82 | #define MAX_TIMEOUT 500 | 82 | #define MAX_TIMEOUT 500 |
83 | #define ENABLE_INT9 0 | 83 | #define ENABLE_INT9 0 |
84 | 84 | ||
85 | /* VT82C596 constants */ | 85 | /* VT82C596 constants */ |
86 | #define VT596_QUICK 0x00 | 86 | #define VT596_QUICK 0x00 |
87 | #define VT596_BYTE 0x04 | 87 | #define VT596_BYTE 0x04 |
88 | #define VT596_BYTE_DATA 0x08 | 88 | #define VT596_BYTE_DATA 0x08 |
89 | #define VT596_WORD_DATA 0x0C | 89 | #define VT596_WORD_DATA 0x0C |
90 | #define VT596_BLOCK_DATA 0x14 | 90 | #define VT596_BLOCK_DATA 0x14 |
91 | 91 | ||
92 | 92 | ||
93 | /* If force is set to anything different from 0, we forcibly enable the | 93 | /* If force is set to anything different from 0, we forcibly enable the |
@@ -115,19 +115,19 @@ static int vt596_transaction(void) | |||
115 | int timeout = 0; | 115 | int timeout = 0; |
116 | 116 | ||
117 | dev_dbg(&vt596_adapter.dev, "Transaction (pre): CNT=%02x, CMD=%02x, " | 117 | dev_dbg(&vt596_adapter.dev, "Transaction (pre): CNT=%02x, CMD=%02x, " |
118 | "ADD=%02x, DAT0=%02x, DAT1=%02x\n", inb_p(SMBHSTCNT), | 118 | "ADD=%02x, DAT0=%02x, DAT1=%02x\n", inb_p(SMBHSTCNT), |
119 | inb_p(SMBHSTCMD), inb_p(SMBHSTADD), inb_p(SMBHSTDAT0), | 119 | inb_p(SMBHSTCMD), inb_p(SMBHSTADD), inb_p(SMBHSTDAT0), |
120 | inb_p(SMBHSTDAT1)); | 120 | inb_p(SMBHSTDAT1)); |
121 | 121 | ||
122 | /* Make sure the SMBus host is ready to start transmitting */ | 122 | /* Make sure the SMBus host is ready to start transmitting */ |
123 | if ((temp = inb_p(SMBHSTSTS)) & 0x1F) { | 123 | if ((temp = inb_p(SMBHSTSTS)) & 0x1F) { |
124 | dev_dbg(&vt596_adapter.dev, "SMBus busy (0x%02x). " | 124 | dev_dbg(&vt596_adapter.dev, "SMBus busy (0x%02x). " |
125 | "Resetting...\n", temp); | 125 | "Resetting...\n", temp); |
126 | 126 | ||
127 | outb_p(temp, SMBHSTSTS); | 127 | outb_p(temp, SMBHSTSTS); |
128 | if ((temp = inb_p(SMBHSTSTS)) & 0x1F) { | 128 | if ((temp = inb_p(SMBHSTSTS)) & 0x1F) { |
129 | dev_dbg(&vt596_adapter.dev, "Failed! (0x%02x)\n", temp); | 129 | dev_dbg(&vt596_adapter.dev, "Failed! (0x%02x)\n", temp); |
130 | 130 | ||
131 | return -1; | 131 | return -1; |
132 | } else { | 132 | } else { |
133 | dev_dbg(&vt596_adapter.dev, "Successfull!\n"); | 133 | dev_dbg(&vt596_adapter.dev, "Successfull!\n"); |
@@ -137,7 +137,7 @@ static int vt596_transaction(void) | |||
137 | /* start the transaction by setting bit 6 */ | 137 | /* start the transaction by setting bit 6 */ |
138 | outb_p(inb(SMBHSTCNT) | 0x040, SMBHSTCNT); | 138 | outb_p(inb(SMBHSTCNT) | 0x040, SMBHSTCNT); |
139 | 139 | ||
140 | /* We will always wait for a fraction of a second! | 140 | /* We will always wait for a fraction of a second! |
141 | I don't know if VIA needs this, Intel did */ | 141 | I don't know if VIA needs this, Intel did */ |
142 | do { | 142 | do { |
143 | msleep(1); | 143 | msleep(1); |
@@ -177,16 +177,16 @@ static int vt596_transaction(void) | |||
177 | 177 | ||
178 | dev_dbg(&vt596_adapter.dev, "Transaction (post): CNT=%02x, CMD=%02x, " | 178 | dev_dbg(&vt596_adapter.dev, "Transaction (post): CNT=%02x, CMD=%02x, " |
179 | "ADD=%02x, DAT0=%02x, DAT1=%02x\n", inb_p(SMBHSTCNT), | 179 | "ADD=%02x, DAT0=%02x, DAT1=%02x\n", inb_p(SMBHSTCNT), |
180 | inb_p(SMBHSTCMD), inb_p(SMBHSTADD), inb_p(SMBHSTDAT0), | 180 | inb_p(SMBHSTCMD), inb_p(SMBHSTADD), inb_p(SMBHSTDAT0), |
181 | inb_p(SMBHSTDAT1)); | 181 | inb_p(SMBHSTDAT1)); |
182 | 182 | ||
183 | return result; | 183 | return result; |
184 | } | 184 | } |
185 | 185 | ||
186 | /* Return -1 on error. */ | 186 | /* Return -1 on error. */ |
187 | static s32 vt596_access(struct i2c_adapter *adap, u16 addr, | 187 | static s32 vt596_access(struct i2c_adapter *adap, u16 addr, |
188 | unsigned short flags, char read_write, u8 command, | 188 | unsigned short flags, char read_write, u8 command, |
189 | int size, union i2c_smbus_data *data) | 189 | int size, union i2c_smbus_data *data) |
190 | { | 190 | { |
191 | int i, len; | 191 | int i, len; |
192 | 192 | ||
@@ -256,7 +256,7 @@ static s32 vt596_access(struct i2c_adapter *adap, u16 addr, | |||
256 | case VT596_BYTE: | 256 | case VT596_BYTE: |
257 | /* Where is the result put? I assume here it is in | 257 | /* Where is the result put? I assume here it is in |
258 | * SMBHSTDAT0 but it might just as well be in the | 258 | * SMBHSTDAT0 but it might just as well be in the |
259 | * SMBHSTCMD. No clue in the docs | 259 | * SMBHSTCMD. No clue in the docs |
260 | */ | 260 | */ |
261 | data->byte = inb_p(SMBHSTDAT0); | 261 | data->byte = inb_p(SMBHSTDAT0); |
262 | break; | 262 | break; |
@@ -302,7 +302,7 @@ static int __devinit vt596_probe(struct pci_dev *pdev, | |||
302 | { | 302 | { |
303 | unsigned char temp; | 303 | unsigned char temp; |
304 | int error = -ENODEV; | 304 | int error = -ENODEV; |
305 | 305 | ||
306 | /* Determine the address of the SMBus areas */ | 306 | /* Determine the address of the SMBus areas */ |
307 | if (force_addr) { | 307 | if (force_addr) { |
308 | vt596_smba = force_addr & 0xfff0; | 308 | vt596_smba = force_addr & 0xfff0; |
@@ -333,10 +333,10 @@ static int __devinit vt596_probe(struct pci_dev *pdev, | |||
333 | return -ENODEV; | 333 | return -ENODEV; |
334 | } | 334 | } |
335 | 335 | ||
336 | found: | 336 | found: |
337 | if (!request_region(vt596_smba, 8, "viapro-smbus")) { | 337 | if (!request_region(vt596_smba, 8, "viapro-smbus")) { |
338 | dev_err(&pdev->dev, "SMBus region 0x%x already in use!\n", | 338 | dev_err(&pdev->dev, "SMBus region 0x%x already in use!\n", |
339 | vt596_smba); | 339 | vt596_smba); |
340 | return -ENODEV; | 340 | return -ENODEV; |
341 | } | 341 | } |
342 | 342 | ||
@@ -348,13 +348,13 @@ static int __devinit vt596_probe(struct pci_dev *pdev, | |||
348 | pci_write_config_word(pdev, id->driver_data, vt596_smba); | 348 | pci_write_config_word(pdev, id->driver_data, vt596_smba); |
349 | pci_write_config_byte(pdev, SMBHSTCFG, temp | 0x01); | 349 | pci_write_config_byte(pdev, SMBHSTCFG, temp | 0x01); |
350 | dev_warn(&pdev->dev, "WARNING: SMBus interface set to new " | 350 | dev_warn(&pdev->dev, "WARNING: SMBus interface set to new " |
351 | "address 0x%04x!\n", vt596_smba); | 351 | "address 0x%04x!\n", vt596_smba); |
352 | } else if ((temp & 1) == 0) { | 352 | } else if ((temp & 1) == 0) { |
353 | if (force) { | 353 | if (force) { |
354 | /* NOTE: This assumes I/O space and other allocations | 354 | /* NOTE: This assumes I/O space and other allocations |
355 | * WERE done by the Bios! Don't complain if your | 355 | * WERE done by the Bios! Don't complain if your |
356 | * hardware does weird things after enabling this. | 356 | * hardware does weird things after enabling this. |
357 | * :') Check for Bios updates before resorting to | 357 | * :') Check for Bios updates before resorting to |
358 | * this. | 358 | * this. |
359 | */ | 359 | */ |
360 | pci_write_config_byte(pdev, SMBHSTCFG, temp | 1); | 360 | pci_write_config_byte(pdev, SMBHSTCFG, temp | 1); |
@@ -381,8 +381,8 @@ static int __devinit vt596_probe(struct pci_dev *pdev, | |||
381 | 381 | ||
382 | vt596_adapter.dev.parent = &pdev->dev; | 382 | vt596_adapter.dev.parent = &pdev->dev; |
383 | snprintf(vt596_adapter.name, I2C_NAME_SIZE, | 383 | snprintf(vt596_adapter.name, I2C_NAME_SIZE, |
384 | "SMBus Via Pro adapter at %04x", vt596_smba); | 384 | "SMBus Via Pro adapter at %04x", vt596_smba); |
385 | 385 | ||
386 | vt596_pdev = pci_dev_get(pdev); | 386 | vt596_pdev = pci_dev_get(pdev); |
387 | if (i2c_add_adapter(&vt596_adapter)) { | 387 | if (i2c_add_adapter(&vt596_adapter)) { |
388 | pci_dev_put(vt596_pdev); | 388 | pci_dev_put(vt596_pdev); |
@@ -395,7 +395,7 @@ static int __devinit vt596_probe(struct pci_dev *pdev, | |||
395 | */ | 395 | */ |
396 | return -ENODEV; | 396 | return -ENODEV; |
397 | 397 | ||
398 | release_region: | 398 | release_region: |
399 | release_region(vt596_smba, 8); | 399 | release_region(vt596_smba, 8); |
400 | return error; | 400 | return error; |
401 | } | 401 | } |
@@ -420,7 +420,7 @@ static struct pci_device_id vt596_ids[] = { | |||
420 | { 0, } | 420 | { 0, } |
421 | }; | 421 | }; |
422 | 422 | ||
423 | MODULE_DEVICE_TABLE (pci, vt596_ids); | 423 | MODULE_DEVICE_TABLE(pci, vt596_ids); |
424 | 424 | ||
425 | static struct pci_driver vt596_driver = { | 425 | static struct pci_driver vt596_driver = { |
426 | .name = "vt596_smbus", | 426 | .name = "vt596_smbus", |