Messages in this thread | | | From | "Rafael J. Wysocki" <> | Subject | Re: [Regression] USB wakeup problem on Toshiba Portege R500 | Date | Tue, 13 Jan 2009 12:13:25 +0100 |
| |
On Tuesday 13 January 2009, Alan Stern wrote: > On Mon, 12 Jan 2009, Rafael J. Wysocki wrote: > > > On Monday 12 January 2009, Alan Stern wrote: > > > On Fri, 9 Jan 2009, Rafael J. Wysocki wrote: > > > > > > > > My proposal: > > > > > > > > > > Devices and events that are clearly associated with system > > > > > wakeup should be enabled by default. For example: Power > > > > > button and laptop lid. > > > > > > > > Agreed. > > > > > > > > > All other devices capable of waking up the system should be > > > > > disabled by default. This presumably includes every PCI > > > > > device. If users want keyboard or mouse events to cause > > > > > a system resume then they will have to configure their > > > > > desktop management program to enable it. > > > > > > > > I generally agree, with one exception. There are network adapters which > > > > can be enabled to wake up by the BIOS and their drivers set them up for WoL > > > > currently on this basis. These should remain enabled IMO. > > > > > > How do we know which adapters these are? IMO the PCI core should > > > disable wakeup by default for all devices when they are detected. > > > > This is what we do. > > > > > Is it sufficient to have the adapter drivers enable wakeup during > > > their probe routines? > > > > Yes, it is. > > Therefore all we need is a patch to change the USB host controller > drivers. They should call device_set_wakeup_capable() instead of > device_init_wakeup(). > > I just got back from vacation so things will be busy for a few days, > but I'll send out such a patch soon.
Great, thanks a lot!
Rafael
| |