Changeset 4fd3faf in mainline for uspace/drv/uhci-hcd

Timestamp:

2011-04-08T11:58:18Z (15 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

ffc63b0

Parents:

cd1cec3b

Message:

Use per endpoint communication mutex

Location:

uspace/drv/uhci-hcd

Files:

• hc.c (modified) (2 diffs)
• transfer_list.c (modified) (2 diffs)

uspace/drv/uhci-hcd/hc.c

@@ -332 +332 @@
             instance->transfers[batch->speed][batch->transfer_type];
         assert(list);
-        if (batch->transfer_type == USB_TRANSFER_CONTROL) {
-                usb_device_keeper_use_control(
-                    &instance->manager, batch->target);
-        }
         transfer_list_add_batch(list, batch);
 
@@ -373 +369 @@
                         usb_transfer_batch_t *batch =
                             list_get_instance(item, usb_transfer_batch_t, link);
-                        switch (batch->transfer_type)
-                        {
-                        case USB_TRANSFER_CONTROL:
-                                usb_device_keeper_release_control(
-                                    &instance->manager, batch->target);
-                                break;
-                        case USB_TRANSFER_INTERRUPT:
-                        case USB_TRANSFER_ISOCHRONOUS: {
-                                }
-                        default:
-                                break;
-                        }
                         usb_transfer_batch_finish(batch);
                 }

uspace/drv/uhci-hcd/transfer_list.c

@@ -132 +132 @@
 }
 /*----------------------------------------------------------------------------*/
-/** Check list for finished batches.
- *
- * @param[in] instance List to use.
- * @return Error code
- *
- * Creates a local list of finished batches and calls next_step on each and
- * every one. This is safer because next_step may theoretically access
- * this transfer list leading to the deadlock if its done inline.
+/** Create list for finished batches.
+ *
+ * @param[in] instance List to use.
+ * @param[in] done list to fill
  */
 void transfer_list_remove_finished(transfer_list_t *instance, link_t *done)
@@ -161 +157 @@
         }
         fibril_mutex_unlock(&instance->guard);
-
 }
 /*----------------------------------------------------------------------------*/