]> git.karo-electronics.de Git - karo-tx-linux.git/commitdiff
USB: ehci-mv: remove unnecessary platform_set_drvdata()
authorJingoo Han <jg1.han@samsung.com>
Mon, 6 May 2013 10:05:54 +0000 (19:05 +0900)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 17 May 2013 00:47:14 +0000 (17:47 -0700)
The driver core clears the driver data to NULL after device_release
or on probe failure, since commit 0998d0631001288a5974afc0b2a5f568bcdecb4d
(device-core: Ensure drvdata = NULL when no driver is bound).
Thus, it is not needed to manually clear the device driver data to NULL.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/host/ehci-mv.c

index 402062973f032ca58e048a0d331ce526be4b8e0c..915c2db96dce8fdc3dded7b17915e501367f356e 100644 (file)
@@ -166,14 +166,14 @@ static int mv_ehci_probe(struct platform_device *pdev)
        if (IS_ERR(ehci_mv->clk)) {
                dev_err(&pdev->dev, "error getting clock\n");
                retval = PTR_ERR(ehci_mv->clk);
-               goto err_clear_drvdata;
+               goto err_put_hcd;
        }
 
        r = platform_get_resource_byname(pdev, IORESOURCE_MEM, "phyregs");
        if (r == NULL) {
                dev_err(&pdev->dev, "no phy I/O memory resource defined\n");
                retval = -ENODEV;
-               goto err_clear_drvdata;
+               goto err_put_hcd;
        }
 
        ehci_mv->phy_regs = devm_ioremap(&pdev->dev, r->start,
@@ -181,14 +181,14 @@ static int mv_ehci_probe(struct platform_device *pdev)
        if (ehci_mv->phy_regs == 0) {
                dev_err(&pdev->dev, "failed to map phy I/O memory\n");
                retval = -EFAULT;
-               goto err_clear_drvdata;
+               goto err_put_hcd;
        }
 
        r = platform_get_resource_byname(pdev, IORESOURCE_MEM, "capregs");
        if (!r) {
                dev_err(&pdev->dev, "no I/O memory resource defined\n");
                retval = -ENODEV;
-               goto err_clear_drvdata;
+               goto err_put_hcd;
        }
 
        ehci_mv->cap_regs = devm_ioremap(&pdev->dev, r->start,
@@ -196,13 +196,13 @@ static int mv_ehci_probe(struct platform_device *pdev)
        if (ehci_mv->cap_regs == NULL) {
                dev_err(&pdev->dev, "failed to map I/O memory\n");
                retval = -EFAULT;
-               goto err_clear_drvdata;
+               goto err_put_hcd;
        }
 
        retval = mv_ehci_enable(ehci_mv);
        if (retval) {
                dev_err(&pdev->dev, "init phy error %d\n", retval);
-               goto err_clear_drvdata;
+               goto err_put_hcd;
        }
 
        offset = readl(ehci_mv->cap_regs) & CAPLENGTH_MASK;
@@ -274,8 +274,6 @@ err_set_vbus:
                pdata->set_vbus(0);
 err_disable_clk:
        mv_ehci_disable(ehci_mv);
-err_clear_drvdata:
-       platform_set_drvdata(pdev, NULL);
 err_put_hcd:
        usb_put_hcd(hcd);
 
@@ -300,8 +298,6 @@ static int mv_ehci_remove(struct platform_device *pdev)
                mv_ehci_disable(ehci_mv);
        }
 
-       platform_set_drvdata(pdev, NULL);
-
        usb_put_hcd(hcd);
 
        return 0;