Re: [PATCH 0/9 v2] IR: few fixes, additions and ENE driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 <jarod@xxxxxxxxxx>
> 
> I have no ene hardware to actually test with, but it did build. :)
> 
> For 1-9 v2:
> 
> Acked-by: Jarod Wilson <jarod@xxxxxxxxxx>
> 

Best regards,
Maxim Levitsky

--
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


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux