2011-06-06 20:52:35

by Hartley Sweeten

Subject: [PATCH] gpio: vr41xx_giu: use resource_size()

Signed-off-by: H Hartley Sweeten <[email protected]>
Cc: Grant Likely <[email protected]>

---

diff --git a/drivers/gpio/vr41xx_giu.c b/drivers/gpio/vr41xx_giu.c
index a365be0..98723cb 100644
--- a/drivers/gpio/vr41xx_giu.c
+++ b/drivers/gpio/vr41xx_giu.c
@@ -518,7 +518,7 @@ static int __devinit giu_probe(struct platform_device *pdev)
if (!res)
return -EBUSY;

- giu_base = ioremap(res->start, res->end - res->start + 1);
+ giu_base = ioremap(res->start, resource_size(res));
if (!giu_base)
return -ENOMEM;

2011-06-27 23:03:31

by Grant Likely

Subject: Re: [PATCH] gpio: vr41xx_giu: use resource_size()

On Mon, Jun 06, 2011 at 01:52:19PM -0700, H Hartley Sweeten wrote:
> Signed-off-by: H Hartley Sweeten <[email protected]>
> Cc: Grant Likely <[email protected]>

Thanks. I've already merged a patch that makes this change.

g.

>
> ---
>
> diff --git a/drivers/gpio/vr41xx_giu.c b/drivers/gpio/vr41xx_giu.c
> index a365be0..98723cb 100644
> --- a/drivers/gpio/vr41xx_giu.c
> +++ b/drivers/gpio/vr41xx_giu.c
> @@ -518,7 +518,7 @@ static int __devinit giu_probe(struct platform_device *pdev)
> if (!res)
> return -EBUSY;
>
> - giu_base = ioremap(res->start, res->end - res->start + 1);
> + giu_base = ioremap(res->start, resource_size(res));
> if (!giu_base)
> return -ENOMEM;
>