On Fri, Dec 13, 2013 at 09:23:37AM +0800, Peter Chen wrote:
> They are used to notify PHY that the controller enters suspend
> or finishes resume.
>
> Signed-off-by: Peter Chen <peter.chen@xxxxxxxxxxxxx>
> ---
> include/linux/usb/phy.h | 23 +++++++++++++++++++++++
> 1 files changed, 23 insertions(+), 0 deletions(-)
>
> diff --git a/include/linux/usb/phy.h b/include/linux/usb/phy.h
> index 6c0b1c5..a747960 100644
> --- a/include/linux/usb/phy.h
> +++ b/include/linux/usb/phy.h
> @@ -116,6 +116,11 @@ struct usb_phy {
> enum usb_device_speed speed);
> int (*notify_disconnect)(struct usb_phy *x,
> enum usb_device_speed speed);
> + int (*notify_suspend)(struct usb_phy *x,
> + enum usb_device_speed speed);
> + int (*notify_resume)(struct usb_phy *x,
> + enum usb_device_speed speed);
> +
I still don't think this is necessary. Why don't you just call
usb_phy_set_suspend() directly ? And why do you need the extra speed
argument ?
--
balbi
Attachment:
signature.asc
Description: Digital signature
