Changeset d369b3b in mainline for uspace/drv/bus/usb/uhci

Timestamp:

2018-01-25T02:05:57Z (8 years ago)

Author:

Ondřej Hlavatý <aearsis@…>

Branches:

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

Children:

296d22fc

Parents:

b357377

git-author:

Ondřej Hlavatý <aearsis@…> (2018-01-25 01:52:13)

git-committer:

Ondřej Hlavatý <aearsis@…> (2018-01-25 02:05:57)

Message:

usb2_bus: no longer be a bus

As the number of implemented functions got to 3, it's not so beneficial
to inherit usb2 bus to get the functionality. Overall, four trampolines
needed to be added, which is an acceptable number.

Now, the usb2_bus has become a usb2_bus_helper, to be used as
a companion to the common bus.

This is mostly a preparation to remove the runtime binding of the bus
methods.

Location:

uspace/drv/bus/usb/uhci

Files:

• hc.c (modified) (4 diffs)
• hc.h (modified) (1 diff)

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

@@ -329 +329 @@
         hc_t * const hc = bus_to_hc(endpoint_get_bus(ep));
 
-        const int err = usb2_bus_ops.endpoint_register(ep);
+        const int err = usb2_bus_endpoint_register(&hc->bus_helper, ep);
         if (err)
                 return err;
@@ -350 +350 @@
 {
         hc_t * const hc = bus_to_hc(endpoint_get_bus(ep));
-        usb2_bus_ops.endpoint_unregister(ep);
+        usb2_bus_endpoint_unregister(&hc->bus_helper, ep);
 
         // Check for the roothub, as it does not schedule into lists
@@ -406 +406 @@
 }
 
+static int device_enumerate(device_t *dev)
+{
+        hc_t * const hc = bus_to_hc(dev->bus);
+        return usb2_bus_device_enumerate(&hc->bus_helper, dev);
+}
+
 static int hc_status(bus_t *, uint32_t *);
 static int hc_schedule(usb_transfer_batch_t *);
 
 static const bus_ops_t uhci_bus_ops = {
-        .parent = &usb2_bus_ops,
-
         .interrupt = hc_interrupt,
         .status = hc_status,
+
+        .device_enumerate = device_enumerate,
 
         .endpoint_create = endpoint_create,
@@ -438 +444 @@
         assert(instance);
 
-        usb2_bus_init(&instance->bus, &bandwidth_accounting_usb11);
-
-        bus_t *bus = (bus_t *) &instance->bus;
-        bus->ops = &uhci_bus_ops;
-
-        hc_device_setup(&instance->base, bus);
+        usb2_bus_helper_init(&instance->bus_helper, &bandwidth_accounting_usb11);
+
+        bus_init(&instance->bus, sizeof(device_t));
+        instance->bus.ops = &uhci_bus_ops;
+
+        hc_device_setup(&instance->base, &instance->bus);
 
         /* Init USB frame list page */

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

@@ -104 +104 @@
 
         uhci_rh_t rh;
-        usb2_bus_t bus;
+        bus_t bus;
+        usb2_bus_helper_t bus_helper;
+
         /** Addresses of I/O registers */
         uhci_regs_t *registers;