On Fri, Dec 06, 2013 at 02:16:23PM -0600, Felipe Balbi wrote: > On Sat, Nov 30, 2013 at 07:45:05PM -0300, Ezequiel Garcia wrote: > > On Thu, Nov 21, 2013 at 07:01:55AM -0600, Felipe Balbi wrote: > > > On Thu, Nov 21, 2013 at 08:55:20AM -0300, Ezequiel Garcia wrote: > > > > On Thu, Nov 21, 2013 at 12:44:51PM +0100, Sebastian Andrzej Siewior wrote: > > > > > On 11/21/2013 12:30 PM, Ezequiel Garcia wrote: > > > > > > Ah, good to know. That patch should be picked ASAP, without it the USB > > > > > > in AM335x is broken. Is it already too late to -rc1? > > > > > Yes. > > > > > > > > > > > Who is supposed to merge that? Greg? > > > > > > > > > > Felipe will start collecting fixes once -rc1 is out [0] so it should be > > > > > part of -rc2. > > > > > > > > > > > > > OK, fine by me. Please note that Felipe's commit is a much bigger (and > > > > nicer) rework of the code, and that it doesn't clearly state it fixes > > > > the current code. > > > > > > > > Felipe: Maybe you should add some message stating it fixes a regression? > > > > > > sure, makes sense. Sebastian had already mentioned it sounded too nice > > > of a commit log ;-) > > > > > > > I'm looking at -rc2 and it seems the fix was never pushed, so my board > > is probably still broken :-( > > > > Can anybody _please_please_ fix the issue by merging some fix so we can > > use USB with mainline? > > it's in Greg's tree > Yeah, I just saw the pull. Thanks for the notice! I'll re-test in -rc4 and let you know if I find any more problems. FWIW, I'm not entirely happy with the solution. Probably being a bit paranoid, but it seemed to me the fix could be smaller (and fix only the problem) and then pospone your refactoring until v3.14. Just my two cents and thanks again for the notice. -- Ezequiel García, Free Electrons Embedded Linux, Kernel and Android Engineering http://free-electrons.com -- 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
