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

Timestamp:

2017-10-16T12:03:56Z (7 years ago)

Author:

Michal Staruch <salmelu@…>

Branches:

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

Children:

bb784ae

Parents:

3afcf68

Message:

Removed hardcoded target for doorbell

File:

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

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

@@ -167 +167 @@
                 return EINVAL;
         }
-        if (batch->ep->target.endpoint != 0 || batch->ep->transfer_type != USB_TRANSFER_CONTROL) {
+        if (batch->ep->transfer_type != USB_TRANSFER_CONTROL) {
                 /* This method only works for control transfers. */
-                usb_log_error("Attempted to schedule control transfer to non 0 endpoint.");
+                usb_log_error("Attempted to schedule a control transfer to non control endpoint.");
                 return EINVAL;
         }
@@ -176 +176 @@
 
         uint8_t slot_id = xhci_ep->device->slot_id;
-        xhci_trb_ring_t* ring = hc->dcbaa_virt[slot_id].trs[0];
+        xhci_trb_ring_t* ring = hc->dcbaa_virt[slot_id].trs[batch->ep->target.endpoint];
 
         usb_device_request_setup_packet_t* setup =
@@ -254 +254 @@
         /* For control transfers, the target is always 1. */
         // FIXME: ignoring return code
-        hc_ring_doorbell(hc, slot_id, 1);
+        const uint8_t target = xhci_endpoint_ctx_offset(xhci_ep);
+        hc_ring_doorbell(hc, slot_id, target);
 
         // Issue a Configure Endpoint command, if needed.
@@ -300 +301 @@
 
         // TODO: target = endpoint | stream_id << 16
-        hc_ring_doorbell(hc, slot_id, xhci_ep->base.target.endpoint);
+        const uint8_t target = xhci_endpoint_ctx_offset(xhci_ep);
+        hc_ring_doorbell(hc, slot_id, target);
         return EOK;
 }
@@ -337 +339 @@
         list_append(&transfer->link, &hc->transfers);
 
-        const uint8_t target = 2 * batch->ep->target.endpoint
-                + (batch->ep->direction == USB_DIRECTION_IN ? 1 : 0);
+        const uint8_t target = xhci_endpoint_ctx_offset(xhci_ep);
         usb_log_debug("Ringing doorbell for slot_id = %d, target = %d", slot_id, target);
         return hc_ring_doorbell(hc, slot_id, target);