On Fri, Jun 01, 2012 at 11:50:44AM -0400, Steven Rostedt wrote: > > > > @@ -1743,6 +1743,11 @@ int pci_prepare_to_sleep(struct pci_dev > > > > if (target_state == PCI_POWER_ERROR) > > > > return -EIO; > > > > > > > > + /* Some devices mustn't be in D3 during system sleep */ > > > > + if (target_state == PCI_D3hot && > > > > + (dev->dev_flags & PCI_DEV_FLAGS_NO_D3_DURING_SLEEP)) > > > > + return 0; > > > > + > > > > > > Why do you want to skip the wakeup setting in that case? > > > > Because that's what the 151b61284776 commit did. Also, the quirk marks > > the controller as not wakeup-capable. > > > > Still, it's worth testing. Andrey and Steve, here's an updated patch > > which should leave wakeup enabled on your EHCI controllers. If you > > don't have a USB keyboard handy for generating a wakeup signal, you > > can test the wakeup functionality by doing: > > > > echo enabled >/sys/bus/usb/devices/usb1/power/wakeup > > echo enabled >/sys/bus/usb/devices/usb2/power/wakeup > > > > before suspending. (In fact you need only one of those two lines, but > > at the moment I forget which -- probably the usb2 one.) Then while > > the system is asleep, either plugging or unplugging a USB device > > directly into the computer should cause it to wake up. > > I applied the patch, it suspends and resumes fine, but still no wakeup. > > I even plugged in a USB keyboard, suspended, and tried to wake it up > with that. That did not work. I even enabled what you stated above, with > no effect (with keyboard or usb storage). The same here, and the keyboard LEDs turn off on suspending. -- WBR, wRAR
Attachment:
signature.asc
Description: Digital signature
