Changeset 7dddd7b in mainline for uspace/drv/bus/usb/usbhub/usbhub.c

Timestamp:

2018-01-13T20:47:58Z (6 years ago)

Author:

Petr Manek <petr.manek@…>

Branches:

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

Children:

71f211f

Parents:

2489353

git-author:

Petr Manek <petr.manek@…> (2018-01-13 19:20:41)

git-committer:

Petr Manek <petr.manek@…> (2018-01-13 20:47:58)

Message:

usbdev: refactor polling

Until now, device polling had to be executed by calling one of four
proxy functions, which served as a syntax sugar and had no clear
distinction between each other (not to mention misleading names and high
number of arguments).

In this commit, the four mentioned functions are discarded in favor of
one main function, which was proxied by them either way. The number of
arguments have decreased in favor of named struct fields in the auto
polling config structure.

Drivers, which make use of polling, such as usbhid and usbhub were
updated to the latest API design.

File:

• uspace/drv/bus/usb/usbhub/usbhub.c (modified) (1 diff)

uspace/drv/bus/usb/usbhub/usbhub.c

@@ -164 +164 @@
 
         /* Start hub operation. */
-        opResult = usb_device_auto_poll_desc(hub_dev->usb_device,
-            &hub_status_change_endpoint_description,
-            hub_port_changes_callback, ((hub_dev->port_count + 1 + 7) / 8),
-            -1, usb_hub_polling_error_callback,
-            usb_hub_polling_terminated_callback, hub_dev);
+        const usb_device_auto_polling_t auto_polling = {
+                .debug = 1,
+                .auto_clear_halt = true,
+                .delay = -1,
+                .max_failures = 3,
+                .on_data = hub_port_changes_callback,
+                .on_polling_end = usb_hub_polling_terminated_callback,
+                .on_error = usb_hub_polling_error_callback,
+                .arg = hub_dev,
+        };
+
+        usb_endpoint_mapping_t *epm =
+            usb_device_get_mapped_ep_desc(hub_dev->usb_device,
+            &hub_status_change_endpoint_description);
+        opResult = usb_device_auto_polling(hub_dev->usb_device, epm,
+            &auto_polling, ((hub_dev->port_count + 1 + 7) / 8));
+        
         if (opResult != EOK) {
                 /* Function is already bound */