Changeset 17c5e62 in mainline for uspace/drv/bus/usb/xhci/transfers.c

Timestamp:

2018-01-08T22:30:12Z (7 years ago)

Author:

Ondřej Hlavatý <aearsis@…>

Branches:

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

Children:

0eadfd1e

Parents:

bdd8842c

Message:

xhci: make isoch part of endpoint optional

File:

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

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

@@ -237 +237 @@
 
 static xhci_isoch_transfer_t* isoch_transfer_get_enqueue(xhci_endpoint_t *ep) {
-        if (((ep->isoch_enqueue + 1) % XHCI_ISOCH_BUFFER_COUNT) == ep->isoch_dequeue) {
+        if (((ep->isoch->enqueue + 1) % XHCI_ISOCH_BUFFER_COUNT) == ep->isoch->dequeue) {
                 /* None ready */
                 return NULL;
         }
-        xhci_isoch_transfer_t *isoch_transfer = &ep->isoch_transfers[ep->isoch_enqueue];
-        ep->isoch_enqueue = (ep->isoch_enqueue + 1) % XHCI_ISOCH_BUFFER_COUNT;
+        xhci_isoch_transfer_t *isoch_transfer = &ep->isoch->transfers[ep->isoch->enqueue];
+        ep->isoch->enqueue = (ep->isoch->enqueue + 1) % XHCI_ISOCH_BUFFER_COUNT;
         return isoch_transfer;
 }
 
 static xhci_isoch_transfer_t* isoch_transfer_get_dequeue(xhci_endpoint_t *ep) {
-        xhci_isoch_transfer_t *isoch_transfer = &ep->isoch_transfers[ep->isoch_dequeue];
-        ep->isoch_dequeue = (ep->isoch_dequeue + 1) % XHCI_ISOCH_BUFFER_COUNT;
+        xhci_isoch_transfer_t *isoch_transfer = &ep->isoch->transfers[ep->isoch->dequeue];
+        ep->isoch->dequeue = (ep->isoch->dequeue + 1) % XHCI_ISOCH_BUFFER_COUNT;
         return isoch_transfer;
 }
@@ -283 +283 @@
         xhci_trb_clean(&trb);
 
-        fibril_mutex_lock(&xhci_ep->isoch_guard);
+        fibril_mutex_lock(&xhci_ep->isoch->guard);
         xhci_isoch_transfer_t *isoch_transfer = isoch_transfer_get_enqueue(xhci_ep);
         while (!isoch_transfer) {
-                fibril_condvar_wait(&xhci_ep->isoch_avail, &xhci_ep->isoch_guard);
+                fibril_condvar_wait(&xhci_ep->isoch->avail, &xhci_ep->isoch->guard);
                 isoch_transfer = isoch_transfer_get_enqueue(xhci_ep);
         }
@@ -301 +301 @@
                 &isoch_transfer->interrupt_trb_phys);
         if (err) {
-                fibril_mutex_unlock(&xhci_ep->isoch_guard);
+                fibril_mutex_unlock(&xhci_ep->isoch->guard);
                 return err;
         }
@@ -307 +307 @@
         /* If not yet started, start the isochronous endpoint transfers - after buffer count - 1 writes */
         /* The -1 is there because of the enqueue != dequeue check. The buffer must have at least 2 transfers. */
