usb: Avoid unnecessary LPM enabling and disabling during suspend and resume
authorMathias Nyman <mathias.nyman@linux.intel.com>
Thu, 15 Jun 2017 08:55:41 +0000 (11:55 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 15 Jun 2017 20:17:46 +0000 (22:17 +0200)
The original motivation for disabling/enabling Link PM at device
suspend/resume was to force link state to go via U0 before suspend sets
the link state to U3. Going directly from U2 to U3 is not allowed.

Disabling LPM will forced the link state to U0, but will send a lot of
Set port feature requests for evert suspend and resume.

This is not needed as Hub hardware will take care of going via U0
when a U2 -> U3 transition is requested [1]

[1] USB 3.1 specification section 10.16.2.10 Set Port Feature:

"If the value is 3, then host software wants to selectively suspend the
device connected to this port. The hub shall transition the link to U3
from any of the other U states using allowed link state transitions.
If the port is not already in the U0 state, then it shall transition the
port to the U0 state and then initiate the transition to U3.
While this state is active, the hub does not propagate downstream-directed
traffic to this port, but the hub will respond to resume signaling from the
port"

Signed-off-by: Mathias Nyman <mathias.nyman@linux.intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/core/hub.c

index b8bb20d7acdb9f1480009605f10e2f5ae4045ec9..59e0418a5ed564fe470d1e8826e485872fceb312 100644 (file)
@@ -3155,12 +3155,6 @@ int usb_port_suspend(struct usb_device *udev, pm_message_t msg)
                if (PMSG_IS_AUTO(msg))
                        goto err_ltm;
        }
-       if (usb_unlocked_disable_lpm(udev)) {
-               dev_err(&udev->dev, "Failed to disable LPM before suspend\n.");
-               status = -ENOMEM;
-               if (PMSG_IS_AUTO(msg))
-                       goto err_lpm3;
-       }
 
        /* see 7.1.7.6 */
        if (hub_is_superspeed(hub->hdev))
@@ -3187,9 +3181,7 @@ int usb_port_suspend(struct usb_device *udev, pm_message_t msg)
        if (status) {
                dev_dbg(&port_dev->dev, "can't suspend, status %d\n", status);
 
-               /* Try to enable USB3 LPM and LTM again */
-               usb_unlocked_enable_lpm(udev);
- err_lpm3:
+               /* Try to enable USB3 LTM again */
                usb_enable_ltm(udev);
  err_ltm:
                /* Try to enable USB2 hardware LPM again */
@@ -3473,9 +3465,8 @@ int usb_port_resume(struct usb_device *udev, pm_message_t msg)
                if (udev->usb2_hw_lpm_capable == 1)
                        usb_set_usb2_hardware_lpm(udev, 1);
 
-               /* Try to enable USB3 LTM and LPM */
+               /* Try to enable USB3 LTM */
                usb_enable_ltm(udev);
-               usb_unlocked_enable_lpm(udev);
        }
 
        usb_unlock_port(port_dev);