Hi, Roger Quadros <[email protected]> writes: >> Peter Chen <[email protected]> writes: >>> Add clock operation at .set_suspend if the PHY has >>> suspend requirement, it can be benefit of power saving for >>> phy and the whole system (parent clock may also be disabled). >>> >>> Signed-off-by: Peter Chen <[email protected]> >>> --- >>> drivers/usb/phy/phy-generic.c | 9 +++++++++ >>> 1 file changed, 9 insertions(+) >>> >>> diff --git a/drivers/usb/phy/phy-generic.c b/drivers/usb/phy/phy-generic.c >>> index 8311ba2..89d6e7a 100644 >>> --- a/drivers/usb/phy/phy-generic.c >>> +++ b/drivers/usb/phy/phy-generic.c >>> @@ -59,6 +59,15 @@ EXPORT_SYMBOL_GPL(usb_phy_generic_unregister); >>> >>> static int nop_set_suspend(struct usb_phy *x, int suspend) >>> { >>> + struct usb_phy_generic *nop = dev_get_drvdata(x->dev); >>> + >>> + if (!IS_ERR(nop->clk)) { >>> + if (suspend) >>> + clk_disable_unprepare(nop->clk); >>> + else >>> + clk_prepare_enable(nop->clk); >>> + } >>> + >> >> Bin, Roger, can you make sure this causes no regressions for AM335x >> devices? >> > If I understood right am335x doesn't use nop-phy so this patch shouldn't > impact AM335x. Bin, do you agree?
phy-am335x.c uses phy-generic.c as a library. Look for usb_phy_gen_create_phy(). -- balbi
signature.asc
Description: PGP signature
