Hello. On 01/27/2015 06:35 PM, Felipe Balbi wrote:
Fix using the bare numbers 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.shtylyov@xxxxxxxxxxxxxxxxxx>
--- drivers/usb/musb/musb_virthub.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-)
Index: usb/drivers/usb/musb/musb_virthub.c =================================================================== --- usb.orig/drivers/usb/musb/musb_virthub.c +++ usb/drivers/usb/musb/musb_virthub.c @@ -349,9 +349,10 @@ int musb_hub_control( desc->bDescriptorType = 0x29; desc->bNbrPorts = 1; desc->wHubCharacteristics = cpu_to_le16( - 0x0001 /* per-port power switching */ - | 0x0010 /* no overcurrent reporting */ - ); + HUB_CHAR_INDV_PORT_LPSM /* per-port power switching */ +
Ugh, don't know how this empty line sneaked into the patch. Greg, could you please drop it?
fixed
You mean you've merged the patch to your tree? If so, be prepared for the conflict with Greg's tree since Greg has already merged the patch to his 'usb-next' branch... Thanks anyway!
WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
