Changeset e0d8b740 in mainline for uspace/drv/bus/usb/uhci/hc.c

Timestamp:

2013-08-07T08:26:40Z (11 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

6340a6ff

Parents:

cce3228

Message:

uhci: remove some more error handling macros

File:

• uspace/drv/bus/usb/uhci/hc.c (modified) (3 diffs)

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

@@ -208 +208 @@
 {
         assert(reg_size >= sizeof(uhci_regs_t));
-        int ret;
-
-#define CHECK_RET_RETURN(ret, message...) \
-        if (ret != EOK) { \
-                usb_log_error(message); \
-                return ret; \
-        } else (void) 0
 
         instance->hw_interrupts = interrupts;
@@ -221 +214 @@
         /* allow access to hc control registers */
         uhci_regs_t *io;
-        ret = pio_enable(regs, reg_size, (void **)&io);
-        CHECK_RET_RETURN(ret, "Failed to gain access to registers at %p: %s.\n",
-            io, str_error(ret));
+        int ret = pio_enable(regs, reg_size, (void **)&io);
+        if (ret != EOK) {
+                usb_log_error("Failed to gain access to registers at %p: %s.\n",
+                    io, str_error(ret));
+                return ret;
+        }
         instance->registers = io;
+
         usb_log_debug(
             "Device registers at %p (%zuB) accessible.\n", io, reg_size);
 
         ret = hc_init_mem_structures(instance);
-        CHECK_RET_RETURN(ret,
-            "Failed to initialize UHCI memory structures: %s.\n",
-            str_error(ret));
-
-#undef CHECK_RET_RETURN
+        if (ret != EOK) {
+                usb_log_error("Failed to init UHCI memory structures: %s.\n",
+                    str_error(ret));
+                // TODO: we should disable pio here
+                return ret;
+        }
 
         hc_init_hw(instance);
@@ -391 +389 @@
 
         return EOK;
-#undef CHECK_RET_CLEAR_RETURN
 }