Re: [patch] mac80211: remove always true condition

2015-08-13 Thread Johannes Berg
On Mon, 2015-07-27 at 11:11 +0300, Dan Carpenter wrote: > The outside if statement checks that IEEE80211_TX_INTFL_MLME_CONN_TX > is > set so this condition is always true. Checking twice upsets the > static > checkers. > Yeah, this was left over in some refactoring - thanks, applied. johannes

[patch] mac80211: remove always true condition

2015-07-27 Thread Dan Carpenter
The outside if statement checks that IEEE80211_TX_INTFL_MLME_CONN_TX is set so this condition is always true. Checking twice upsets the static checkers. Signed-off-by: Dan Carpenter diff --git a/net/mac80211/status.c b/net/mac80211/status.c index 45628f3..8ba5832 100644 --- a/net/mac80211/statu