Hi, On 03/11/16 10:53, Felipe Balbi wrote: > > Hi, > > 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?
cheers, -roger -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
