Merge tag 'iommu-updates-v3.16' of git://git.kernel.org/pub/scm/linux/kernel/git...
[firefly-linux-kernel-4.4.55.git] / drivers / usb / core / port.c
index aea54e8dfe470dd7dad1cdece357629788f6d7ce..62036faf56c00beafc636acbb6509976b7c50f31 100644 (file)
@@ -76,21 +76,29 @@ static int usb_port_runtime_resume(struct device *dev)
        struct usb_device *hdev = to_usb_device(dev->parent->parent);
        struct usb_interface *intf = to_usb_interface(dev->parent);
        struct usb_hub *hub = usb_hub_to_struct_hub(hdev);
+       struct usb_device *udev = port_dev->child;
+       struct usb_port *peer = port_dev->peer;
        int port1 = port_dev->portnum;
        int retval;
 
        if (!hub)
                return -EINVAL;
        if (hub->in_reset) {
-               port_dev->power_is_on = 1;
+               set_bit(port1, hub->power_bits);
                return 0;
        }
 
-       usb_autopm_get_interface(intf);
-       set_bit(port1, hub->busy_bits);
+       /*
+        * Power on our usb3 peer before this usb2 port to prevent a usb3
+        * device from degrading to its usb2 connection
+        */
+       if (!port_dev->is_superspeed && peer)
+               pm_runtime_get_sync(&peer->dev);
 
+       usb_autopm_get_interface(intf);
        retval = usb_hub_set_port_power(hdev, hub, port1, true);
-       if (port_dev->child && !retval) {
+       msleep(hub_power_on_good_delay(hub));
+       if (udev && !retval) {
                /*
                 * Attempt to wait for usb hub port to be reconnected in order
                 * to make the resume procedure successful.  The device may have
@@ -101,12 +109,17 @@ static int usb_port_runtime_resume(struct device *dev)
                if (retval < 0)
                        dev_dbg(&port_dev->dev, "can't get reconnection after setting port  power on, status %d\n",
                                        retval);
-               usb_clear_port_feature(hdev, port1, USB_PORT_FEAT_C_ENABLE);
                retval = 0;
+
+               /* Force the child awake to revalidate after the power loss. */
+               if (!test_and_set_bit(port1, hub->child_usage_bits)) {
+                       pm_runtime_get_noresume(&port_dev->dev);
+                       pm_request_resume(&udev->dev);
+               }
        }
 
-       clear_bit(port1, hub->busy_bits);
        usb_autopm_put_interface(intf);
+
        return retval;
 }
 
@@ -116,6 +129,7 @@ static int usb_port_runtime_suspend(struct device *dev)
        struct usb_device *hdev = to_usb_device(dev->parent->parent);
        struct usb_interface *intf = to_usb_interface(dev->parent);
        struct usb_hub *hub = usb_hub_to_struct_hub(hdev);
+       struct usb_port *peer = port_dev->peer;
        int port1 = port_dev->portnum;
        int retval;
 
@@ -129,12 +143,20 @@ static int usb_port_runtime_suspend(struct device *dev)
                return -EAGAIN;
 
        usb_autopm_get_interface(intf);
-       set_bit(port1, hub->busy_bits);
        retval = usb_hub_set_port_power(hdev, hub, port1, false);
        usb_clear_port_feature(hdev, port1, USB_PORT_FEAT_C_CONNECTION);
-       usb_clear_port_feature(hdev, port1,     USB_PORT_FEAT_C_ENABLE);
-       clear_bit(port1, hub->busy_bits);
+       if (!port_dev->is_superspeed)
+               usb_clear_port_feature(hdev, port1, USB_PORT_FEAT_C_ENABLE);
        usb_autopm_put_interface(intf);
+
+       /*
+        * Our peer usb3 port may now be able to suspend, so
+        * asynchronously queue a suspend request to observe that this
+        * usb2 port is now off.
+        */
+       if (!port_dev->is_superspeed && peer)
+               pm_runtime_put(&peer->dev);
+
        return retval;
 }
 #endif
@@ -157,10 +179,13 @@ static struct device_driver usb_port_driver = {
        .owner = THIS_MODULE,
 };
 
-static void link_peers(struct usb_port *left, struct usb_port *right)
+static int link_peers(struct usb_port *left, struct usb_port *right)
 {
+       struct usb_port *ss_port, *hs_port;
+       int rc;
+
        if (left->peer == right && right->peer == left)
-               return;
+               return 0;
 
        if (left->peer || right->peer) {
                struct usb_port *lpeer = left->peer;
@@ -170,21 +195,98 @@ static void link_peers(struct usb_port *left, struct usb_port *right)
                        dev_name(&left->dev), dev_name(&right->dev),
                        dev_name(&left->dev), lpeer,
                        dev_name(&right->dev), rpeer);
-               return;
+               return -EBUSY;
        }
 
