Changeset b5f813c in mainline for uspace/lib/usbhost/src

Timestamp:

2015-07-04T03:28:02Z (10 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

55346870

Parents:

2dbfe44

Message:

libusbhost,ehci,ohci,uhci,vhc: Pass ops structure instead of function pointers to intialization

Location:

uspace/lib/usbhost/src

Files:

• ddf_helpers.c (modified) (4 diffs)
• hcd.c (modified) (6 diffs)

uspace/lib/usbhost/src/ddf_helpers.c

@@ -690 +690 @@
 }
 
+//TODO: Move this to generic ddf?
 int hcd_ddf_get_registers(ddf_dev_t *device, hw_res_list_parsed_t *hw_res)
 {
@@ -779 +780 @@
         assert(dev);
         hcd_t *hcd = dev_to_hcd(dev);
-        if (!hcd || !hcd->driver.irq_hook) {
+        if (!hcd || !hcd->ops.irq_hook) {
                 usb_log_error("Interrupt on not yet initialized device.\n");
                 return;
         }
         const uint32_t status = IPC_GET_ARG1(*call);
-        hcd->driver.irq_hook(hcd, status);
+        hcd->ops.irq_hook(hcd, status);
 }
 
@@ -791 +792 @@
         hcd_t *hcd = arg;
         assert(hcd);
-        if (!hcd->driver.status_hook || !hcd->driver.irq_hook)
+        if (!hcd->ops.status_hook || !hcd->ops.irq_hook)
                 return ENOTSUP;
         uint32_t status = 0;
-        while (hcd->driver.status_hook(hcd, &status) == EOK) {
-                hcd->driver.irq_hook(hcd, status);
+        while (hcd->ops.status_hook(hcd, &status) == EOK) {
+                hcd->ops.irq_hook(hcd, status);
                 status = 0;
                 /* We should wait 1 frame - 1ms here, but this polling is a
@@ -885 +886 @@
 
         /* Need working irq replacement to setup root hub */
-        if ((irq < 0) && hcd->driver.status_hook) {
+        if ((irq < 0) && hcd->ops.status_hook) {
                 hcd->polling_fibril = fibril_create(interrupt_polling, hcd);
                 if (hcd->polling_fibril == 0) {

uspace/lib/usbhost/src/hcd.c

@@ -54 +54 @@
         assert(ep);
         assert(hcd);
-        if (hcd->driver.ep_add_hook)
-                return hcd->driver.ep_add_hook(hcd, ep);
+        if (hcd->ops.ep_add_hook)
+                return hcd->ops.ep_add_hook(hcd, ep);
         return EOK;
 }
@@ -68 +68 @@
         assert(ep);
         assert(hcd);
-        if (hcd->driver.ep_remove_hook)
-                hcd->driver.ep_remove_hook(hcd, ep);
+        if (hcd->ops.ep_remove_hook)
+                hcd->ops.ep_remove_hook(hcd, ep);
 }
 
@@ -99 +99 @@
         usb_bus_init(&hcd->bus, bandwidth, bw_count, max_speed);
 
-        hcd->driver.data = NULL;
-        hcd->driver.schedule = NULL;
-        hcd->driver.ep_add_hook = NULL;
-        hcd->driver.ep_remove_hook = NULL;
+        hcd_set_implementation(hcd, NULL, NULL);
 }
 
@@ -127 +124 @@
         assert(hcd);
         usb_address_t address = 0;
-        return usb_bus_request_address(
-            &hcd->bus, &address, true, speed);
+        return usb_bus_request_address(&hcd->bus, &address, true, speed);
 }
 
@@ -209 +205 @@
                 return ENOSPC;
         }
-        if (!hcd->driver.schedule) {
+        if (!hcd->ops.schedule) {
                 usb_log_error("HCD does not implement scheduler.\n");
                 return ENOTSUP;
@@ -241 +237 @@
         }
 
-        const int ret = hcd->driver.schedule(hcd, batch);
+        const int ret = hcd->ops.schedule(hcd, batch);
         if (ret != EOK)
                 usb_transfer_batch_destroy(batch);