Hi, Peter Chen <peter.chen@xxxxxxx> 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 <peter.chen@xxxxxxx> > --- > 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? -- balbi
Attachment:
signature.asc
Description: PGP signature