+       rc = sysfs_create_link(&left->dev.kobj, &right->dev.kobj, "peer");
+       if (rc)
+               return rc;
+       rc = sysfs_create_link(&right->dev.kobj, &left->dev.kobj, "peer");
+       if (rc) {
+               sysfs_remove_link(&left->dev.kobj, "peer");
+               return rc;
+       }
+
+       /*
+        * We need to wake the HiSpeed port to make sure we don't race
+        * setting ->peer with usb_port_runtime_suspend().  Otherwise we
+        * may miss a suspend event for the SuperSpeed port.
+        */
+       if (left->is_superspeed) {
+               ss_port = left;
+               WARN_ON(right->is_superspeed);
+               hs_port = right;
+       } else {
+               ss_port = right;
+               WARN_ON(!right->is_superspeed);
+               hs_port = left;
+       }
+       pm_runtime_get_sync(&hs_port->dev);
+
        left->peer = right;
        right->peer = left;
+
+       /*
+        * The SuperSpeed reference is dropped when the HiSpeed port in
+        * this relationship suspends, i.e. when it is safe to allow a
+        * SuperSpeed connection to drop since there is no risk of a
+        * device degrading to its powered-off HiSpeed connection.
+        *
+        * Also, drop the HiSpeed ref taken above.
+        */
+       pm_runtime_get_sync(&ss_port->dev);
+       pm_runtime_put(&hs_port->dev);
+
+       return 0;
+}
+
+static void link_peers_report(struct usb_port *left, struct usb_port *right)
+{
+       int rc;
+
+       rc = link_peers(left, right);
+       if (rc == 0) {
+               dev_dbg(&left->dev, "peered to %s\n", dev_name(&right->dev));
+       } else {
+               dev_warn(&left->dev, "failed to peer to %s (%d)\n",
+                               dev_name(&right->dev), rc);
+               pr_warn_once("usb: port power management may be unreliable\n");
+       }
 }
 
 static void unlink_peers(struct usb_port *left, struct usb_port *right)
 {
+       struct usb_port *ss_port, *hs_port;
+
        WARN(right->peer != left || left->peer != right,
                        "%s and %s are not peers?\n",
                        dev_name(&left->dev), dev_name(&right->dev));
 
+       /*
+        * We wake the HiSpeed port to make sure we don't race its
+        * usb_port_runtime_resume() event which takes a SuperSpeed ref
+        * when ->peer is !NULL.
+        */
+       if (left->is_superspeed) {
+               ss_port = left;
+               hs_port = right;
+       } else {
+               ss_port = right;
+               hs_port = left;
+       }
+
+       pm_runtime_get_sync(&hs_port->dev);
+
+       sysfs_remove_link(&left->dev.kobj, "peer");
        right->peer = NULL;
+       sysfs_remove_link(&right->dev.kobj, "peer");
        left->peer = NULL;
+
+       /* Drop the SuperSpeed ref held on behalf of the active HiSpeed port */
+       pm_runtime_put(&ss_port->dev);
+
+       /* Drop the ref taken above */
+       pm_runtime_put(&hs_port->dev);
 }
 
 /*
@@ -212,7 +314,7 @@ static int match_location(struct usb_device *peer_hdev, void *p)
        for (port1 = 1; port1 <= peer_hdev->maxchild; port1++) {
                peer = peer_hub->ports[port1 - 1];
                if (peer && peer->location == port_dev->location) {
-                       link_peers(port_dev, peer);
+                       link_peers_report(port_dev, peer);
                        return 1; /* done */
                }
        }
@@ -275,7 +377,7 @@ static void find_and_link_peer(struct usb_hub *hub, int port1)
         */
        peer = peer_hub->ports[port1 - 1];
        if (peer && peer->location == 0)
-               link_peers(port_dev, peer);
+               link_peers_report(port_dev, peer);
 }
 
 int usb_hub_create_port_device(struct usb_hub *hub, int port1)
@@ -291,13 +393,16 @@ int usb_hub_create_port_device(struct usb_hub *hub, int port1)
 
        hub->ports[port1 - 1] = port_dev;
        port_dev->portnum = port1;
-       port_dev->power_is_on = true;
+       set_bit(port1, hub->power_bits);
        port_dev->dev.parent = hub->intfdev;
        port_dev->dev.groups = port_dev_group;
        port_dev->dev.type = &usb_port_device_type;
        port_dev->dev.driver = &usb_port_driver;
+       if (hub_is_superspeed(hub->hdev))
+               port_dev->is_superspeed = 1;
        dev_set_name(&port_dev->dev, "%s-port%d", dev_name(&hub->hdev->dev),
                        port1);
+       mutex_init(&port_dev->status_lock);
        retval = device_register(&port_dev->dev);
        if (retval)
                goto error_register;