Changeset e4d7363 in mainline for uspace/drv/bus/usb/xhci/hc.h

Timestamp:

2017-06-22T21:34:39Z (9 years ago)

Author:

Ondřej Hlavatý <aearsis@…>

Branches:

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

Children:

91ca111

Parents:

cb89430

Message:

usbhost: refactor the initialization

Before that, drivers had to setup MMIO range multiple times, or even parse hw
resources themselves again. The former init method was split in half - init and
start. Init shall allocate and initialize inner structures, start shall start
the HC.

In the XHCI it is demonstrated how to isolate inner HC implementation from the
fact this driver is using libusbhost. It adds some boilerplate code, but
I think it leads to cleaner design.

File:

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

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

@@ -34 +34 @@
  */
 
-#include <usb/host/ddf_helpers.h>
+#include <usb/host/usb_transfer_batch.h>
 #include "hw_struct/regs.h"
 #include "hw_struct/context.h"
@@ -44 +44 @@
         xhci_rt_regs_t *rt_regs;
         xhci_doorbell_t *db_arry;
+
+        addr_range_t mmio_range;
 
         xhci_trb_ring_t command_ring;
@@ -55 +57 @@
 } xhci_hc_t;
 
-extern const ddf_hc_driver_t xhci_ddf_hc_driver;
-
+int hc_init_mmio(xhci_hc_t *, const hw_res_list_parsed_t *);
+int hc_init_memory(xhci_hc_t *);
+int hc_claim(xhci_hc_t *, ddf_dev_t *);
+int hc_irq_code_gen(irq_code_t *, xhci_hc_t *, const hw_res_list_parsed_t *);
+int hc_start(xhci_hc_t *, bool);
+int hc_schedule(xhci_hc_t *hc, usb_transfer_batch_t *batch);
+int hc_status(xhci_hc_t *, uint32_t *);
+void hc_interrupt(xhci_hc_t *, uint32_t);
+void hc_fini(xhci_hc_t *);
 
 /**