From: Varka Bhadram <varkabhad...@gmail.com>

Sanity check on resource happening with devm_ioremap_resource().

Signed-off-by: Varka Bhadram <var...@cdac.in>
Acked-by: Alan Stern <st...@rowland.harvard.edu>
---
 drivers/usb/host/ohci-pxa27x.c |   11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

diff --git a/drivers/usb/host/ohci-pxa27x.c b/drivers/usb/host/ohci-pxa27x.c
index e68f3d0..1339981 100644
--- a/drivers/usb/host/ohci-pxa27x.c
+++ b/drivers/usb/host/ohci-pxa27x.c
@@ -447,20 +447,13 @@ int usb_hcd_pxa27x_probe (const struct hc_driver *driver, 
struct platform_device
                return -ENOMEM;
 
        r = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-       if (!r) {
-               pr_err("no resource of IORESOURCE_MEM");
-               retval = -ENXIO;
-               goto err;
-       }
-
-       hcd->rsrc_start = r->start;
-       hcd->rsrc_len = resource_size(r);
-
        hcd->regs = devm_ioremap_resource(&pdev->dev, r);
        if (IS_ERR(hcd->regs)) {
                retval = PTR_ERR(hcd->regs);
                goto err;
        }
+       hcd->rsrc_start = r->start;
+       hcd->rsrc_len = resource_size(r);
 
        /* initialize "struct pxa27x_ohci" */
        pxa_ohci = to_pxa27x_ohci(hcd);
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to