Hi Thomas, On Thursday 17 April 2014 09:53:32 Thomas Pugliese wrote: > On Thu, 17 Apr 2014, Laurent Pinchart wrote: > > On Wednesday 16 April 2014 12:29:22 Thomas Pugliese wrote:
[snip] > > > As you had mentioned previously, it should be possible to support both > > > formats by ignoring the endpoint descriptor and looking at the > > > bMaxBurst, bOverTheAirInterval and wOverTheAirPacketSize fields in the > > > WUSB endpoint companion descriptor. That is a more involved change to > > > the UVC driver and also would require changes to USB core to store the > > > WUSB endpoint companion descriptor in struct usb_host_endpoint similar > > > to what is done for super speed devices. > > > > It's more complex indeed, but I believe it would be worth it. Any > > volunteer ? ;-) In the meantime I'm fine with a patch that reverts to the > > previous behaviour. Please include the explanation of the problem in the > > commit message. > > I may make an attempt at the more complete fix once I finish some of the > other items in my queue. > > For clarification, would you like a patch that reverts to the pre-super > speed behavior where windows-compatible devices work not but spec > compliant devices will not (i.e. treat USB_SPEED_HIGH and > USB_SPEED_WIRELESS the same)? I'll trust your judgment on that, if you believe it would be better from a user point of view, please send a patch. Otherwise we can wait until you find time to work on a proper fix. -- Regards, Laurent Pinchart -- 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
