On Wed, 2010-07-28 at 23:52 -0400, Jarod Wilson wrote: > On Thu, Jul 29, 2010 at 02:40:43AM +0300, Maxim Levitsky wrote: > > Hi, > > This is second version of the patchset. > > Hopefully, I didn't forget to address all comments. > > > > In addition to comments, I changed helper function that processes samples > > so it sends last space as soon as timeout is reached. > > This breaks somewhat lirc, because now it gets 2 spaces in row. > > However, if it uses timeout reports (which are now fully supported) > > it will get such report in middle. > > > > Note that I send timeout report with zero value. > > I don't think that this value is importaint. > > I just patched the entire series into a branch here and tested, no > regressions with an mceusb transceiver with in-kernel decode, lirc decode > or lirc tx. Only issue I had (which I neglected to mention earlier) was > some pedantic issues w/whitespace. Here's the tree I built and tested: > > http://git.wilsonet.com/linux-2.6-ir-wip.git/?a=shortlog;h=refs/heads/maxim > > 7486d6ae3 addresses all the whitespace/formatting issues I had. Could > either merge that into your patches, or I can just send it along as an > additional patch after the fact. In either case, for 1-7 v2: About whitespace, I usually fix what checkpacth.pl tells me. Nothing beyond that :-)
> > Tested-by: Jarod Wilson <ja...@redhat.com> > > I have no ene hardware to actually test with, but it did build. :) > > For 1-9 v2: > > Acked-by: Jarod Wilson <ja...@redhat.com> > Best regards, Maxim Levitsky -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
