diff options
author | Linus Walleij <linus.walleij@stericsson.com> | 2009-06-13 18:20:36 -0400 |
---|---|---|
committer | Ben Dooks <ben-linux@fluff.org> | 2009-06-16 17:42:16 -0400 |
commit | c6ffddea36dd576b70dfbd10eb5d2b287b786dca (patch) | |
tree | 9849cac5ca77534e38b4cb3c35a09d980ae84ff3 /drivers/i2c/busses/i2c-pxa.c | |
parent | 18904c0ecdf2cf22347da2adc4b273e9570333d8 (diff) |
i2c: Use resource_size macro
This replace all instances in the i2c busses tree of
res->end - res->start + 1 with the handy macro resource_size(res)
from ioport.h (coming in from platform_device.h).
This was created with a simple
sed -i -e 's/\([a-z]*\)->end *- *[a-z]*->start *+ *1/resource_size(\1)/g'
Then manually replacing the PXA redefiniton of the same kind
of macro manually. Recompiled some ARM defconfigs I could find to
make a rough test so it shouldn't break anything, though I
couldn't see exactly which configs you need for all the drivers.
Signed-off-by: Linus Walleij <linus.walleij@stericsson.com>
Signed-off-by: Ben Dooks <ben-linux@fluff.org>
Diffstat (limited to 'drivers/i2c/busses/i2c-pxa.c')
-rw-r--r-- | drivers/i2c/busses/i2c-pxa.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/i2c/busses/i2c-pxa.c b/drivers/i2c/busses/i2c-pxa.c index 035a6c7e59df..762e1e530882 100644 --- a/drivers/i2c/busses/i2c-pxa.c +++ b/drivers/i2c/busses/i2c-pxa.c | |||
@@ -993,7 +993,6 @@ static const struct i2c_algorithm i2c_pxa_pio_algorithm = { | |||
993 | .functionality = i2c_pxa_functionality, | 993 | .functionality = i2c_pxa_functionality, |
994 | }; | 994 | }; |
995 | 995 | ||
996 | #define res_len(r) ((r)->end - (r)->start + 1) | ||
997 | static int i2c_pxa_probe(struct platform_device *dev) | 996 | static int i2c_pxa_probe(struct platform_device *dev) |
998 | { | 997 | { |
999 | struct pxa_i2c *i2c; | 998 | struct pxa_i2c *i2c; |
@@ -1008,7 +1007,7 @@ static int i2c_pxa_probe(struct platform_device *dev) | |||
1008 | if (res == NULL || irq < 0) | 1007 | if (res == NULL || irq < 0) |
1009 | return -ENODEV; | 1008 | return -ENODEV; |
1010 | 1009 | ||
1011 | if (!request_mem_region(res->start, res_len(res), res->name)) | 1010 | if (!request_mem_region(res->start, resource_size(res), res->name)) |
1012 | return -ENOMEM; | 1011 | return -ENOMEM; |
1013 | 1012 | ||
1014 | i2c = kzalloc(sizeof(struct pxa_i2c), GFP_KERNEL); | 1013 | i2c = kzalloc(sizeof(struct pxa_i2c), GFP_KERNEL); |
@@ -1038,7 +1037,7 @@ static int i2c_pxa_probe(struct platform_device *dev) | |||
1038 | goto eclk; | 1037 | goto eclk; |
1039 | } | 1038 | } |
1040 | 1039 | ||
1041 | i2c->reg_base = ioremap(res->start, res_len(res)); | 1040 | i2c->reg_base = ioremap(res->start, resource_size(res)); |
1042 | if (!i2c->reg_base) { | 1041 | if (!i2c->reg_base) { |
1043 | ret = -EIO; | 1042 | ret = -EIO; |
1044 | goto eremap; | 1043 | goto eremap; |
@@ -1046,7 +1045,7 @@ static int i2c_pxa_probe(struct platform_device *dev) | |||
1046 | i2c->reg_shift = REG_SHIFT(id->driver_data); | 1045 | i2c->reg_shift = REG_SHIFT(id->driver_data); |
1047 | 1046 | ||
1048 | i2c->iobase = res->start; | 1047 | i2c->iobase = res->start; |
1049 | i2c->iosize = res_len(res); | 1048 | i2c->iosize = resource_size(res); |
1050 | 1049 | ||
1051 | i2c->irq = irq; | 1050 | i2c->irq = irq; |
1052 | 1051 | ||
@@ -1110,7 +1109,7 @@ eremap: | |||
1110 | eclk: | 1109 | eclk: |
1111 | kfree(i2c); | 1110 | kfree(i2c); |
1112 | emalloc: | 1111 | emalloc: |
1113 | release_mem_region(res->start, res_len(res)); | 1112 | release_mem_region(res->start, resource_size(res)); |
1114 | return ret; | 1113 | return ret; |
1115 | } | 1114 | } |
1116 | 1115 | ||