> -----Original Message----- > From: Sarah Sharp [mailto:sarah.a.sharp@xxxxxxxxxxxxxxx] > Sent: Friday, July 16, 2010 5:58 AM > To: Xu, Andiry > Cc: linux-usb@xxxxxxxxxxxxxxx; gregkh@xxxxxxx > Subject: Re: [PATCH 9/9 v7] xHCI: Isoc urb enqueue > > Hi Andiry, > > I tested this patchset with several of my USB devices, and all seem to > work fine: > - SIIG USB3 hard drive > - USB2 flash drives > - USB to serial adapters (pl2303, ftdi, and pegasus drivers) > - USB to ethernet adapters > - single-TT and mulit-TT HS hubs > - HS logitech webcam (uvcvideo driver) > - FS logitech Clicksmart 310 webcam (gspca_spca500 driver) > - C-Media Electronics, Inc USB speaker (snd_usb_audio) > > However, to get my high speed webcam to work, I needed to apply your > older patch to statically add more ring segments for isochronous > endpoints. That patch is attached. Can you include it in your next > patchset? The work to dynamically expand rings is on my > xhci-ring-expansion-simplified branch, but it really needs to be cleaned > up and tested. It's probably better to include the static ring > expansion patch for now. > Sure. I'll include the patch in the next submit. > If you make the other small changes I suggested (including > not touching urb->status) and update to the master branch, then I can > ack the patchset. Thanks for putting so much time and effort into > these. :) > Thank you very much for your review and verification. It's really appreciated. > I will test the power management patchset next, but I need to debug a > suspend issue with my laptop on 2.6.35 first. > Thanks. I'll resubmit the PM patchset ASAP. Thanks, Andiry -- 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
