Em Mon, 20 Jan 2014 12:36:26 -0500 Jarod Wilson <jarod@xxxxxxxxxx> escreveu: > On Sun, Jan 19, 2014 at 09:56:48PM +0000, Sean Young wrote: > > On Sun, Jan 19, 2014 at 10:05:15PM +0100, Martin Kittel wrote: > > > Hi Mauro, hi Sean, > ... > > > >From a71676dad29adef9cafb08598e693ec308ba2e95 Mon Sep 17 00:00:00 2001 > > > From: Martin Kittel <linux@xxxxxxxxxxxxxxxx> > > > Date: Sun, 19 Jan 2014 21:24:55 +0100 > > > Subject: [PATCH] mceusb: use endpoint xfer mode as advertised > > > > > > mceusb always sets endpoints to interrupt transfer mode no matter > > > what the device itself is advertising. This causes trouble on xhci > > > hubs. This patch changes the behavior to honor the device endpoint > > > settings. > > > > This patch is wrong. I get: > > > > [ 60.962727] ------------[ cut here ]------------ > > [ 60.962729] WARNING: CPU: 0 PID: 0 at drivers/usb/core/urb.c:452 usb_submit_u > > rb+0x1fd/0x5b0() > > [ 60.962730] usb 3-2: BOGUS urb xfer, pipe 1 != type 3 > > > > This is because the patch no longer sets the endpoints to interrupt > > endpoints, but still uses the interrupt functions like > > usb_fill_int_urb(). > > Crap, I sent a working patch to everyone a few days ago, but from a new > host that didn't have relay stuff set up yet, so I don't think anyone got > the message. Oops... I'll try to dig it back up. Its a quick fix, but its > tested as fully functional on multiple devices here, including a mix of > ones that claim bulk and interrupt, ones with no bInterval, ones with > different non-0 bIntervals, etc. Hi All, This is still pending on my queue. Any news? Regards, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
