2009-12-14 23:05:31

by Hartley Sweeten

Subject: [PATCH] pxa27x_udc.c: use resource_size()

Use resource_size().

Signed-off-by: H Hartley Sweeten <[email protected]>
Cc: David Brownell <[email protected]>
Cc: Robert Jarzmik <[email protected]>

---

diff --git a/drivers/usb/gadget/pxa27x_udc.c b/drivers/usb/gadget/pxa27x_udc.c
index adda120..72c89d9 100644
--- a/drivers/usb/gadget/pxa27x_udc.c
+++ b/drivers/usb/gadget/pxa27x_udc.c
@@ -2439,7 +2439,7 @@ static int __init pxa_udc_probe(struct platform_device *pdev)
}

retval = -ENOMEM;
- udc->regs = ioremap(regs->start, regs->end - regs->start + 1);
+ udc->regs = ioremap(regs->start, resource_size(regs));
if (!udc->regs) {
dev_err(&pdev->dev, "Unable to map UDC I/O memory\n");
goto err_map;

2009-12-16 20:46:32

by Robert Jarzmik

Subject: Re: [PATCH] pxa27x_udc.c: use resource_size()

"H Hartley Sweeten" <[email protected]> writes:

> Use resource_size().
>
> Signed-off-by: H Hartley Sweeten <[email protected]>
> Cc: David Brownell <[email protected]>
> Cc: Robert Jarzmik <[email protected]>
>
> ---
>
> diff --git a/drivers/usb/gadget/pxa27x_udc.c b/drivers/usb/gadget/pxa27x_udc.c
> index adda120..72c89d9 100644
> --- a/drivers/usb/gadget/pxa27x_udc.c
> +++ b/drivers/usb/gadget/pxa27x_udc.c
> @@ -2439,7 +2439,7 @@ static int __init pxa_udc_probe(struct platform_device *pdev)
> }
>
> retval = -ENOMEM;
> - udc->regs = ioremap(regs->start, regs->end - regs->start + 1);
> + udc->regs = ioremap(regs->start, resource_size(regs));
> if (!udc->regs) {
> dev_err(&pdev->dev, "Unable to map UDC I/O memory\n");
> goto err_map;

Acked-by: Robert Jarzmik <[email protected]>

--
Robert