Commit 1a641fce authored by Jean Delvare's avatar Jean Delvare Committed by Linus Torvalds

hwmon/w83627ehf: Don't redefine REGION_OFFSET

On ia64, kernel headers define REGION_OFFSET so we can't use that.
Reported by Andrew Morton.
Signed-off-by: default avatarJean Delvare <khali@linux-fr.org>
Acked-by: default avatarDavid Hubbard <david.c.hubbard@gmail.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 179fb0c7
...@@ -121,9 +121,9 @@ superio_exit(void) ...@@ -121,9 +121,9 @@ superio_exit(void)
* ISA constants * ISA constants
*/ */
#define REGION_ALIGNMENT ~7 #define IOREGION_ALIGNMENT ~7
#define REGION_OFFSET 5 #define IOREGION_OFFSET 5
#define REGION_LENGTH 2 #define IOREGION_LENGTH 2
#define ADDR_REG_OFFSET 5 #define ADDR_REG_OFFSET 5
#define DATA_REG_OFFSET 6 #define DATA_REG_OFFSET 6
...@@ -1194,7 +1194,7 @@ static int w83627ehf_detect(struct i2c_adapter *adapter) ...@@ -1194,7 +1194,7 @@ static int w83627ehf_detect(struct i2c_adapter *adapter)
u8 fan4pin, fan5pin; u8 fan4pin, fan5pin;
int i, err = 0; int i, err = 0;
if (!request_region(address + REGION_OFFSET, REGION_LENGTH, if (!request_region(address + IOREGION_OFFSET, IOREGION_LENGTH,
w83627ehf_driver.driver.name)) { w83627ehf_driver.driver.name)) {
err = -EBUSY; err = -EBUSY;
goto exit; goto exit;
...@@ -1322,7 +1322,7 @@ exit_remove: ...@@ -1322,7 +1322,7 @@ exit_remove:
exit_free: exit_free:
kfree(data); kfree(data);
exit_release: exit_release:
release_region(address + REGION_OFFSET, REGION_LENGTH); release_region(address + IOREGION_OFFSET, IOREGION_LENGTH);
exit: exit:
return err; return err;
} }
...@@ -1337,7 +1337,7 @@ static int w83627ehf_detach_client(struct i2c_client *client) ...@@ -1337,7 +1337,7 @@ static int w83627ehf_detach_client(struct i2c_client *client)
if ((err = i2c_detach_client(client))) if ((err = i2c_detach_client(client)))
return err; return err;
release_region(client->addr + REGION_OFFSET, REGION_LENGTH); release_region(client->addr + IOREGION_OFFSET, IOREGION_LENGTH);
kfree(data); kfree(data);
return 0; return 0;
...@@ -1380,7 +1380,7 @@ static int __init w83627ehf_find(int sioaddr, unsigned short *addr) ...@@ -1380,7 +1380,7 @@ static int __init w83627ehf_find(int sioaddr, unsigned short *addr)
superio_select(W83627EHF_LD_HWM); superio_select(W83627EHF_LD_HWM);
val = (superio_inb(SIO_REG_ADDR) << 8) val = (superio_inb(SIO_REG_ADDR) << 8)
| superio_inb(SIO_REG_ADDR + 1); | superio_inb(SIO_REG_ADDR + 1);
*addr = val & REGION_ALIGNMENT; *addr = val & IOREGION_ALIGNMENT;
if (*addr == 0) { if (*addr == 0) {
superio_exit(); superio_exit();
return -ENODEV; return -ENODEV;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment