Thread (13 messages) 13 messages, 4 authors, 2014-05-09

[PATCH v12 4/4] usb: ehci-exynos: Change to use phy provided by the generic phy framework

From: Tomasz Figa <hidden>
Date: 2014-05-09 08:19:59
Also in: linux-devicetree, linux-samsung-soc, lkml

Hi Vivek,

On 05.05.2014 07:02, Vivek Gautam wrote:
From: Kamil Debski <redacted>

Add the phy provider, supplied by new Exynos-usb2phy using
Generic phy framework.
Keeping the support for older USB phy intact right now, in order
to prevent any functionality break in absence of relevant
device tree side change for ehci-exynos.
Once we move to new phy in the device nodes for ehci, we can
remove the support for older phys.

Signed-off-by: Kamil Debski <redacted>
[gautam.vivek at samsung.com: Addressed review comments from mailing list]
[gautam.vivek at samsung.com: Kept the code for old usb-phy, and just
added support for new exynos5-usb2phy in generic phy framework]
[gautam.vivek at samsung.com: Edited the commit message]
Signed-off-by: Vivek Gautam <redacted>
Cc: Jingoo Han <redacted>
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Acked-by: Kukjin Kim <redacted>
---

Changes from v11:
 - Removed setting phy explicitly to error pointer.
 - Changed error check to '-ENOSYS' instead of '-ENXIO' in failure case of
   devm_of_phy_get().

Changes from v10:
 - Removed 'phy-names' property from the bindings since we don't need it.
 - Restructured exynos_ehci_get_phy() function to handle error codes as
   well as return relevant error codes effectively.
 - Added IS_ERR() check for PHYs in exynos_ehci_phy_enable()/disable().

Changes from v9:
 - Calling usb_phy_shutdown() when exynos_ehci_phy_enable() is failing.
 - Made exynos_ehci_phy_disable() return void, since its return value
   did not serve any purpose.
 - Calling clk_disable_unprepare() in exynos_ehci_resume() when
   exynos_ehci_phy_enable() is failed.

 .../devicetree/bindings/usb/exynos-usb.txt         |   15 +++
 drivers/usb/host/ehci-exynos.c                     |  129 +++++++++++++++++---
 2 files changed, 124 insertions(+), 20 deletions(-)
Reviewed-by: Tomasz Figa <redacted>

--
Best regards,
Tomasz
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help