From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [Regression] USB wakeup problem on Toshiba Portege R500 Date: Tue, 13 Jan 2009 12:13:25 +0100 Message-ID: <200901131213.26836.rjw@sisk.pl> References: Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Return-path: Received: from ogre.sisk.pl ([217.79.144.158]:49320 "EHLO ogre.sisk.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752162AbZAMLOP (ORCPT ); Tue, 13 Jan 2009 06:14:15 -0500 In-Reply-To: Content-Disposition: inline Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Alan Stern Cc: Frans Pop , oliver@neukum.org, gregkh@suse.de, akpm@linux-foundation.org, linux-kernel@vger.kernel.org, linux-pm@lists.linux-foundation.org, pavel@suse.cz, torvalds@linux-foundation.org, ACPI Devel Maling List , Len Brown 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