Changeset 415c5116 in mainline for uspace/lib/usbhost/src/usb2_bus.c

Timestamp:

2017-12-29T00:23:34Z (6 years ago)

Author:

Petr Manek <petr.manek@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

5dfb70c9

Parents:

deb2e55

Message:

usbhost: first stab at usb2 disconnect (wip)

File:

• uspace/lib/usbhost/src/usb2_bus.c (modified) (1 diff)

uspace/lib/usbhost/src/usb2_bus.c

@@ -367 +367 @@
         assert(bus);
 
-        // FIXME: Implement me!
-
-        return ENOTSUP;
+        int err;
+        /* Tear down all drivers working with the device. */
+        if ((err = ddf_fun_offline(device->fun))) {
+                return err;
+        }
+
+        /* Block creation of new endpoints and transfers. */
+        usb_log_info("Device(%d): Going offline.", device->address);
+        fibril_mutex_lock(&device->guard);
+        device->online = false;
+        fibril_mutex_unlock(&device->guard);
+
+        /* FIXME: This implementation leaves sleeping parts of drivers around.
+         * With XHCI bus, the HID driver disengages and completely deactivates
+         * when the DDF function is offlined. In USB2 bus, the driver receives
+         * dev_remove and disengages "on paper" but later when interrupt message arrives,
+         * some sleeping code is woken up and crashes the driver.
+         *
+         * The XHCI does 2 extra things that might prevent this behavior:
+         *   (1) deconfigure the device,
+         *   (2) deallocate all transfer TRB rings
+         */
+
+        return EOK;
 }