Changeset a501aaba in mainline for uspace/drv/bus/usb/xhci/trb_ring.c

Timestamp:

2017-10-26T22:21:30Z (6 years ago)

Author:

Ondřej Hlavatý <aearsis@…>

Branches:

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

Children:

5bc8250

Parents:

928afc8d

Message:

xhci trb_ring: fix ring wrapping

The Toggle Cycle of the Link TRB must be set, not the Cycle flag.

File:

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

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

@@ -107 +107 @@
         xhci_trb_t *last = segment_end(segment) - 1;
         xhci_trb_link_fill(last, segment->phys);
-        xhci_trb_set_cycle(last, true);
+        TRB_LINK_SET_TC(*last, true);
 
         ring->enqueue_segment = segment;
@@ -219 +219 @@
         trb = first_trb;
         for (size_t i = 0; i < trbs; ++i, ++trb) {
-                xhci_trb_set_cycle(trb, ring->pcs);
+                TRB_SET_CYCLE(*trb, ring->pcs);
                 xhci_trb_copy(ring->enqueue_trb, trb);
 
@@ -226 +226 @@
 
                 if (TRB_TYPE(*ring->enqueue_trb) == XHCI_TRB_TYPE_LINK) {
-                        // XXX: Check, whether the order here is correct (ambiguous instructions in 4.11.5.1)
-                        xhci_trb_set_cycle(ring->enqueue_trb, ring->pcs);
+                        TRB_SET_CYCLE(*ring->enqueue_trb, ring->pcs);
 
                         if (TRB_LINK_TC(*ring->enqueue_trb)) {