Messages in this thread |  | | | Date | Fri, 10 Jun 2011 23:48:32 -0600 | | From | Grant Likely <> | | Subject | Re: [PATCH] spi: Convert uses of struct resource * to resource_size(ptr) |
| |
On Fri, Jun 10, 2011 at 06:11:25PM -0700, Joe Perches wrote: > Done via coccinelle scripts like: > > @@ > struct resource *ptr; > @@ > > - ptr->end - ptr->start + 1 > + resource_size(ptr) > > and some grep and typing. > > Mostly uncompiled, no cross-compilers. > > Signed-off-by: Joe Perches <joe@perches.com>
I've put it in my tree, we'll see how it shakes out.
g.
> > --- > > This is a remainder patch for files that > have been moved around since 3.0-rc1 > > Jiri Kosina has picked up the majority of > these conversions via the trivial tree. > > drivers/spi/spi-ath79.c | 2 +- > drivers/spi/spi-omap2-mcspi.c | 10 +++++----- > drivers/spi/spi-orion.c | 6 +++--- > drivers/spi/spi-ppc4xx.c | 2 +- > drivers/spi/spi-tegra.c | 8 ++++---- > 5 files changed, 14 insertions(+), 14 deletions(-) > > diff --git a/drivers/spi/spi-ath79.c b/drivers/spi/spi-ath79.c > index fcff810..03019bf 100644 > --- a/drivers/spi/spi-ath79.c > +++ b/drivers/spi/spi-ath79.c > @@ -232,7 +232,7 @@ static __devinit int ath79_spi_probe(struct platform_device *pdev) > goto err_put_master; > } > > - sp->base = ioremap(r->start, r->end - r->start + 1); > + sp->base = ioremap(r->start, resource_size(r)); > if (!sp->base) { > ret = -ENXIO; > goto err_put_master; > diff --git a/drivers/spi/spi-omap2-mcspi.c b/drivers/spi/spi-omap2-mcspi.c > index 969cdd2..fde3a2d 100644 > --- a/drivers/spi/spi-omap2-mcspi.c > +++ b/drivers/spi/spi-omap2-mcspi.c > @@ -1116,8 +1116,8 @@ static int __init omap2_mcspi_probe(struct platform_device *pdev) > status = -ENODEV; > goto err1; > } > - if (!request_mem_region(r->start, (r->end - r->start) + 1, > - dev_name(&pdev->dev))) { > + if (!request_mem_region(r->start, resource_size(r), > + dev_name(&pdev->dev))) { > status = -EBUSY; > goto err1; > } > @@ -1125,7 +1125,7 @@ static int __init omap2_mcspi_probe(struct platform_device *pdev) > r->start += pdata->regs_offset; > r->end += pdata->regs_offset; > mcspi->phys = r->start; > - mcspi->base = ioremap(r->start, r->end - r->start + 1); > + mcspi->base = ioremap(r->start, resource_size(r)); > if (!mcspi->base) { > dev_dbg(&pdev->dev, "can't ioremap MCSPI\n"); > status = -ENOMEM; > @@ -1190,7 +1190,7 @@ err4: > err3: > kfree(mcspi->dma_channels); > err2: > - release_mem_region(r->start, (r->end - r->start) + 1); > + release_mem_region(r->start, resource_size(r)); > iounmap(mcspi->base); > err1: > return status; > @@ -1210,7 +1210,7 @@ static int __exit omap2_mcspi_remove(struct platform_device *pdev) > > omap2_mcspi_disable_clocks(mcspi); > r = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - release_mem_region(r->start, (r->end - r->start) + 1); > + release_mem_region(r->start, resource_size(r)); > > base = mcspi->base; > spi_unregister_master(master); > diff --git a/drivers/spi/spi-orion.c b/drivers/spi/spi-orion.c > index d482628..9421a39 100644 > --- a/drivers/spi/spi-orion.c > +++ b/drivers/spi/spi-orion.c > @@ -489,7 +489,7 @@ static int __init orion_spi_probe(struct platform_device *pdev) > goto out; > } > > - if (!request_mem_region(r->start, (r->end - r->start) + 1, > + if (!request_mem_region(r->start, resource_size(r), > dev_name(&pdev->dev))) { > status = -EBUSY; > goto out; > @@ -511,7 +511,7 @@ static int __init orion_spi_probe(struct platform_device *pdev) > return status; > > out_rel_mem: > - release_mem_region(r->start, (r->end - r->start) + 1); > + release_mem_region(r->start, resource_size(r)); > > out: > spi_master_put(master); > @@ -531,7 +531,7 @@ static int __exit orion_spi_remove(struct platform_device *pdev) > cancel_work_sync(&spi->work); > > r = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - release_mem_region(r->start, (r->end - r->start) + 1); > + release_mem_region(r->start, resource_size(r)); > > spi_unregister_master(master); > > diff --git a/drivers/spi/spi-ppc4xx.c b/drivers/spi/spi-ppc4xx.c > index 2a298c0..b267fd9 100644 > --- a/drivers/spi/spi-ppc4xx.c > +++ b/drivers/spi/spi-ppc4xx.c > @@ -502,7 +502,7 @@ static int __init spi_ppc4xx_of_probe(struct platform_device *op) > goto free_gpios; > } > hw->mapbase = resource.start; > - hw->mapsize = resource.end - resource.start + 1; > + hw->mapsize = resource_size(&resource); > > /* Sanity check */ > if (hw->mapsize < sizeof(struct spi_ppc4xx_regs)) { > diff --git a/drivers/spi/spi-tegra.c b/drivers/spi/spi-tegra.c > index 6c3aa6e..8e30727 100644 > --- a/drivers/spi/spi-tegra.c > +++ b/drivers/spi/spi-tegra.c > @@ -498,14 +498,14 @@ static int __init spi_tegra_probe(struct platform_device *pdev) > goto err0; > } > > - if (!request_mem_region(r->start, (r->end - r->start) + 1, > + if (!request_mem_region(r->start, resource_size(r), > dev_name(&pdev->dev))) { > ret = -EBUSY; > goto err0; > } > > tspi->phys = r->start; > - tspi->base = ioremap(r->start, r->end - r->start + 1); > + tspi->base = ioremap(r->start, resource_size(r)); > if (!tspi->base) { > dev_err(&pdev->dev, "can't ioremap iomem\n"); > ret = -ENOMEM; > @@ -563,7 +563,7 @@ err3: > err2: > iounmap(tspi->base); > err1: > - release_mem_region(r->start, (r->end - r->start) + 1); > + release_mem_region(r->start, resource_size(r)); > err0: > spi_master_put(master); > return ret; > @@ -588,7 +588,7 @@ static int __devexit spi_tegra_remove(struct platform_device *pdev) > iounmap(tspi->base); > > r = platform_get_resource(pdev, IORESOURCE_MEM, 0); > - release_mem_region(r->start, (r->end - r->start) + 1); > + release_mem_region(r->start, resource_size(r)); > > return 0; > } > -- > 1.7.6.rc1 >
|  |