Changeset 708d8fcd in mainline for uspace/drv/bus/usb/xhci/bus.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/bus.c (modified) (1 diff)

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

@@ -58 +58 @@
 static endpoint_t *endpoint_create(device_t *, const usb_endpoint_descriptors_t *);
 
-/** Ops receive generic bus_t pointer. */
-static inline xhci_bus_t *bus_to_xhci_bus(bus_t *bus_base)
-{
-        assert(bus_base);
-        return (xhci_bus_t *) bus_base;
-}
-
 /**
  * Assign address and control endpoint to a new XHCI device. Once this function