diff options
author | Luciano Coelho <luciano.coelho@intel.com> | 2015-03-01 02:10:07 -0500 |
---|---|---|
committer | Johannes Berg <johannes.berg@intel.com> | 2015-03-04 04:34:09 -0500 |
commit | 8bb6f4b9c50eef838931e0e667f1a451f34a76ae (patch) | |
tree | 03b40b947badb6df074f7b4409f23deb77c53a08 /net | |
parent | ef7c67257c9a0765bd1a53f83fd63d55a0f7b224 (diff) |
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>
Diffstat (limited to 'net')
-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) | |||
59 | cancel_work_sync(&local->dynamic_ps_enable_work); | 59 | cancel_work_sync(&local->dynamic_ps_enable_work); |
60 | del_timer_sync(&local->dynamic_ps_timer); | 60 | del_timer_sync(&local->dynamic_ps_timer); |
61 | 61 | ||
62 | local->wowlan = wowlan && local->open_count; | 62 | local->wowlan = wowlan; |
63 | if (local->wowlan) { | 63 | if (local->wowlan) { |
64 | int err; | 64 | int err; |
65 | 65 | ||