-        if (((xhci_ep->isoch_enqueue + 1) % XHCI_ISOCH_BUFFER_COUNT) == xhci_ep->isoch_dequeue && !xhci_ep->isoch_started) {
+        if (((xhci_ep->isoch->enqueue + 1) % XHCI_ISOCH_BUFFER_COUNT) == xhci_ep->isoch->dequeue && !xhci_ep->isoch->started) {
                 const uint8_t slot_id = xhci_dev->slot_id;
                 const uint8_t target = xhci_endpoint_index(xhci_ep) + 1; /* EP Doorbells start at 1 */
                 err = hc_ring_doorbell(hc, slot_id, target);
-                xhci_ep->isoch_started = true;
-        }
-        fibril_mutex_unlock(&xhci_ep->isoch_guard);
+                xhci_ep->isoch->started = true;
+        }
+        fibril_mutex_unlock(&xhci_ep->isoch->guard);
         if (err) {
                 return err;
@@ -331 +331 @@
                 xhci_trb_clean(&trb);
                 trb.parameter = isoch_transfer->data.phys;
-                isoch_transfer->size = xhci_ep->isoch_max_size;
+                isoch_transfer->size = xhci_ep->isoch->max_size;
 
                 int err = schedule_isochronous_trb(ring, xhci_ep, &trb, isoch_transfer->size,
@@ -344 +344 @@
         xhci_device_t *xhci_dev)
 {
-        fibril_mutex_lock(&xhci_ep->isoch_guard);
+        fibril_mutex_lock(&xhci_ep->isoch->guard);
         /* If not yet started, start the isochronous endpoint transfers - before first read */
-        if (!xhci_ep->isoch_started) {
+        if (!xhci_ep->isoch->started) {
                 xhci_trb_ring_t *ring = get_ring(hc, transfer);
                 /* Fill the TRB ring. */
                 int err = schedule_isochronous_in_trbs(xhci_ep, ring);
                 if (err) {
-                        fibril_mutex_unlock(&xhci_ep->isoch_guard);
+                        fibril_mutex_unlock(&xhci_ep->isoch->guard);
                         return err;
                 }
@@ -359 +359 @@
                 err = hc_ring_doorbell(hc, slot_id, target);
                 if (err) {
-                        fibril_mutex_unlock(&xhci_ep->isoch_guard);
+                        fibril_mutex_unlock(&xhci_ep->isoch->guard);
                         return err;
                 }
-                xhci_ep->isoch_started = true;
+                xhci_ep->isoch->started = true;
         }
 
         xhci_isoch_transfer_t *isoch_transfer = isoch_transfer_get_enqueue(xhci_ep);
         while(!isoch_transfer) {
-                fibril_condvar_wait(&xhci_ep->isoch_avail, &xhci_ep->isoch_guard);
+                fibril_condvar_wait(&xhci_ep->isoch->avail, &xhci_ep->isoch->guard);
                 isoch_transfer = isoch_transfer_get_enqueue(xhci_ep);
         }
@@ -391 +391 @@
 
         trb.parameter = isoch_transfer->data.phys;
-        isoch_transfer->size = xhci_ep->isoch_max_size;
+        isoch_transfer->size = xhci_ep->isoch->max_size;
 
         xhci_trb_ring_t *ring = get_ring(hc, transfer);
         int err = schedule_isochronous_trb(ring, xhci_ep, &trb, isoch_transfer->size,
                 &isoch_transfer->interrupt_trb_phys);
-        fibril_mutex_unlock(&xhci_ep->isoch_guard);
+        fibril_mutex_unlock(&xhci_ep->isoch->guard);
 
         if (err) {
@@ -411 +411 @@
         xhci_device_t *xhci_dev)
 {
-        if (transfer->batch.buffer_size > xhci_ep->isoch_max_size) {
+        if (transfer->batch.buffer_size > xhci_ep->isoch->max_size) {
                 usb_log_error("Cannot schedule an oversized isochronous transfer.");
                 return EINVAL;
@@ -425 +425 @@
 
 static int handle_isochronous_transfer_event(xhci_hc_t *hc, xhci_trb_t *trb, xhci_endpoint_t *ep) {
-        fibril_mutex_lock(&ep->isoch_guard);
+        fibril_mutex_lock(&ep->isoch->guard);
 
         int err = EOK;
@@ -434 +434 @@
                 case XHCI_TRBC_RING_UNDERRUN:
                         /* Rings are unscheduled by xHC now */
-                        ep->isoch_started = false;
+                        ep->isoch->started = false;
                         /* For OUT, there was nothing to process */
                         /* For IN, the buffer has overfilled, we empty the buffers and readd TRBs */
-                        ep->isoch_enqueue = ep->isoch_dequeue = 0;
+                        ep->isoch->enqueue = ep->isoch->dequeue = 0;
                         err = EIO;
                         break;
@@ -462 +462 @@
         }
 
-        fibril_condvar_signal(&ep->isoch_avail);
-        fibril_mutex_unlock(&ep->isoch_guard);
+        fibril_condvar_signal(&ep->isoch->avail);
+        fibril_mutex_unlock(&ep->isoch->guard);
         return err;
 }