Changeset aa81adc in mainline for uspace/drv/bus/usb/ohci/hc.c

Timestamp:

2011-07-11T09:28:00Z (14 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

ece7f78

Parents:

1cb4f05

Message:

libusbhost: endpoint_t uses get/destroy pair, instead of init/destroy.

endpoint_init always returned EOK, thus allocation could be moved in.
Also, it makes no sense to create this structures on stack.

File:

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

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

@@ -235 +235 @@
     size_t mps, size_t size, unsigned interval)
 {
-        endpoint_t *ep = malloc(sizeof(endpoint_t));
+        endpoint_t *ep =
+            endpoint_get(address, endpoint, direction, type, speed, mps);
         if (ep == NULL)
                 return ENOMEM;
-        int ret =
-            endpoint_init(ep, address, endpoint, direction, type, speed, mps);
-        if (ret != EOK) {
-                free(ep);
-                return ret;
-        }
 
         hcd_endpoint_t *hcd_ep = hcd_endpoint_assign(ep);
@@ -251 +246 @@
         }
 
-        ret = usb_endpoint_manager_register_ep(&instance->ep_manager, ep, size);
+        int ret =
+            usb_endpoint_manager_register_ep(&instance->ep_manager, ep, size);
         if (ret != EOK) {
                 hcd_endpoint_clear(ep);
@@ -279 +275 @@
                     &instance->lists[ep->transfer_type], hcd_ep);
                 instance->registers->control |= C_PLE | C_IE;
-                break;
-        default:
                 break;
         }