Phil Dibowitz wrote: > On Wed, Jun 23, 2010 at 10:27:02AM -0400, Alan Stern wrote: >> On Tue, 22 Jun 2010, Stephen Warren wrote: >> >> > I tested the sleep on a second laptop and it seems to work fine there >> too. >> > >> > I also tested using the congruity/libconcord application against the >> > remote, and the previous problems (re-enumerating the remote after >> > upgrading its configuration and resetting it) are also fixed. >> > >> > So, I'd call this fix/hack/workaround fully tested from my side. >> >> Do you want to write a patch adding a new quirk and the appropriate >> delay? > > Do we know why taking the whole routine out didn't work? Seems odd... I think it's because what the remote really needs is a delay after the command before the get-descriptor (I guess this must kick off some slow processing on the remote; perhaps the slow screen draw that shows "USB connected" on the LCD?), not a delay before get-descriptor; it's just that get-descriptor happens to (currently?) be the first command after whatever the delay-needing command is. A couple emails back I included the USB trace that showed whatever the previous command was. For a complete patch, the delay should probably be added to the tail of that function. I'll try and look into generating a quirk patch for this. It's been a very long time since I wrote a kernel patch. I'm also leaving on vacation for a couple weeks, but hopefully will have email/'net access, and can find some time. -- 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
