On Tue, 2015-12-08 at 21:25 +0530, Krishna Chaitanya wrote: > On Tue, Dec 8, 2015 at 9:23 PM, Johannes Berg <johannes@sipsolutions. > net> wrote: > > On Tue, 2015-12-08 at 21:17 +0530, Krishna Chaitanya wrote: > > > > > > > If the rate control algorithm doesn't look at sta->sta.rx_nss > > > > then > > > > that's their bug. > > > > > > Yes, it looks like it. Only BW is handled, not the NSS change, > > > and > > > without this patch OP MODE IE in beacon updates neither NSS nor > > > BW. > > > > > > For Action frame OP MODE IE, NSS will be updated. > Typo BW will be updated. Ok. > > I don't see how that would happen - they end up in exactly the same > > code path. > > As nss_only is false, BW will be update in case of action frame. Well, this patch removes that since the whole concept of nss_only was wrong. So basically you're just saying that minstrel should be fixed to look at rx_nss (and the relevant change bit, perhaps)... johannes -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html