diff options
| author | Julia Lawall <julia@diku.dk> | 2009-08-06 16:05:08 -0700 |
|---|---|---|
| committer | Jeff Garzik <jgarzik@redhat.com> | 2009-09-11 02:25:58 -0400 |
| commit | 041b5eac254107cd3ba60034c38a411531cc64ee (patch) | |
| tree | a1abcfd2b97985ca06504c493fda9cd4123307aa /drivers/ata/pata_octeon_cf.c | |
| parent | f90f613c3ed7381209de8aa36458442b5130170b (diff) | |
| download | op-kernel-dev-041b5eac254107cd3ba60034c38a411531cc64ee.zip op-kernel-dev-041b5eac254107cd3ba60034c38a411531cc64ee.tar.gz | |
drivers/ata: use resource_size
Use the function resource_size, which reduces the chance of introducing
off-by-one errors in calculating the resource size.
The semantic patch that makes this change is as follows:
(http://www.emn.fr/x-info/coccinelle/)
// <smpl>
@@
struct resource *res;
@@
- (res->end - res->start) + 1
+ resource_size(res)
// </smpl>
Signed-off-by: Julia Lawall <julia@diku.dk>
Cc: Jeff Garzik <jeff@garzik.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Diffstat (limited to 'drivers/ata/pata_octeon_cf.c')
| -rw-r--r-- | drivers/ata/pata_octeon_cf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/ata/pata_octeon_cf.c b/drivers/ata/pata_octeon_cf.c index e807cc3..d6f6956 100644 --- a/drivers/ata/pata_octeon_cf.c +++ b/drivers/ata/pata_octeon_cf.c @@ -841,7 +841,7 @@ static int __devinit octeon_cf_probe(struct platform_device *pdev) ocd = pdev->dev.platform_data; cs0 = devm_ioremap_nocache(&pdev->dev, res_cs0->start, - res_cs0->end - res_cs0->start + 1); + resource_size(res_cs0)); if (!cs0) return -ENOMEM; |
