Fix  using the  bare number  to set the 'wHubCharacteristics' field of the Hub
Descriptor while the values are #define'd in <linux/usb/ch11.h>.

Signed-off-by: Sergei Shtylyov <sergei.shtyl...@cogentembedded.com>

---
 drivers/usb/host/isp116x-hcd.c |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Index: usb/drivers/usb/host/isp116x-hcd.c
===================================================================
--- usb.orig/drivers/usb/host/isp116x-hcd.c
+++ usb/drivers/usb/host/isp116x-hcd.c
@@ -948,7 +948,10 @@ static void isp116x_hub_descriptor(struc
        desc->bHubContrCurrent = 0;
        desc->bNbrPorts = (u8) (reg & 0x3);
        /* Power switching, device type, overcurrent. */
-       desc->wHubCharacteristics = cpu_to_le16((u16) ((reg >> 8) & 0x1f));
+       desc->wHubCharacteristics = cpu_to_le16((u16) ((reg >> 8) &
+                                                      (HUB_CHAR_LPSM |
+                                                       HUB_CHAR_COMPOUND |
+                                                       HUB_CHAR_OCPM)));
        desc->bPwrOn2PwrGood = (u8) ((reg >> 24) & 0xff);
        /* ports removable, and legacy PortPwrCtrlMask */
        desc->u.hs.DeviceRemovable[0] = 0;

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to