Changes in uspace/drv/bus/usb/ohci/hw_struct/endpoint_descriptor.c [acdb5bac:0d4b110] in mainline

File:

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

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

@@ -32 +32 @@
  * @brief OHCI driver
  */
+
+#include <assert.h>
+#include <macros.h>
+#include <mem.h>
+
+#include <usb/usb.h>
+
+#include "../utils/malloc32.h"
+#include "mem_access.h"
+
 #include "endpoint_descriptor.h"
 
@@ -48 +58 @@
  * @param td TD to put in the list.
  *
- * If @param ep is NULL, dummy ED is initalized with only skip flag set.
+ * If @param ep is NULL, dummy ED is initialized with only skip flag set.
  */
 void ed_init(ed_t *instance, const endpoint_t *ep, const td_t *td)
@@ -61 +71 @@
                 return;
         }
-        /* Non-dummy ED must have TD assigned */
+        /* Non-dummy ED must have corresponding EP and TD assigned */
         assert(td);
+        assert(ep);
+        assert(ep->direction < ARRAY_SIZE(dir));
 
         /* Status: address, endpoint nr, direction mask and max packet size. */
@@ -77 +89 @@
 
         /* Isochronous format flag */
+        // TODO: We need iTD instead of TD for iso transfers
         if (ep->transfer_type == USB_TRANSFER_ISOCHRONOUS)
                 OHCI_MEM32_SET(instance->status, ED_STATUS_F_FLAG);