Changeset 772a172 in mainline for uspace/drv/bus/usb/ohci/ohci.c

Timestamp:

2013-09-21T05:22:43Z (11 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

f65d9cc

Parents:

19d21728

Message:

ohci,uhci: Switch to library provided irq setup routine.

File:

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

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

@@ -103 +103 @@
         }
 
-        ret = hc_register_irq_handler(device, &regs, irq, irq_handler);
-        if (ret != EOK) {
-                usb_log_error("Failed to register interrupt handler: %s.\n",
-                    str_error(ret));
+        hc_t *hc = malloc(sizeof(hc_t));
+        if (!hc) {
+                usb_log_error("Failed to allocate driver structure.\n");
+                ret = ENOMEM;
                 goto ddf_hc_clean;
         }
@@ -112 +112 @@
         /* Try to enable interrupts */
         bool interrupts = false;
-        ret = hcd_ddf_enable_interrupts(device);
+        ret = hcd_ddf_setup_interrupts(device, &regs, irq, irq_handler,
+            hc_gen_irq_code);
         if (ret != EOK) {
                 usb_log_warning("Failed to enable interrupts: %s."
@@ -123 +124 @@
         }
 
-        hc_t *hc = malloc(sizeof(hc_t));
-        if (!hc) {
-                usb_log_error("Failed to allocate driver structure.\n");
-                ret = ENOMEM;
-                goto unregister_irq;
-        }
-
         /* Initialize OHCI HC */
         ret = hc_init(hc, &regs, interrupts);
         if (ret != EOK) {
                 usb_log_error("Failed to init hc: %s.\n", str_error(ret));
-                goto hc_free;
+                goto unregister_irq;
         }
 
@@ -147 +141 @@
                     str_error(ret));
                 hc_fini(hc);
-hc_free:
-                free(hc);
 unregister_irq:
                 unregister_interrupt_handler(device, irq);
+                free(hc);
 ddf_hc_clean:
                 hcd_ddf_clean_hc(device);