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 <[email protected]> > --- > 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
signature.asc
Description: Digital signature
