On Fri, Mar 05, 2010 at 04:01:43PM +0800, Libin wrote: > On Wed, 2010-03-03 at 15:34 -0500, Alan Stern wrote: > > On Wed, 3 Mar 2010, Sarah Sharp wrote: > > > > > On Wed, Mar 03, 2010 at 06:07:55PM +0800, Libin wrote: > > > > >From d3055401927f70e4de7717138b4c8ecad8b8223a Mon Sep 17 00:00:00 2001 > > > > From: Libin Yang <libin.yang@xxxxxxx> > > > > Date: Wed, 3 Mar 2010 13:50:41 +0800 > > > > Subject: [PATCH 3/5] xHCI: bus power management implementation > > > > > > > > This patch implements xHCI bus suspend/resume function hook. > > > > > > > > In the patch it goes through all the ports and suspend/resume > > > > the ports if needed. > > > > > > I thought the USB core ensured that all devices were suspended before > > > suspending the root hub. Alan, can you confirm? > > > > It does not ensure this if CONFIG_USB_SUSPEND isn't enabled. In such > > cases it never suspends any ports, but it still calls the HCDs' > > bus_suspend routines if CONFIG_PM is set. > > So does this mean that if CONFIG_USB_SUSPEND isn't enabled, > bus_suspend() can straight skip the port suspend and return? Or > bus_suspend() needs suspend the ports? > > BTW: Is Greg's git address: > > git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb-2.6.git No, that is the tree I use to send Linus patches. Please read the MAINTAINERS file for where to find my tree. thanks, greg k-h -- 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
