Changeset fc3dfe6d in mainline for uspace/lib/usbhost/src/utility.c

Timestamp:

2018-02-11T17:18:17Z (7 years ago)

Author:

Ondřej Hlavatý <aearsis@…>

Branches:

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

Children:

d345ce2

Parents:

239eea41

git-author:

Ondřej Hlavatý <aearsis@…> (2018-02-10 22:16:55)

git-committer:

Ondřej Hlavatý <aearsis@…> (2018-02-11 17:18:17)

Message:

usb: fix relying on negative errnos

File:

• uspace/lib/usbhost/src/utility.c (modified) (3 diffs)

uspace/lib/usbhost/src/utility.c

@@ -94 +94 @@
 
         usb_log_debug("Requesting first 8B of device descriptor to determine MPS.");
-        ssize_t got = bus_device_send_batch_sync(dev, control_ep, USB_DIRECTION_IN,
-            (char *) &desc, CTRL_PIPE_MIN_PACKET_SIZE, *(uint64_t *)&get_device_desc_8,
-            "read first 8 bytes of dev descriptor");
+        size_t got;
+        const errno_t err = bus_device_send_batch_sync(dev, control_ep,
+            USB_DIRECTION_IN, (char *) &desc, CTRL_PIPE_MIN_PACKET_SIZE,
+            *(uint64_t *)&get_device_desc_8,
+            "read first 8 bytes of dev descriptor", &got);
 
         if (got != CTRL_PIPE_MIN_PACKET_SIZE) {
-                const int err = got < 0 ? got : EOVERFLOW;
                 usb_log_error("Failed to get 8B of dev descr: %s.", str_error(err));
                 return err;
@@ -133 +134 @@
         usb_log_debug("Device(%d): Requesting full device descriptor.",
             device->address);
-        ssize_t got = bus_device_send_batch_sync(device, control_ep, USB_DIRECTION_IN,
-            (char *) desc, sizeof(*desc), *(uint64_t *)&get_device_desc,
-            "read device descriptor");
-
-        if (got < 0)
-                return got;
-
-        return got == sizeof(*desc) ? EOK : EOVERFLOW;
+        size_t got;
+        errno_t err = bus_device_send_batch_sync(device, control_ep,
+            USB_DIRECTION_IN, (char *) desc, sizeof(*desc),
+            *(uint64_t *)&get_device_desc, "read device descriptor", &got);
+
+        if (!err && got != sizeof(*desc))
+                err = EOVERFLOW;
+
+        return err;
 }
 
@@ -164 +166 @@
         usb_log_debug("Device(%d): Requesting hub descriptor.",
             device->address);
-        ssize_t got = bus_device_send_batch_sync(device, control_ep, USB_DIRECTION_IN,
-            (char *) desc, sizeof(*desc), *(uint64_t *)&get_hub_desc,
-            "get hub descriptor");
-
-        if (got < 0)
-                return got;
-
-        return got == sizeof(*desc) ? EOK : EOVERFLOW;
+
+        size_t got;
+        errno_t err = bus_device_send_batch_sync(device, control_ep,
+            USB_DIRECTION_IN, (char *) desc, sizeof(*desc),
+            *(uint64_t *)&get_hub_desc, "get hub descriptor", &got);
+
+        if (!err && got != sizeof(*desc))
+                err = EOVERFLOW;
+
+        return err;
 }