Changeset 9d3536e in mainline for uspace/drv/bus/usb/usbhub/port.c

Timestamp:

2018-01-20T22:17:40Z (6 years ago)

Author:

Ondřej Hlavatý <aearsis@…>

Branches:

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

Children:

defaab2

Parents:

d648e83

Message:

usbhub: clear ignored changes and report them only once

File:

• uspace/drv/bus/usb/usbhub/port.c (modified) (3 diffs)

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

@@ -255 +255 @@
         }
 
-        for (uint32_t feature = 0; feature < sizeof(usb_port_status_t) * 8; ++feature) {
+        for (uint32_t feature = 16; feature < sizeof(usb_port_status_t) * 8; ++feature) {
                 uint32_t mask = 1 << feature;
 
@@ -261 +261 @@
                         continue;
 
+                /* Clear the change so it won't come again */
+                usb_hub_clear_port_feature(port->hub, port->port_number, feature);
+
                 if (!port_change_handlers[feature])
                         continue;
@@ -266 +269 @@
                 /* ACK this change */
                 status &= ~mask;
-                usb_hub_clear_port_feature(port->hub, port->port_number, feature);
 
                 port_change_handlers[feature](port, status);
         }
 
-        port_log(debug2, port, "Port status after handling: %#08" PRIx32, status);
+        /* Check for changes we ignored */
+        if (status & 0xffff0000) {
+                port_log(debug, port, "Port status change igored. Status: %#08" PRIx32, status);
+        }
 }