Changeset 888238e9 in mainline for uspace/lib/usbhost/src/bandwidth.c

Timestamp:

2017-11-20T12:56:00Z (7 years ago)

Author:

Aearsis <Hlavaty.Ondrej@…>

Branches:

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

Children:

d3086873

Parents:

ff14aede

git-author:

Aearsis <Hlavaty.Ondrej@…> (2017-11-20 12:55:58)

git-committer:

Aearsis <Hlavaty.Ondrej@…> (2017-11-20 12:56:00)

Message:

usbhost: endpoints do not have speed on their own

This information was redundant, and the fact it was never set proves it
should be removed because it is source of errors.

File:

• uspace/lib/usbhost/src/bandwidth.c (modified) (4 diffs)

uspace/lib/usbhost/src/bandwidth.c

@@ -36 +36 @@
 #include <usb/host/bandwidth.h>
 #include <usb/host/endpoint.h>
+#include <usb/host/bus.h>
 
 #include <assert.h>
@@ -42 +43 @@
 /** Calculate bandwidth that needs to be reserved for communication with EP.
  * Calculation follows USB 1.1 specification.
- * @param speed Device's speed.
- * @param type Type of the transfer.
- * @param size Number of byte to transfer.
+ * @param ep Registered endpoint
+ * @param size Number of bytes to transfer.
  * @param max_packet_size Maximum bytes in one packet.
  */
@@ -50 +50 @@
 {
         assert(ep);
+        assert(ep->device);
 
         const usb_transfer_type_t type = ep->transfer_type;
@@ -66 +67 @@
          * transaction, but I did not find text in USB spec to confirm this */
         /* NOTE: All data packets will be considered to be max_packet_size */
-        switch (ep->speed)
+        switch (ep->device->speed)
         {
         case USB_SPEED_LOW: