Changeset 8b71f3e in mainline for uspace/drv/hid

Timestamp:

2018-01-14T21:16:03Z (8 years ago)

Author:

Petr Manek <petr.manek@…>

Branches:

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

Children:

17c1d9db

Parents:

edc51615

git-author:

Petr Manek <petr.manek@…> (2018-01-14 21:16:00)

git-committer:

Petr Manek <petr.manek@…> (2018-01-14 21:16:03)

Message:

usbdev: refactor polling more

For clarity, the opaque usb_device_polling_t and its complementary
configuration data structure usb_device_polling_config_t have been
merged into usb_polling_t. All related methods have dropped the
"device" from their prefix as well.

The usage semantics have transitioned to malloc-free model, where the
user is entirely responsible for (de)allocation of the polling structure
and its data buffer, and (de)initialization happens in designated
functions during its lifetime in the system.

In addition, the distinction between mandatory / optional / internal
parameters has been documented. Optional parameters now have default
values, which are set to sensible constants in order to allow dropping
some lines in USB driver implementations.

The drivers usbhid and usbhub were refactored to match the API
changes.

Location:

uspace/drv/hid/usbhid

Files:

• main.c (modified) (2 diffs)
• usbhid.c (modified) (2 diffs)
• usbhid.h (modified) (1 diff)

uspace/drv/hid/usbhid/main.c

@@ -89 +89 @@
          * This will create a separate fibril that will query the device
          * for the data continuously. */
-        const usb_device_polling_config_t config = {
-                .debug = 1,
-                .auto_clear_halt = true,
-                .delay = -1,
-                .max_failures = 3,
-                .on_data = usb_hid_polling_callback,
-                .on_polling_end = usb_hid_polling_ended_callback,
-                .on_error = usb_hid_polling_error_callback,
-                .arg = hid_dev,
-        };
-
-        rc = usb_device_poll(dev, hid_dev->poll_pipe_mapping, &config,
-            hid_dev->poll_pipe_mapping->pipe.desc.max_transfer_size, &hid_dev->polling);
+        rc = usb_polling_start(&hid_dev->polling);
 
         if (rc != EOK) {
@@ -122 +110 @@
         assert(hid_dev);
 
-        /* Join polling fibril. */
-        usb_device_poll_join(hid_dev->polling);
+        /* Join polling fibril (ignoring error code). */
+        usb_polling_join(&hid_dev->polling);
 
         /* Clean up. */

uspace/drv/hid/usbhid/usbhid.c

@@ -445 +445 @@
                             ".\n");
                 }
+
+                usb_polling_t *polling = &hid_dev->polling;
+                if ((rc = usb_polling_init(polling))) {
+                        // FIXME
+                }
+
+                polling->device = hid_dev->usb_dev;
+                polling->ep_mapping = hid_dev->poll_pipe_mapping;
+                polling->request_size = hid_dev->poll_pipe_mapping->pipe.desc.max_transfer_size;
+                polling->buffer = malloc(polling->request_size);
+                polling->on_data = usb_hid_polling_callback,
+                polling->on_polling_end = usb_hid_polling_ended_callback,
+                polling->on_error = usb_hid_polling_error_callback,
+                polling->arg = hid_dev;
         }
 
@@ -537 +551 @@
         assert(hid_dev->subdrivers != NULL || hid_dev->subdriver_count == 0);
 
+        free(hid_dev->polling.buffer);
+        usb_polling_fini(&hid_dev->polling);
+
         usb_log_debug("Subdrivers: %p, subdriver count: %d\n",
             hid_dev->subdrivers, hid_dev->subdriver_count);

uspace/drv/hid/usbhid/usbhid.h

@@ -107 +107 @@
         usb_endpoint_mapping_t *poll_pipe_mapping;
 
-        /** Device polling handle. */
-        usb_device_polling_t *polling;
+        /** Device polling structure. */
+        usb_polling_t polling;
 
         /** Subdrivers. */