USB: Update USB default wakeup settings
authorAlan Stern <stern@rowland.harvard.edu>
Mon, 26 Sep 2011 15:23:38 +0000 (11:23 -0400)
committerGreg Kroah-Hartman <gregkh@suse.de>
Mon, 26 Sep 2011 22:54:11 +0000 (15:54 -0700)
This patch (as1486) implements the kernel's new wakeup policy for USB
host controllers.  Since they don't generate wakeup requests on their
but merely forward requests from their root hubs toward the CPU, they
should be enabled for wakeup by default.

Also, to be compliant with both the old and new policies, root hubs
should not be enabled for remote wakeup by default.  Userspace must
enable it explicitly if it is desired.

Signed-off-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/core/hcd.c

index 73cbbd85219fa5525b2968443d61a5c6f5086abd..100d4b002642e0c6e272c3f465d465701a7aeec5 100644 (file)
@@ -2429,7 +2429,7 @@ int usb_add_hcd(struct usb_hcd *hcd,
         * but drivers can override it in reset() if needed, along with
         * recording the overall controller's system wakeup capability.
         */
-       device_init_wakeup(&rhdev->dev, 1);
+       device_set_wakeup_capable(&rhdev->dev, 1);
 
        /* HCD_FLAG_RH_RUNNING doesn't matter until the root hub is
         * registered.  But since the controller can die at any time,
@@ -2478,6 +2478,13 @@ int usb_add_hcd(struct usb_hcd *hcd,
        }
        if (hcd->uses_new_polling && HCD_POLL_RH(hcd))
                usb_hcd_poll_rh_status(hcd);
+
+       /*
+        * Host controllers don't generate their own wakeup requests;
+        * they only forward requests from the root hub.  Therefore
+        * controllers should always be enabled for remote wakeup.
+        */
+       device_wakeup_enable(hcd->self.controller);
        return retval;
 
 error_create_attr_group: