Changeset 7278cbc9 in mainline for uspace/lib/usbhost

Timestamp:

2018-01-22T23:53:11Z (7 years ago)

Author:

Ondřej Hlavatý <aearsis@…>

Branches:

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

Children:

e546142

Parents:

3ac86a4

Message:

usbhost: dispose the EP0 properly (+some ehci cleanup while debugging)

Location:

uspace/lib/usbhost/src

Files:

• bus.c (modified) (2 diffs)
• usb2_bus.c (modified) (2 diffs)

uspace/lib/usbhost/src/bus.c

@@ -208 +208 @@
 
         const bus_ops_t *ops = BUS_OPS_LOOKUP(dev->bus->ops, device_gone);
+        const bus_ops_t *ep_ops = BUS_OPS_LOOKUP(dev->bus->ops, endpoint_unregister);
         assert(ops);
 
@@ -240 +241 @@
 
         /* Tell the HC to release its resources. */
-        ops->device_gone(dev);
-
-        /* Release the EP0 bus reference */
-        endpoint_del_ref(dev->endpoints[0]);
+        if (ops)
+                ops->device_gone(dev);
+
+        /* Check whether the driver didn't forgot EP0 */
+        if (dev->endpoints[0]) {
+                if (ep_ops)
+                        ep_ops->endpoint_unregister(dev->endpoints[0]);
+                /* Release the EP0 bus reference */
+                endpoint_del_ref(dev->endpoints[0]);
+        }
 
         /* Destroy the function, freeing also the device, unlocking mutex. */

uspace/lib/usbhost/src/usb2_bus.c

@@ -209 +209 @@
 }
 
-static void usb2_bus_device_gone(device_t *dev)
-{
-        // TODO: Implement me!
-}
-
 /**
  * Register an endpoint to the bus. Reserves bandwidth.
@@ -245 +240 @@
 const bus_ops_t usb2_bus_ops = {
         .device_enumerate = usb2_bus_device_enumerate,
-        .device_gone = usb2_bus_device_gone,
         .endpoint_register = usb2_bus_register_ep,
         .endpoint_unregister = usb2_bus_unregister_ep,