On Wed, Jul 24, 2013 at 11:55:57AM -0400, Alan Stern wrote: > On Tue, 23 Jul 2013, Sarah Sharp wrote: > > > From: Vikas Sajjan <vikas.sajjan@xxxxxxxxxx> > > > > Adds power management support to xHCI platform driver. > > > > This patch facilitates the transition of xHCI host controller > > between S0 and S3/S4 power states, during suspend/resume cycles. > > > > Signed-off-by: Abhilash Kesavan <a.kesavan@xxxxxxxxxxx> > > Signed-off-by: Vikas C Sajjan <vikas.sajjan@xxxxxxxxxx> > > CC: Doug Anderson <dianders@xxxxxxxxxxxx> > > Signed-off-by: Felipe Balbi <balbi@xxxxxx> > > Signed-off-by: Sarah Sharp <sarah.a.sharp@xxxxxxxxxxxxxxx> > > --- > > drivers/usb/host/xhci-plat.c | 26 ++++++++++++++++++++++++++ > > 1 files changed, 26 insertions(+), 0 deletions(-) > > > > diff --git a/drivers/usb/host/xhci-plat.c b/drivers/usb/host/xhci-plat.c > > index 51e22bf..412fe8d 100644 > > --- a/drivers/usb/host/xhci-plat.c > > +++ b/drivers/usb/host/xhci-plat.c > > @@ -186,11 +186,37 @@ static int xhci_plat_remove(struct platform_device *dev) > > return 0; > > } > > > > +#ifdef CONFIG_PM > > +static int xhci_plat_suspend(struct device *dev) > > +{ > > + struct usb_hcd *hcd = dev_get_drvdata(dev); > > + struct xhci_hcd *xhci = hcd_to_xhci(hcd); > > + > > + return xhci_suspend(xhci); > > +} > > Where does the wakeup setting get taken into account? Which wakeup setting are you talking about? Do you mean making sure the wake on bits are set for the roothub ports when the bus is suspended? Or do you mean that the platform device needs to have some way to enable wake from S3/S4 for the xHCI host controller itself? Sarah Sharp -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html