Changeset 30fc56f in mainline for uspace/drv/bus/usb/xhci/bus.c

Timestamp:

2018-01-11T09:38:40Z (6 years ago)

Author:

Ondřej Hlavatý <aearsis@…>

Branches:

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

Children:

92caadd

Parents:

53a9d02

git-author:

Ondřej Hlavatý <aearsis@…> (2018-01-11 09:38:32)

git-committer:

Ondřej Hlavatý <aearsis@…> (2018-01-11 09:38:40)

Message:

xhci: stop endpoint on unregister

File:

• uspace/drv/bus/usb/xhci/bus.c (modified) (2 diffs)

uspace/drv/bus/usb/xhci/bus.c

@@ -323 +323 @@
 
 /**
+ * Abort a transfer on an endpoint.
+ */
+static int endpoint_abort(endpoint_t *ep)
+{
+        xhci_bus_t *bus = bus_to_xhci_bus(endpoint_get_bus(ep));
+        xhci_device_t *dev = xhci_device_get(ep->device);
+
+        usb_transfer_batch_t *batch = NULL;
+        fibril_mutex_lock(&ep->guard);
+        if (ep->active_batch) {
+                if (dev->slot_id) {
+                        const int err = hc_stop_endpoint(bus->hc, dev->slot_id, xhci_endpoint_dci(xhci_endpoint_get(ep)));
+                        if (err) {
+                                usb_log_warning("Failed to stop endpoint %u of device " XHCI_DEV_FMT ": %s",
+                                    ep->endpoint, XHCI_DEV_ARGS(*dev), str_error(err));
+                        }
+
+                        endpoint_wait_timeout_locked(ep, 2000);
+                }
+
+                batch = ep->active_batch;
+                if (batch) {
+                        endpoint_deactivate_locked(ep);
+                }
+        }
+        fibril_mutex_unlock(&ep->guard);
+
+        if (batch) {
+                batch->error = EINTR;
+                batch->transfered_size = 0;
+                usb_transfer_batch_finish(batch);
+        }
+        return EOK;
+}
+
+/**
  * Unregister an endpoint. If the device is still available, inform the xHC
  * about it.
@@ -335 +371 @@
         xhci_device_t *dev = xhci_device_get(ep_base->device);
 
+        endpoint_abort(ep_base);
+
         /* If device slot is still available, drop the endpoint. */
         if (dev->slot_id) {
+
                 if ((err = hc_drop_endpoint(bus->hc, dev->slot_id, xhci_endpoint_index(ep)))) {
                         usb_log_error("Failed to drop endpoint " XHCI_EP_FMT ": %s", XHCI_EP_ARGS(*ep), str_error(err));