Re: [PATCH 3/3] usb: ohci-at91: use device managed clk retrieval

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/12/2013 15:07, Boris BREZILLON :
Replace clk_get calls by devm_clk_get calls.

Signed-off-by: Boris BREZILLON <b.brezillon@xxxxxxxxxxx>
Tested-by: Robert Nelson <robertcnelson@xxxxxxxxx>

Acked-by: Nicolas Ferre <nicolas.ferre@xxxxxxxxx>

Thanks Boris for these fixes.

Alan, Greg, can you take the whole series as fixes for 3.13?

Thanks, best regards,

---
  drivers/usb/host/ohci-at91.c |   32 ++++++++------------------------
  1 file changed, 8 insertions(+), 24 deletions(-)

diff --git a/drivers/usb/host/ohci-at91.c b/drivers/usb/host/ohci-at91.c
index c406f1e..3652962 100644
--- a/drivers/usb/host/ohci-at91.c
+++ b/drivers/usb/host/ohci-at91.c
@@ -164,30 +164,30 @@ static int usb_hcd_at91_probe(const struct hc_driver *driver,
  		goto err;
  	}

-	iclk = clk_get(&pdev->dev, "ohci_clk");
+	iclk = devm_clk_get(dev, "ohci_clk");
  	if (IS_ERR(iclk)) {
-		dev_err(&pdev->dev, "failed to get ohci_clk\n");
+		dev_err(dev, "failed to get ohci_clk\n");
  		retval = PTR_ERR(iclk);
  		goto err;
  	}
-	fclk = clk_get(&pdev->dev, "uhpck");
+	fclk = devm_clk_get(dev, "uhpck");
  	if (IS_ERR(fclk)) {
  		dev_err(&pdev->dev, "failed to get uhpck\n");
  		retval = PTR_ERR(fclk);
-		goto err4;
+		goto err;
  	}
-	hclk = clk_get(&pdev->dev, "hclk");
+	hclk = devm_clk_get(dev, "hclk");
  	if (IS_ERR(hclk)) {
  		dev_err(&pdev->dev, "failed to get hclk\n");
  		retval = PTR_ERR(hclk);
-		goto err5;
+		goto err;
  	}
  	if (IS_ENABLED(CONFIG_COMMON_CLK)) {
-		uclk = clk_get(&pdev->dev, "usb_clk");
+		uclk = devm_clk_get(&pdev->dev, "usb_clk");
  		if (IS_ERR(uclk)) {
  			dev_err(&pdev->dev, "failed to get uclk\n");
  			retval = PTR_ERR(uclk);
-			goto err6;
+			goto err;
  		}
  	}

@@ -203,15 +203,6 @@ static int usb_hcd_at91_probe(const struct hc_driver *driver,
  	/* Error handling */
  	at91_stop_hc(pdev);

-	if (IS_ENABLED(CONFIG_COMMON_CLK))
-		clk_put(uclk);
- err6:
-	clk_put(hclk);
- err5:
-	clk_put(fclk);
- err4:
-	clk_put(iclk);
-
   err:
  	usb_put_hcd(hcd);
  	return retval;
@@ -236,13 +227,6 @@ static void usb_hcd_at91_remove(struct usb_hcd *hcd,
  	usb_remove_hcd(hcd);
  	at91_stop_hc(pdev);
  	usb_put_hcd(hcd);
-
-	if (IS_ENABLED(CONFIG_COMMON_CLK))
-		clk_put(uclk);
-	clk_put(hclk);
-	clk_put(fclk);
-	clk_put(iclk);
-	fclk = iclk = hclk = NULL;
  }

  /*-------------------------------------------------------------------------*/



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




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux