Changeset 708d8fcd in mainline for uspace/drv/bus/usb/xhci/hc.c

Timestamp:

2018-01-10T00:46:29Z (7 years ago)

Author:

Ondřej Hlavatý <aearsis@…>

Branches:

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

Children:

f92f6b1

Parents:

4793023

git-author:

Ondřej Hlavatý <aearsis@…> (2018-01-09 19:26:55)

git-committer:

Ondřej Hlavatý <aearsis@…> (2018-01-10 00:46:29)

Message:

xhci: rewritten isochronous transfers

There was a fundamental problem with relying on hardware to send
RING_OVERRUN/UNDERRUN events, which QEMU (and possibly others) do not
send. That resulted in not knowing if the transfer is still on schedule,
and having to ring the doorbell every time. That is not feasible,
because then the transfer can be more frequent than it should be.
Furthermore, it ignored the fact that isochronous TRBs are to be
scheduled not too late, but also not too soon (see 4.11.2.5 of the xHCI
spec).

Now, scheduling the TRBs to hardware is called feeding, and can be
delayed by setting a timer. Ring overruns/underruns are detected also at
the end of handling an event.

File:

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

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

@@ -192 +192 @@
         hc->ac64 = XHCI_REG_RD(hc->cap_regs, XHCI_CAP_AC64);
         hc->max_slots = XHCI_REG_RD(hc->cap_regs, XHCI_CAP_MAX_SLOTS);
+        unsigned ist = XHCI_REG_RD(hc->cap_regs, XHCI_CAP_IST);
+        hc->ist = (ist & 0x10 >> 1) * (ist & 0xf);
 
         if ((err = hc_parse_ec(hc))) {
@@ -611 +613 @@
  * Ring a xHC Doorbell. Implements section 4.7.
  */
-int hc_ring_doorbell(xhci_hc_t *hc, unsigned doorbell, unsigned target)
+void hc_ring_doorbell(xhci_hc_t *hc, unsigned doorbell, unsigned target)
 {
         assert(hc);
@@ -617 +619 @@
         pio_write_32(&hc->db_arry[doorbell], v);
         usb_log_debug2("Ringing doorbell %d (target: %d)", doorbell, target);
-        return EOK;
 }