Changeset 8c877b2 in mainline for uspace/drv/uhci-hcd/main.c

Timestamp:

2011-03-04T13:05:35Z (13 years ago)

Author:

Matus Dekanek <smekideki@…>

Branches:

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

Children:

d49728c

Parents:

dff940f8 (diff), 9a422574 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

merge with \usb\development

File:

• uspace/drv/uhci-hcd/main.c (modified) (4 diffs)

uspace/drv/uhci-hcd/main.c

@@ -34 +34 @@
 #include <ddf/driver.h>
 #include <ddf/interrupt.h>
+#include <device/hw_res.h>
+#include <errno.h>
+#include <str_error.h>
+
 #include <usb_iface.h>
 #include <usb/ddfiface.h>
-#include <device/hw_res.h>
-
-#include <errno.h>
-
 #include <usb/debug.h>
 
@@ -50 +50 @@
 
 static int uhci_add_device(ddf_dev_t *device);
-
 /*----------------------------------------------------------------------------*/
 static driver_ops_t uhci_driver_ops = {
@@ -70 +69 @@
 }
 /*----------------------------------------------------------------------------*/
-#define CHECK_RET_RETURN(ret, message...) \
+static int uhci_add_device(ddf_dev_t *device)
+{
+        assert(device);
+        uhci_t *hcd = NULL;
+#define CHECK_RET_FREE_HC_RETURN(ret, message...) \
 if (ret != EOK) { \
         usb_log_error(message); \
+        if (hcd != NULL) \
+                free(hcd); \
         return ret; \
 }
 
-static int uhci_add_device(ddf_dev_t *device)
-{
-        assert(device);
-
         usb_log_info("uhci_add_device() called\n");
 
-
-        uintptr_t io_reg_base;
-        size_t io_reg_size;
-        int irq;
+        uintptr_t io_reg_base = 0;
+        size_t io_reg_size = 0;
+        int irq = 0;
 
         int ret =
             pci_get_my_registers(device, &io_reg_base, &io_reg_size, &irq);
-
-        CHECK_RET_RETURN(ret,
+        CHECK_RET_FREE_HC_RETURN(ret,
             "Failed(%d) to get I/O addresses:.\n", ret, device->handle);
         usb_log_info("I/O regs at 0x%X (size %zu), IRQ %d.\n",
             io_reg_base, io_reg_size, irq);
 
+        ret = pci_disable_legacy(device);
+        CHECK_RET_FREE_HC_RETURN(ret,
+            "Failed(%d) disable legacy USB: %s.\n", ret, str_error(ret));
+
+#if 0
         ret = pci_enable_interrupts(device);
-        CHECK_RET_RETURN(ret, "Failed(%d) to get enable interrupts:\n", ret);
+        if (ret != EOK) {
+                usb_log_warning(
+                    "Failed(%d) to enable interrupts, fall back to polling.\n",
+                    ret);
+        }
+#endif
 
-        uhci_t *uhci_hc = malloc(sizeof(uhci_t));
-        ret = (uhci_hc != NULL) ? EOK : ENOMEM;
-        CHECK_RET_RETURN(ret, "Failed to allocate memory for uhci hcd driver.\n");
+        hcd = malloc(sizeof(uhci_t));
+        ret = (hcd != NULL) ? EOK : ENOMEM;
+        CHECK_RET_FREE_HC_RETURN(ret,
+            "Failed(%d) to allocate memory for uhci hcd.\n", ret);
 
-        ret = uhci_init(uhci_hc, device, (void*)io_reg_base, io_reg_size);
-        if (ret != EOK) {
-                usb_log_error("Failed to init uhci-hcd.\n");
-                free(uhci_hc);
-                return ret;
-        }
+        ret = uhci_init(hcd, device, (void*)io_reg_base, io_reg_size);
+        CHECK_RET_FREE_HC_RETURN(ret, "Failed(%d) to init uhci-hcd.\n",
+            ret);
+#undef CHECK_RET_FREE_HC_RETURN
 
         /*
-         * We might free uhci_hc, but that does not matter since no one
+         * We might free hcd, but that does not matter since no one
          * else would access driver_data anyway.
          */
-        device->driver_data = uhci_hc;
+        device->driver_data = hcd;
 
+        ddf_fun_t *rh = NULL;
+#define CHECK_RET_FINI_FREE_RETURN(ret, message...) \
+if (ret != EOK) { \
+        usb_log_error(message); \
+        if (hcd != NULL) {\
+                uhci_fini(hcd); \
+                free(hcd); \
+        } \
+        if (rh != NULL) \
+                free(rh); \
+        return ret; \
+}
+
+        /* It does no harm if we register this on polling */
         ret = register_interrupt_handler(device, irq, irq_handler,
-            &uhci_hc->interrupt_code);
-        if (ret != EOK) {
-                usb_log_error("Failed to register interrupt handler.\n");
-                uhci_fini(uhci_hc);
-                free(uhci_hc);
-                return ret;
-        }
+            &hcd->interrupt_code);
+        CHECK_RET_FINI_FREE_RETURN(ret,
+            "Failed(%d) to register interrupt handler.\n", ret);
 
-        ddf_fun_t *rh;
         ret = setup_root_hub(&rh, device);
-        if (ret != EOK) {
-                usb_log_error("Failed to setup uhci root hub.\n");
-                uhci_fini(uhci_hc);
-                free(uhci_hc);
-                return ret;
-        }
-        rh->driver_data = uhci_hc->ddf_instance;
+        CHECK_RET_FINI_FREE_RETURN(ret,
+            "Failed(%d) to setup UHCI root hub.\n", ret);
+        rh->driver_data = hcd->ddf_instance;
 
         ret = ddf_fun_bind(rh);
-        if (ret != EOK) {
-                usb_log_error("Failed to register root hub.\n");
-                uhci_fini(uhci_hc);
-                free(uhci_hc);
-                free(rh);
-                return ret;
-        }
+        CHECK_RET_FINI_FREE_RETURN(ret,
+            "Failed(%d) to register UHCI root hub.\n", ret);
 
         return EOK;
+#undef CHECK_RET_FINI_FREE_RETURN
 }
 /*----------------------------------------------------------------------------*/
@@ -149 +158 @@
 {
         sleep(3);
-        usb_log_enable(USB_LOG_LEVEL_INFO, NAME);
+        usb_log_enable(USB_LOG_LEVEL_DEBUG, NAME);
 
         return ddf_driver_main(&uhci_driver);