Changeset 508a0ca in mainline

Timestamp:

2011-04-12T10:23:03Z (13 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

f324635

Parents:

d017cea

Message:

Rename transfer ⇒ td

File:

• uspace/drv/uhci-hcd/batch.c (modified) (14 diffs)

uspace/drv/uhci-hcd/batch.c

@@ -48 +48 @@
         qh_t *qh;
         td_t *tds;
-        size_t transfers;
+        size_t td_count;
 } uhci_batch_t;
 
@@ -63 +63 @@
  * @param[in] target Device and endpoint target of the transaction.
  * @param[in] transfer_type Interrupt, Control or Bulk.
- * @param[in] max_packet_size maximum allowed size of data transfers.
+ * @param[in] max_packet_size maximum allowed size of data transactions.
  * @param[in] speed Speed of the transaction.
  * @param[in] buffer Data source/destination.
@@ -76 +76 @@
  * NULL otherwise.
  *
- * Determines the number of needed transfers (TDs). Prepares a transport buffer
- * (that is accessible by the hardware). Initializes parameters needed for the
- * transaction and callback.
+ * Determines the number of needed transfer descriptors (TDs).
+ * Prepares a transport buffer (that is accessible by the hardware).
+ * Initializes parameters needed for the transaction and callback.
  */
 usb_transfer_batch_t * batch_get(ddf_fun_t *fun, endpoint_t *ep,
@@ -113 +113 @@
         instance->private_data = data;
 
-        data->transfers =
+        data->td_count =
             (buffer_size + ep->max_packet_size - 1) / ep->max_packet_size;
         if (ep->transfer_type == USB_TRANSFER_CONTROL) {
-                data->transfers += 2;
-        }
-
-        data->tds = malloc32(sizeof(td_t) * data->transfers);
+                data->td_count += 2;
+        }
+
+        data->tds = malloc32(sizeof(td_t) * data->td_count);
         CHECK_NULL_DISPOSE_RETURN(
             data->tds, "Failed to allocate transfer descriptors.\n");
-        bzero(data->tds, sizeof(td_t) * data->transfers);
+        bzero(data->tds, sizeof(td_t) * data->td_count);
 
         data->qh = malloc32(sizeof(qh_t));
@@ -164 +164 @@
 
         usb_log_debug2("Batch(%p) checking %d transfer(s) for completion.\n",
-            instance, data->transfers);
+            instance, data->td_count);
         instance->transfered_size = 0;
         size_t i = 0;
-        for (;i < data->transfers; ++i) {
+        for (;i < data->td_count; ++i) {
                 if (td_is_active(&data->tds[i])) {
                         return false;
@@ -290 +290 @@
  *
  * @param[in] instance Batch structure to use.
- * @param[in] pid Pid to use for data transfers.
+ * @param[in] pid Pid to use for data transactions.
  *
  * Packets with alternating toggle bit and supplied pid value.
@@ -305 +305 @@
         assert(toggle == 0 || toggle == 1);
 
-        size_t transfer = 0;
+        size_t td = 0;
         size_t remain_size = instance->buffer_size;
         while (remain_size > 0) {
@@ -316 +316 @@
                     remain_size : instance->ep->max_packet_size;
 
-                td_t *next_transfer = (transfer + 1 < data->transfers)
-                    ? &data->tds[transfer + 1] : NULL;
-
-                assert(transfer < data->transfers);
+                td_t *next_td = (td + 1 < data->td_count)
+                    ? &data->tds[td + 1] : NULL;
+
+                assert(td < data->td_count);
                 assert(packet_size <= remain_size);
                 usb_target_t target =
@@ -325 +325 @@
 
                 td_init(
-                    &data->tds[transfer], DEFAULT_ERROR_COUNT, packet_size,
-                    toggle, false, low_speed, target, pid, trans_data,
-                    next_transfer);
+                    &data->tds[td], DEFAULT_ERROR_COUNT, packet_size,
+                    toggle, false, low_speed, target, pid, trans_data, next_td);
 
 
                 toggle = 1 - toggle;
                 remain_size -= packet_size;
-                ++transfer;
-        }
-        td_set_ioc(&data->tds[transfer - 1]);
+                ++td;
+        }
+        td_set_ioc(&data->tds[td - 1]);
         endpoint_toggle_set(instance->ep, toggle);
 }
@@ -341 +340 @@
  *
  * @param[in] instance Batch structure to use.
- * @param[in] data_stage Pid to use for data transfers.
- * @param[in] status_stage Pid to use for data transfers.
+ * @param[in] data_stage Pid to use for data tds.
+ * @param[in] status_stage Pid to use for data tds.
  *
  * Setup stage with toggle 0 and USB_PID_SETUP.
@@ -355 +354 @@
         uhci_batch_t *data = instance->private_data;
         assert(data);
-        assert(data->transfers >= 2);
+        assert(data->td_count >= 2);
 
         const bool low_speed = instance->ep->speed == USB_SPEED_LOW;
@@ -369 +368 @@
 
         /* data stage */
-        size_t transfer = 1;
+        size_t td = 1;
         size_t remain_size = instance->buffer_size;
         while (remain_size > 0) {
@@ -383 +382 @@
 
                 td_init(
-                    &data->tds[transfer], DEFAULT_ERROR_COUNT, packet_size,
+                    &data->tds[td], DEFAULT_ERROR_COUNT, packet_size,
                     toggle, false, low_speed, target, data_stage,
-                    control_data, &data->tds[transfer + 1]);
-
-                ++transfer;
-                assert(transfer < data->transfers);
+                    control_data, &data->tds[td + 1]);
+
+                ++td;
+                assert(td < data->td_count);
                 assert(packet_size <= remain_size);
                 remain_size -= packet_size;
@@ -394 +393 @@
 
         /* status stage */
-        assert(transfer == data->transfers - 1);
+        assert(td == data->td_count - 1);
 
         td_init(
-            &data->tds[transfer], DEFAULT_ERROR_COUNT, 0, 1, false, low_speed,
+            &data->tds[td], DEFAULT_ERROR_COUNT, 0, 1, false, low_speed,
             target, status_stage, NULL, NULL);
-        td_set_ioc(&data->tds[transfer]);
+        td_set_ioc(&data->tds[td]);
 
         usb_log_debug2("Control last TD status: %x.\n",
-            data->tds[transfer].status);
+            data->tds[td].status);
 }
 /*----------------------------------------------------------------------------*/