USB: change bInterval default to 10 ms
authorAlan Stern <stern@rowland.harvard.edu>
Fri, 16 Sep 2016 14:24:26 +0000 (10:24 -0400)
committerDanny Wood <danwood76@gmail.com>
Tue, 29 Jan 2019 13:14:36 +0000 (13:14 +0000)
commit4207f880f1562e0ee9ff1d8cb7dc870968f4268e
tree2d2f825e3a5150d3e0d37ef6d781c5975caee0ee
parent88905441c326f3b76777a171d06a853264f7a426
USB: change bInterval default to 10 ms

commit 08c5cd37480f59ea39682f4585d92269be6b1424 upstream.

Some full-speed mceusb infrared transceivers contain invalid endpoint
descriptors for their interrupt endpoints, with bInterval set to 0.
In the past they have worked out okay with the mceusb driver, because
the driver sets the bInterval field in the descriptor to 1,
overwriting whatever value may have been there before.  However, this
approach was never sanctioned by the USB core, and in fact it does not
work with xHCI controllers, because they use the bInterval value that
was present when the configuration was installed.

Currently usbcore uses 32 ms as the default interval if the value in
the endpoint descriptor is invalid.  It turns out that these IR
transceivers don't work properly unless the interval is set to 10 ms
or below.  To work around this mceusb problem, this patch changes the
endpoint-descriptor parsing routine, making the default interval value
be 10 ms rather than 32 ms.

Signed-off-by: Alan Stern <stern@rowland.harvard.edu>
Tested-by: Wade Berrier <wberrier@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Willy Tarreau <w@1wt.eu>
drivers/usb/core/config.c