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

Timestamp:

2011-07-11T12:22:56Z (15 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

5fd0dc23

Parents:

01bbbb2

Message:

libusb: Fix hub descriptor parsing: USB is little endian.

usbhub: Fix checks for power capabilities.

File:

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

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

@@ -249 +249 @@
         usb_log_debug("setting port count to %d\n", descriptor->ports_count);
         hub_info->port_count = descriptor->ports_count;
-        bool is_power_switched =
-            ((descriptor->hub_characteristics & 1) == 0);
-        bool has_individual_port_powering =
-            ((descriptor->hub_characteristics & 1) != 0);
+        const bool is_power_switched =
+            ((descriptor->hub_characteristics & 0x2) == 0);
         hub_info->ports = malloc(
             sizeof (usb_hub_port_t) * (hub_info->port_count + 1));
@@ -264 +262 @@
         if (is_power_switched) {
                 usb_log_debug("Hub power switched\n");
+                const bool has_individual_port_powering =
+                    descriptor->hub_characteristics & 0x1;
 
                 if (!has_individual_port_powering) {