Changeset 46577995 in mainline for uspace/drv/bus/usb/vhc/transfer.c

Timestamp:

2018-01-04T20:50:52Z (8 years ago)

Author:

Jiří Zárevúcky <zarevucky.jiri@…>

Children:

e211ea04

Parents:

facacc71

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2018-01-04 20:47:53)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2018-01-04 20:50:52)

Message:

Use errno_t in all uspace and kernel code.

Change type of every variable, parameter and return value that holds an
<errno.h> constant to either errno_t (the usual case), or sys_errno_t
(some places in kernel). This is for the purpose of self-documentation,
as well as for type-checking with a bit of type definition hackery.

After this commit, HelenOS is free of code that mixes error codes with non-error
values on the assumption that error codes are negative.

File:

• uspace/drv/bus/usb/vhc/transfer.c (modified) (7 diffs)

uspace/drv/bus/usb/vhc/transfer.c

@@ -58 +58 @@
 }
 
-static int process_transfer_local(usb_transfer_batch_t *batch,
+static errno_t process_transfer_local(usb_transfer_batch_t *batch,
     usbvirt_device_t *dev, size_t *actual_data_size)
 {
-        int rc;
+        errno_t rc;
         
         const usb_direction_t dir = usb_transfer_batch_direction(batch);
@@ -94 +94 @@
 }
 
-static int process_transfer_remote(usb_transfer_batch_t *batch,
+static errno_t process_transfer_remote(usb_transfer_batch_t *batch,
     async_sess_t *sess, size_t *actual_data_size)
 {
-        int rc;
+        errno_t rc;
 
         const usb_direction_t dir = usb_transfer_batch_direction(batch);
@@ -143 +143 @@
 
 static void execute_transfer_callback_and_free(vhc_transfer_t *transfer,
-    size_t data_transfer_size, int outcome)
+    size_t data_transfer_size, errno_t outcome)
 {
         assert(outcome != ENAK);
@@ -154 +154 @@
 }
 
-int vhc_init(vhc_data_t *instance)
+errno_t vhc_init(vhc_data_t *instance)
 {
         assert(instance);
@@ -163 +163 @@
 }
 
-int vhc_schedule(hcd_t *hcd, usb_transfer_batch_t *batch)
+errno_t vhc_schedule(hcd_t *hcd, usb_transfer_batch_t *batch)
 {
         assert(hcd);
@@ -199 +199 @@
 }
 
-int vhc_transfer_queue_processor(void *arg)
+errno_t vhc_transfer_queue_processor(void *arg)
 {
         vhc_virtdev_t *dev = arg;
@@ -214 +214 @@
                 fibril_mutex_unlock(&dev->guard);
 
-                int rc = EOK;
+                errno_t rc = EOK;
                 size_t data_transfer_size = 0;
                 if (dev->dev_sess) {