diff options
author | Luciano Coelho <luciano.coelho@intel.com> | 2015-03-01 08:10:07 +0100 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2015-03-04 10:34:09 +0100 |
commit | 8bb6f4b9c50eef838931e0e667f1a451f34a76ae (patch) | |
tree | 03b40b947badb6df074f7b4409f23deb77c53a08 | |
parent | mac80211: don't do driver suspend with auth/assoc in progress (diff) | |
download | linux-8bb6f4b9c50eef838931e0e667f1a451f34a76ae.tar.xz linux-8bb6f4b9c50eef838931e0e667f1a451f34a76ae.zip |
mac80211: remove useless double check for open_count in __ieee80211_suspend()
We check local->open_count at the top of the __ieee80211_suspend(), so
there's no need to check for it again. open_count is protected by the
rtnl, so there's no chance for it to have change between the two
calls.
Signed-off-by: Luciano Coelho <luciano.coelho@intel.com>
Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
-rw-r--r-- | net/mac80211/pm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/mac80211/pm.c b/net/mac80211/pm.c index bd185f38def1..46ffe3ad7c46 100644 --- a/net/mac80211/pm.c +++ b/net/mac80211/pm.c @@ -59,7 +59,7 @@ int __ieee80211_suspend(struct ieee80211_hw *hw, struct cfg80211_wowlan *wowlan) cancel_work_sync(&local->dynamic_ps_enable_work); del_timer_sync(&local->dynamic_ps_timer); - local->wowlan = wowlan && local->open_count; + local->wowlan = wowlan; if (local->wowlan) { int err; |