Re: [PATCH 3/6] mac80211: cleanup generic suspend/resume procedures
From: Johannes Berg <johannes@sipsolutions.net>
Date: 2013-02-26 20:49:28
On Mon, 2013-02-25 at 12:50 +0100, Stanislaw Gruszka wrote:
quoted hunk ↗ jump to hunk
@@ -95,17 +75,10 @@ int __ieee80211_suspend(struct ieee80211_hw *hw, struct cfg80211_wowlan *wowlan) WARN_ON(err != 1); local->wowlan = false; } else {
Hmmm! The context here reminds me that it is possible to have WoWLAN configured, but have the driver reject it and then suspend "normally". Our driver seems to be the only one doing it (both dvm and mvm flavours) and it does it only in case we aren't connected ... However overall this patchset will break the functionality, if WoWLAN is enabled but not possible we won't cleanly disconnect. This is a bit of a problem, I guess we need to propagate this return value behaviour all the way into cfg80211 where the cfg80211_leave() can then be called? johannes