RE: [PATCH] usb: ehci-s5p: use clk_prepare_enable and clk_disable_unprepare

2012-11-12 Thread Kukjin Kim
Thomas Abraham wrote:
 
 Convert clk_enable/clk_disable to clk_prepare_enable/clk_disable_unprepare
 calls as required by common clock framework.
 
 Signed-off-by: Thomas Abraham thomas.abra...@linaro.org

(+ Alan Stern)

Acked-by: Kukjin Kim kgene@samsung.com

Thanks.

Best regards,
Kgene.
--
Kukjin Kim kgene@samsung.com, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

 ---
  drivers/usb/host/ehci-s5p.c |   10 +-
  1 files changed, 5 insertions(+), 5 deletions(-)
 
 diff --git a/drivers/usb/host/ehci-s5p.c b/drivers/usb/host/ehci-s5p.c
 index 85b74be..abc178d 100644
 --- a/drivers/usb/host/ehci-s5p.c
 +++ b/drivers/usb/host/ehci-s5p.c
 @@ -136,7 +136,7 @@ static int __devinit s5p_ehci_probe(struct
 platform_device *pdev)
   goto fail_clk;
   }
 
 - err = clk_enable(s5p_ehci-clk);
 + err = clk_prepare_enable(s5p_ehci-clk);
   if (err)
   goto fail_clk;
 
 @@ -183,7 +183,7 @@ static int __devinit s5p_ehci_probe(struct
 platform_device *pdev)
   return 0;
 
  fail_io:
 - clk_disable(s5p_ehci-clk);
 + clk_disable_unprepare(s5p_ehci-clk);
  fail_clk:
   usb_put_hcd(hcd);
   return err;
 @@ -200,7 +200,7 @@ static int __devexit s5p_ehci_remove(struct
 platform_device *pdev)
   if (pdata  pdata-phy_exit)
   pdata-phy_exit(pdev, S5P_USB_PHY_HOST);
 
 - clk_disable(s5p_ehci-clk);
 + clk_disable_unprepare(s5p_ehci-clk);
 
   usb_put_hcd(hcd);
 
 @@ -231,7 +231,7 @@ static int s5p_ehci_suspend(struct device *dev)
   if (pdata  pdata-phy_exit)
   pdata-phy_exit(pdev, S5P_USB_PHY_HOST);
 
 - clk_disable(s5p_ehci-clk);
 + clk_disable_unprepare(s5p_ehci-clk);
 
   return rc;
  }
 @@ -243,7 +243,7 @@ static int s5p_ehci_resume(struct device *dev)
   struct platform_device *pdev = to_platform_device(dev);
   struct s5p_ehci_platdata *pdata = pdev-dev.platform_data;
 
 - clk_enable(s5p_ehci-clk);
 + clk_prepare_enable(s5p_ehci-clk);
 
   if (pdata  pdata-phy_init)
   pdata-phy_init(pdev, S5P_USB_PHY_HOST);
 --
 1.7.4.1

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


RE: [PATCH] usb: ehci-s5p: use clk_prepare_enable and clk_disable_unprepare

2012-10-03 Thread Jingoo Han
On Wednesday, October 03, 2012 8:41 AM Thomas Abraham wrote
 
 Convert clk_enable/clk_disable to clk_prepare_enable/clk_disable_unprepare
 calls as required by common clock framework.
 
 Signed-off-by: Thomas Abraham thomas.abra...@linaro.org


It looks good. Also, I have tested this patch with Exynos4210.

Acked-by: Jingoo Han jg1@samsung.com


