Re: [PATCH 1/1] usb: phy: phy-generic: add the implementation of .set_suspend

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux