Changeset 89a0485a in mainline for uspace/drv/uhci/uhci.c

Timestamp:

2011-01-25T20:44:28Z (13 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

b00163f

Parents:

7977fa1

Message:

Move function implementations to .c files

Unify error reporting in uhci_in()

File:

• uspace/drv/uhci/uhci.c (modified) (2 diffs)

uspace/drv/uhci/uhci.c

@@ -22 +22 @@
         if (!instance)
                 { return ENOMEM; }
-        memset( instance, 0, sizeof(uhci_t) );
+        bzero( instance, sizeof(uhci_t) );
 
         /* init address keeper(libusb) */
@@ -86 +86 @@
                 return ENOTSUP;
 
-        callback_t *job = malloc(sizeof(callback_t));
-        if (job == NULL) {
-                uhci_print_error("Failed to allocate callback structure.\n");
-                return ENOMEM;
-        }
+        transfer_descriptor_t *td = NULL;
+        callback_t *job = NULL;
+        int ret = EOK;
 
-#define CHECK_RET_FREE_JOB(r, message) \
-        if (r != EOK) { \
+#define CHECK_RET_TRANS_FREE_JOB_TD(message) \
+        if (ret != EOK) { \
                 uhci_print_error(message); \
-                callback_fini(job); \
-                trans_free(job); \
-                return r; \
+                if (job) { \
+                        callback_fini(job); \
+                        trans_free(job); \
+                } \
+                if (td) { trans_free(td); } \
+                return ret; \
         } else (void) 0
 
-        int ret = callback_in_init(job, dev, buffer, size, callback, arg);
-        CHECK_RET_FREE_JOB(ret, "Failed to initialize callback structure.\n");
 
-        transfer_descriptor_t *td = trans_malloc(sizeof(transfer_descriptor_t));
-        ret = (td == NULL) ? ENOMEM : EOK;
-        CHECK_RET_FREE_JOB(ret, "Failed to allocate tranfer descriptor.\n");
+        job = malloc(sizeof(callback_t));
+        ret= job ? EOK : ENOMEM;
+        CHECK_RET_TRANS_FREE_JOB_TD("Failed to allocate callback structure.\n");
+
+        ret = callback_in_init(job, dev, buffer, size, callback, arg);
+        CHECK_RET_TRANS_FREE_JOB_TD("Failed to initialize callback structure.\n");
+
+        td = trans_malloc(sizeof(transfer_descriptor_t));
+        ret = td ? ENOMEM : EOK;
+        CHECK_RET_TRANS_FREE_JOB_TD("Failed to allocate tranfer descriptor.\n");
 
         ret = transfer_descriptor_init(td, 3, size, false, target, USB_PID_IN);
-        if (ret != EOK) {
-                uhci_print_error("Failed to initialize transfer descriptor.\n");
-                trans_free(td);
-                trans_free(job);
-                return ret;
-        }
+        CHECK_RET_TRANS_FREE_JOB_TD("Failed to initialize transfer descriptor.\n");
+
         td->callback = job;
 
+        assert(dev);
+        uhci_t *instance = (uhci_t*)dev->driver_data;
+        assert(instance);
 
-        // TODO: add to list
-//      callback(dev, USB_OUTCOME_OK, size, arg);
+        ret = transfer_list_append(&instance->transfers[transfer_type], td);
+        CHECK_RET_TRANS_FREE_JOB_TD("Failed to append transfer descriptor.\n");
 
         return EOK;