Hi Andi, On Fri, Sep 08, 2017 at 11:21:10AM +0900, Andi Shyti wrote: > Hi Ladislav, > > > Serie was rebased on top of current linux.git, but something > > happened there and my userspace decoder no longer works: driver > > reports completely bogus timing such as (rc-5): > > ^427, _1342, ^945, _183, ^1128, _671, ^1586, _91, ^1189, _1525, > > ^1738, _1433, ^915, _1159, ^1464, _1525, ^213, _1067, ^793, _0 > > (^ used for pulse and _ for space) > > As it has nothing to do with my changes, I'm sending it anyway > > for review, which I do not expect to happen until merge window > > ends. > > This means that your patch is anyway untested. Previous version is pretty well tested. GPIO IR stopped working after pulling other changes from linux.git yesterday. And does not work even without this patch set. I'll try to bisect later as omiting linux-media merge did not fix it either. > In any case I don't see much use if patch 1/10 as it doesn't > simplify much, but the rest (from 2 to 10) looks good to me. Just look at patch 9 and imagine how it would look without this change. If you are still unconvinced I'll drop this change. > Once it's tested you can add > > Acked-by: Andi Shyti <andi.shyti@xxxxxxxxxxx> > > Andi > > P.S. I don't see in this V2 the changelog from V1. Next time, > please add the changelog. It was just a rebase with conflicts resolved. I do not see how to describe it better than I did. ladis