Best regards,
Jingoo Han

 ---
  drivers/usb/host/ehci-s5p.c |   10 +-
  1 files changed, 5 insertions(+), 5 deletions(-)
 
 diff --git a/drivers/usb/host/ehci-s5p.c b/drivers/usb/host/ehci-s5p.c
 index 85b74be..abc178d 100644
 --- a/drivers/usb/host/ehci-s5p.c
 +++ b/drivers/usb/host/ehci-s5p.c
 @@ -136,7 +136,7 @@ static int __devinit s5p_ehci_probe(struct 
 platform_device *pdev)
   goto fail_clk;
   }
 
 - err = clk_enable(s5p_ehci-clk);
 + err = clk_prepare_enable(s5p_ehci-clk);
   if (err)
   goto fail_clk;
 
 @@ -183,7 +183,7 @@ static int __devinit s5p_ehci_probe(struct 
 platform_device *pdev)
   return 0;
 
  fail_io:
 - clk_disable(s5p_ehci-clk);
 + clk_disable_unprepare(s5p_ehci-clk);
  fail_clk:
   usb_put_hcd(hcd);
   return err;
 @@ -200,7 +200,7 @@ static int __devexit s5p_ehci_remove(struct 
 platform_device *pdev)
   if (pdata  pdata-phy_exit)
   pdata-phy_exit(pdev, S5P_USB_PHY_HOST);
 
 - clk_disable(s5p_ehci-clk);
 + clk_disable_unprepare(s5p_ehci-clk);
 
   usb_put_hcd(hcd);
 
 @@ -231,7 +231,7 @@ static int s5p_ehci_suspend(struct device *dev)
   if (pdata  pdata-phy_exit)
   pdata-phy_exit(pdev, S5P_USB_PHY_HOST);
 
 - clk_disable(s5p_ehci-clk);
 + clk_disable_unprepare(s5p_ehci-clk);
 
   return rc;
  }
 @@ -243,7 +243,7 @@ static int s5p_ehci_resume(struct device *dev)
   struct platform_device *pdev = to_platform_device(dev);
   struct s5p_ehci_platdata *pdata = pdev-dev.platform_data;
 
 - clk_enable(s5p_ehci-clk);
 + clk_prepare_enable(s5p_ehci-clk);
 
   if (pdata  pdata-phy_init)
   pdata-phy_init(pdev, S5P_USB_PHY_HOST);
 --
 1.7.4.1

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


[PATCH] usb: ehci-s5p: use clk_prepare_enable and clk_disable_unprepare

2012-10-02 Thread Thomas Abraham
Convert clk_enable/clk_disable to clk_prepare_enable/clk_disable_unprepare
calls as required by common clock framework.

Signed-off-by: Thomas Abraham thomas.abra...@linaro.org
---
 drivers/usb/host/ehci-s5p.c |   10 +-
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/usb/host/ehci-s5p.c b/drivers/usb/host/ehci-s5p.c
index 85b74be..abc178d 100644
--- a/drivers/usb/host/ehci-s5p.c
+++ b/drivers/usb/host/ehci-s5p.c
@@ -136,7 +136,7 @@ static int __devinit s5p_ehci_probe(struct platform_device 
*pdev)
goto fail_clk;
}
 
-   err = clk_enable(s5p_ehci-clk);
+   err = clk_prepare_enable(s5p_ehci-clk);
if (err)
goto fail_clk;
 
@@ -183,7 +183,7 @@ static int __devinit s5p_ehci_probe(struct platform_device 
*pdev)
return 0;
 
 fail_io:
-   clk_disable(s5p_ehci-clk);
+   clk_disable_unprepare(s5p_ehci-clk);
 fail_clk:
usb_put_hcd(hcd);
return err;
@@ -200,7 +200,7 @@ static int __devexit s5p_ehci_remove(struct platform_device 
*pdev)
if (pdata  pdata-phy_exit)
pdata-phy_exit(pdev, S5P_USB_PHY_HOST);
 
-   clk_disable(s5p_ehci-clk);
+   clk_disable_unprepare(s5p_ehci-clk);
 
usb_put_hcd(hcd);
 
@@ -231,7 +231,7 @@ static int s5p_ehci_suspend(struct device *dev)
if (pdata  pdata-phy_exit)
pdata-phy_exit(pdev, S5P_USB_PHY_HOST);
 
-   clk_disable(s5p_ehci-clk);
+   clk_disable_unprepare(s5p_ehci-clk);
 
return rc;
 }
@@ -243,7 +243,7 @@ static int s5p_ehci_resume(struct device *dev)
struct platform_device *pdev = to_platform_device(dev);
struct s5p_ehci_platdata *pdata = pdev-dev.platform_data;
 
-   clk_enable(s5p_ehci-clk);
+   clk_prepare_enable(s5p_ehci-clk);
 
if (pdata  pdata-phy_init)
pdata-phy_init(pdev, S5P_USB_PHY_HOST);
-- 
1.7.4.1

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