Re: [PATCH v2] wl12xx: support FW TX inactivity triggers

[Luciano Coelho <coelho@ti.com>]

On Mon, 2011-05-02 at 10:37 -0400, John W. Linville wrote:
> On Mon, May 02, 2011 at 12:50:55PM +0300, Luciano Coelho wrote:
> > Hi John and Arik,
> > 
> > On Sat, 2011-04-30 at 00:09 +0300, Arik Nemtsov wrote:
> > > Hey John,
> > > 
> > > Looking at the latest git, it seems you've picked up this patch
> > > (commit 47684808fd89d6809c0886e06f8ac324252499d8).
> > > This is unfortunate, since there's currently a FW bug that will cause
> > > many disconnections in STA mode with this patch.
> > > 
> > > How would we go about reverting this?
> > 
> > I have reverted this patch in the wl12xx.git tree.  This required a bit
> > of merging and we needed the revert in order to proceed with our tests.
> > 
> > So, John, please don't revert this in your tree.  My revert will come in
> > my next pull-request which will come in a couple of days, after our
> > internal test round is done.
> > 
> > Thanks!
> 
> OK, sounds good.  FWIW, this is what I meant when I said "Do the
> wl12xx guys still want this?"  Granted, that was about the other
> patch in the series, but I kinda presumed that they were related...

Yes, no problem.  I agree that sometimes it makes more sense to apply
this kind of patches together (as you did), especially when both patches
were sent as part of the same series.

The mac80211 part is ready and we thought the wl12xx was good too, but
our internal tests have shown that there are still some problems with
this in our driver, which was decreasing connection stability in
general, thus we wanted this to get reverted.  So, you didn't to
anything wrong (obviously, it was our own fault. ;)

-- 
Cheers,
Luca.