Changes in uspace/drv/vhc/hc.c [e63a4e1:f8597ca] in mainline

File:

• uspace/drv/vhc/hc.c (modified) (10 diffs)

uspace/drv/vhc/hc.c

@@ -68 +68 @@
 static link_t transaction_list;
 
-#define TRANSACTION_FORMAT "T[%d.%d %s/%s (%d)]"
+#define TRANSACTION_FORMAT "T[%d:%d %s (%d)]"
 #define TRANSACTION_PRINTF(t) \
         (t).target.address, (t).target.endpoint, \
-        usb_str_transfer_type((t).transfer_type), \
         usbvirt_str_transaction_type((t).type), \
         (int)(t).len
@@ -77 +76 @@
 #define transaction_get_instance(lnk) \
         list_get_instance(lnk, transaction_t, link)
-
-#define HUB_STATUS_MAX_LEN (HUB_PORT_COUNT + 64)
 
 static inline unsigned int pseudo_random(unsigned int *seed)
@@ -102 +99 @@
 /** Host controller manager main function.
  */
-static int hc_manager_fibril(void *arg)
+void hc_manager(void)
 {
         list_initialize(&transaction_list);
@@ -117 +114 @@
                 }
                 
-                char ports[HUB_STATUS_MAX_LEN + 1];
-                virthub_get_status(&virtual_hub_device, ports, HUB_STATUS_MAX_LEN);
+                char ports[HUB_PORT_COUNT + 2];
+                hub_get_port_statuses(ports, HUB_PORT_COUNT + 1);
+                dprintf(4, "virtual hub: addr=%d ports=%s",
+                    virthub_dev.address, ports);
                 
                 link_t *first_transaction_link = transaction_list.next;
@@ -126 +125 @@
                 
 
-                dprintf(0, "about to process " TRANSACTION_FORMAT " [%s]",
+                dprintf(0, "about to process " TRANSACTION_FORMAT " (vhub:%s)",
                     TRANSACTION_PRINTF(*transaction), ports);
 
@@ -139 +138 @@
                 free(transaction);
         }
-
-        assert(false && "unreachable");
-        return EOK;
-}
-
-void hc_manager(void)
-{
-        fid_t fid = fibril_create(hc_manager_fibril, NULL);
-        if (fid == 0) {
-                printf(NAME ": failed to start HC manager fibril\n");
-                return;
-        }
-        fibril_add_ready(fid);
 }
 
@@ -157 +143 @@
  */
 static transaction_t *transaction_create(usbvirt_transaction_type_t type,
-    usb_target_t target, usb_transfer_type_t transfer_type,
+    usb_target_t target,
     void * buffer, size_t len,
     hc_transaction_done_callback_t callback, void * arg)
@@ -165 +151 @@
         list_initialize(&transaction->link);
         transaction->type = type;
-        transaction->transfer_type = transfer_type;
         transaction->target = target;
         transaction->buffer = buffer;
@@ -181 +166 @@
  */
 void hc_add_transaction_to_device(bool setup, usb_target_t target,
-    usb_transfer_type_t transfer_type,
     void * buffer, size_t len,
     hc_transaction_done_callback_t callback, void * arg)
 {
         transaction_t *transaction = transaction_create(
-            setup ? USBVIRT_TRANSACTION_SETUP : USBVIRT_TRANSACTION_OUT,
-            target, transfer_type,
+            setup ? USBVIRT_TRANSACTION_SETUP : USBVIRT_TRANSACTION_OUT, target,
             buffer, len, callback, arg);
         list_append(&transaction->link, &transaction_list);
@@ -195 +178 @@
  */
 void hc_add_transaction_from_device(usb_target_t target,
-    usb_transfer_type_t transfer_type,
     void * buffer, size_t len,
     hc_transaction_done_callback_t callback, void * arg)
 {
         transaction_t *transaction = transaction_create(USBVIRT_TRANSACTION_IN,
-            target, transfer_type,
-            buffer, len, callback, arg);
+            target, buffer, len, callback, arg);
         list_append(&transaction->link, &transaction_list);
 }