Changeset 4c25c2fb in mainline for uspace/lib/usbhost

Timestamp:

2018-01-15T15:02:57Z (8 years ago)

Author:

Ondřej Hlavatý <aearsis@…>

Branches:

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

Children:

745a3f1

Parents:

01d9707

Message:

usbhost: move managing TT to the library

Location:

uspace/lib/usbhost

Files:

• include/usb/host/bus.h (modified) (1 diff)
• include/usb/host/hcd.h (modified) (1 diff)
• src/bus.c (modified) (2 diffs)
• src/hcd.c (modified) (1 diff)
• src/usb2_bus.c (modified) (1 diff)

uspace/lib/usbhost/include/usb/host/bus.h

@@ -71 +71 @@
 
         /* Transaction translator */
-        usb_tt_address_t tt;
+        struct {
+                device_t *dev;
+                unsigned port;
+        } tt;
 
         /* The following are not set by the library */

uspace/lib/usbhost/include/usb/host/hcd.h

@@ -114 +114 @@
  */
 extern int hcd_get_ep0_max_packet_size(uint16_t *, bus_t *, device_t *);
-extern void hcd_setup_device_tt(device_t *);
 
 /** How many toggles need to be reset */

uspace/lib/usbhost/src/bus.c

@@ -97 +97 @@
 
 /**
+ * Setup devices Transaction Translation.
+ *
+ * This applies for Low/Full speed devices under High speed hub only. Other
+ * devices just inherit TT from the hub.
+ *
+ * Roothub must be handled specially.
+ */
+static void device_setup_tt(device_t *dev)
+{
+        if (!dev->hub)
+                return;
+
+        if (dev->hub->speed == USB_SPEED_HIGH && usb_speed_is_11(dev->speed)) {
+                /* For LS devices under HS hub */
+                dev->tt.dev = dev->hub;
+                dev->tt.port = dev->port;
+        }
+        else {
+                /* Inherit hub's TT */
+                dev->tt = dev->hub->tt;
+        }
+}
+
+/**
  * Invoke the device_enumerate bus operation.
  *
@@ -111 +135 @@
         if (dev->online)
                 return EINVAL;
+
+        device_setup_tt(dev);
 
         const int r = ops->device_enumerate(dev);

uspace/lib/usbhost/src/hcd.c

@@ -426 +426 @@
         }
         return EOK;
-}
-
-/**
- * Setup devices Transaction Translation.
- *
- * This applies for Low/Full speed devices under High speed hub only. Other
- * devices just inherit TT from the hub.
- *
- * Roothub must be handled specially.
- */
-void hcd_setup_device_tt(device_t *dev)
-{
-        if (!dev->hub)
-                return;
-
-        if (dev->hub->speed == USB_SPEED_HIGH && usb_speed_is_11(dev->speed)) {
-                /* For LS devices under HS hub */
-                dev->tt.address = dev->hub->address;
-                dev->tt.port = dev->port;
-        }
-        else {
-                /* Inherit hub's TT */
-                dev->tt = dev->hub->tt;
-        }
 }
 

uspace/lib/usbhost/src/usb2_bus.c

@@ -196 +196 @@
         usb_log_debug("Found new %s speed USB device.", usb_str_speed(dev->speed));
 
-        if (!dev->hub) {
-                /* The device is the roothub */
-                dev->tt = (usb_tt_address_t) {
-                        .address = -1,
-                        .port = 0,
-                };
-        } else {
-                hcd_setup_device_tt(dev);
-        }
-
         /* Assign an address to the device */
         if ((err = address_device(dev))) {