Changeset 6832245 in mainline for uspace/drv/bus/usb/ohci/ohci_bus.c

Timestamp:

2017-12-14T23:01:57Z (7 years ago)

Author:

Ondřej Hlavatý <aearsis@…>

Branches:

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

Children:

837d53d

Parents:

bd05140

git-author:

Ondřej Hlavatý <aearsis@…> (2017-12-14 23:01:54)

git-committer:

Ondřej Hlavatý <aearsis@…> (2017-12-14 23:01:57)

Message:

usbhost bus: refactor the bus ops

This way, method names better represent the entity it is working with.
Their semantics was shifted a bit.

Regarding the tree of structures:

bus ← device ← endpoint ← batch

Previously, devices were kept in DDF function nodes, and endpoints had
pointer to the bus and device. Now, devices have pointer to bus,
endpoints don't.

Pointer to hcd_t in bus is WIP, and will be removed.

File:

• uspace/drv/bus/usb/ohci/ohci_bus.c (modified) (6 diffs)

uspace/drv/bus/usb/ohci/ohci_bus.c

@@ -72 +72 @@
 /** Creates new hcd endpoint representation.
  */
-static endpoint_t *ohci_endpoint_create(bus_t *bus)
+static endpoint_t *ohci_endpoint_create(device_t *dev, const usb_endpoint_desc_t *desc)
 {
-        assert(bus);
+        assert(dev);
 
         ohci_endpoint_t *ohci_ep = malloc(sizeof(ohci_endpoint_t));
@@ -80 +80 @@
                 return NULL;
 
-        endpoint_init(&ohci_ep->base, bus);
+        endpoint_init(&ohci_ep->base, dev, desc);
 
         ohci_ep->ed = malloc32(sizeof(ed_t));
@@ -115 +115 @@
 
 
-static int ohci_register_ep(bus_t *bus_base, device_t *dev, endpoint_t *ep, const usb_endpoint_desc_t *desc)
+static int ohci_register_ep(endpoint_t *ep)
 {
+        bus_t *bus_base = endpoint_get_bus(ep);
         ohci_bus_t *bus = (ohci_bus_t *) bus_base;
         ohci_endpoint_t *ohci_ep = ohci_endpoint_get(ep);
 
-        const int err = bus->parent_ops.register_endpoint(bus_base, dev, ep, desc);
+        const int err = usb2_bus_ops.endpoint_register(ep);
         if (err)
                 return err;
@@ -130 +131 @@
 }
 
-static int ohci_unregister_ep(bus_t *bus_base, endpoint_t *ep)
+static int ohci_unregister_ep(endpoint_t *ep)
 {
-        ohci_bus_t *bus = (ohci_bus_t *) bus_base;
-        assert(bus);
+        ohci_bus_t *bus = (ohci_bus_t *) endpoint_get_bus(ep);
         assert(ep);
 
-        const int err = bus->parent_ops.unregister_endpoint(bus_base, ep);
+        const int err = usb2_bus_ops.endpoint_unregister(ep);
         if (err)
                 return err;
@@ -144 +144 @@
 }
 
-static usb_transfer_batch_t *ohci_bus_create_batch(bus_t *bus, endpoint_t *ep)
+static usb_transfer_batch_t *ohci_create_batch(endpoint_t *ep)
 {
         ohci_transfer_batch_t *batch = ohci_transfer_batch_create(ep);
@@ -150 +150 @@
 }
 
-static void ohci_bus_destroy_batch(usb_transfer_batch_t *batch)
+static void ohci_destroy_batch(usb_transfer_batch_t *batch)
 {
         ohci_transfer_batch_destroy(ohci_transfer_batch_get(batch));
 }
 
-int ohci_bus_init(ohci_bus_t *bus, hc_t *hc)
+static const bus_ops_t ohci_bus_ops = {
+        .parent = &usb2_bus_ops,
+
+        .endpoint_destroy = ohci_endpoint_destroy,
+        .endpoint_create = ohci_endpoint_create,
+        .endpoint_register = ohci_register_ep,
+        .endpoint_unregister = ohci_unregister_ep,
+        .endpoint_count_bw = bandwidth_count_usb11,
+        .endpoint_set_toggle = ohci_ep_toggle_set,
+        .endpoint_get_toggle = ohci_ep_toggle_get,
+        .batch_create = ohci_create_batch,
+        .batch_destroy = ohci_destroy_batch,
+};
+
+
+int ohci_bus_init(ohci_bus_t *bus, hcd_t *hcd, hc_t *hc)
 {
         assert(hc);
         assert(bus);
 
-        usb2_bus_init(&bus->base, BANDWIDTH_AVAILABLE_USB11, bandwidth_count_usb11);
 
-        bus_ops_t *ops = &bus->base.base.ops;
-        bus->parent_ops = *ops;
-        ops->create_endpoint = ohci_endpoint_create;
-        ops->destroy_endpoint = ohci_endpoint_destroy;
-        ops->endpoint_set_toggle = ohci_ep_toggle_set;
-        ops->endpoint_get_toggle = ohci_ep_toggle_get;
+        usb2_bus_t *usb2_bus = (usb2_bus_t *) bus;
+        bus_t *bus_base = (bus_t *) bus;
 
-        ops->register_endpoint = ohci_register_ep;
-        ops->unregister_endpoint = ohci_unregister_ep;
-
-        ops->create_batch = ohci_bus_create_batch;
-        ops->destroy_batch = ohci_bus_destroy_batch;
+        usb2_bus_init(usb2_bus, hcd, BANDWIDTH_AVAILABLE_USB11);
+        bus_base->ops = &ohci_bus_ops;
 
         bus->hc = hc;