Re: [linux-usb-devel] Re: More about USB PM

2003-08-30 Thread David Brownell
Alan Stern wrote: On Fri, 29 Aug 2003, David Brownell wrote: Alan Stern wrote: usb_hcd_pci_suspend() checks that hcd->state isn't already SUSPENDED or HALT. If not, it calls driver->suspend() and then sets hcd->state to SUSPENDED. That does sort of "know" that the only non-running stat

[linux-usb-devel] Re: More about USB PM

2003-08-30 Thread Alan Stern
On Fri, 29 Aug 2003, David Brownell wrote: > Alan Stern wrote: > > > usb_hcd_pci_suspend() checks that hcd->state isn't already > > SUSPENDED or HALT. If not, it calls driver->suspend() and then sets > > hcd->state to SUSPENDED. > > That does sort of "know" that the only non-running states

[linux-usb-devel] Re: More about USB PM

2003-08-30 Thread David Brownell
Alan Stern wrote: David: Ownership of hcd->state is unclear. In hcd-pci.c, I see the following: See also the patch I just posted, which includes part of what BenH sent by recently along with changes based on your comments. That state machine does still need some cleanup. usb_hcd_pci_pro