mac80211: move MU_MIMO_OWNER flag to ieee80211_vif

Drivers may need to track which vif is using VHT MU-MIMO.
Move the flag indicationg the ownership of MU_MIMO to
ieee80211_vif.

Signed-off-by: Sara Sharon <sara.sharon@intel.com>
Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>

4 files changed, 10 insertions, 12 deletions

diff --git a/net/mac80211/ieee80211_i.h b/net/mac80211/ieee80211_i.h
index a49c10361f1c..1630975c89f1 100644
--- a/net/mac80211/ieee80211_i.h
+++ b/net/mac80211/ieee80211_i.h
@@ -716,7 +716,6 @@ struct ieee80211_if_mesh {
  *      back to wireless media and to the local net stack.
  * @IEEE80211_SDATA_DISCONNECT_RESUME: Disconnect after resume.
  * @IEEE80211_SDATA_IN_DRIVER: indicates interface was added to driver
- * @IEEE80211_SDATA_MU_MIMO_OWNER: indicates interface owns MU-MIMO capability
  */
 enum ieee80211_sub_if_data_flags {
         IEEE80211_SDATA_ALLMULTI                = BIT(0),
@@ -724,7 +723,6 @@ enum ieee80211_sub_if_data_flags {
         IEEE80211_SDATA_DONT_BRIDGE_PACKETS     = BIT(3),
         IEEE80211_SDATA_DISCONNECT_RESUME       = BIT(4),
         IEEE80211_SDATA_IN_DRIVER               = BIT(5),
-        IEEE80211_SDATA_MU_MIMO_OWNER           = BIT(6),
 };
 
 /**
diff --git a/net/mac80211/mlme.c b/net/mac80211/mlme.c
index 857089de475f..f41625bcd879 100644
--- a/net/mac80211/mlme.c
+++ b/net/mac80211/mlme.c
@@ -6,7 +6,7 @@
  * Copyright 2006-2007  Jiri Benc <jbenc@suse.cz>
  * Copyright 2007, Michael Wu <flamingice@sourmilk.net>
  * Copyright 2013-2014  Intel Mobile Communications GmbH
- * Copyright (C) 2015 Intel Deutschland GmbH
+ * Copyright (C) 2015 - 2016 Intel Deutschland GmbH
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 2 as
@@ -559,7 +559,7 @@ static void ieee80211_add_vht_ie(struct ieee80211_sub_if_data *sdata,
                 struct ieee80211_sub_if_data *other;
 
                 list_for_each_entry_rcu(other, &local->interfaces, list) {
-                        if (other->flags & IEEE80211_SDATA_MU_MIMO_OWNER) {
+                        if (other->vif.mu_mimo_owner) {
                                 disable_mu_mimo = true;
                                 break;
                         }
@@ -567,7 +567,7 @@ static void ieee80211_add_vht_ie(struct ieee80211_sub_if_data *sdata,
                 if (disable_mu_mimo)
                         cap &= ~IEEE80211_VHT_CAP_MU_BEAMFORMEE_CAPABLE;
                 else
-                        sdata->flags |= IEEE80211_SDATA_MU_MIMO_OWNER;
+                        sdata->vif.mu_mimo_owner = true;
         }
 
         mask = IEEE80211_VHT_CAP_BEAMFORMEE_STS_MASK;
@@ -2052,7 +2052,7 @@ static void ieee80211_set_disassoc(struct ieee80211_sub_if_data *sdata,
         memset(sdata->vif.bss_conf.mu_group.position, 0,
                sizeof(sdata->vif.bss_conf.mu_group.position));
         changed |= BSS_CHANGED_MU_GROUPS;
-        sdata->flags &= ~IEEE80211_SDATA_MU_MIMO_OWNER;
+        sdata->vif.mu_mimo_owner = false;
 
         sdata->ap_power_level = IEEE80211_UNSET_POWER_LEVEL;
 
@@ -2509,7 +2509,8 @@ static void ieee80211_destroy_assoc_data(struct ieee80211_sub_if_data *sdata,
                 eth_zero_addr(sdata->u.mgd.bssid);
                 ieee80211_bss_info_change_notify(sdata, BSS_CHANGED_BSSID);
                 sdata->u.mgd.flags = 0;
-                sdata->flags &= ~IEEE80211_SDATA_MU_MIMO_OWNER;
+                sdata->vif.mu_mimo_owner = false;
+
                 mutex_lock(&sdata->local->mtx);
                 ieee80211_vif_release_channel(sdata);
                 mutex_unlock(&sdata->local->mtx);
diff --git a/net/mac80211/util.c b/net/mac80211/util.c
index f1e5b76eda70..89f71799df84 100644
--- a/net/mac80211/util.c
+++ b/net/mac80211/util.c
@@ -1928,7 +1928,7 @@ int ieee80211_reconfig(struct ieee80211_local *local)
                           BSS_CHANGED_IDLE |
                           BSS_CHANGED_TXPOWER;
 
-                if (sdata->flags & IEEE80211_SDATA_MU_MIMO_OWNER)
+                if (sdata->vif.mu_mimo_owner)
                         changed |= BSS_CHANGED_MU_GROUPS;
 
                 switch (sdata->vif.type) {
diff --git a/net/mac80211/vht.c b/net/mac80211/vht.c
index f8f161179b5d..89e04d55aa18 100644
--- a/net/mac80211/vht.c
+++ b/net/mac80211/vht.c
@@ -450,7 +450,7 @@ void ieee80211_process_mu_groups(struct ieee80211_sub_if_data *sdata,
 {
         struct ieee80211_bss_conf *bss_conf = &sdata->vif.bss_conf;
 
-        if (!(sdata->flags & IEEE80211_SDATA_MU_MIMO_OWNER))
+        if (!sdata->vif.mu_mimo_owner)
                 return;
 
         if (!memcmp(mgmt->u.action.u.vht_group_notif.position,
@@ -472,10 +472,9 @@ void ieee80211_process_mu_groups(struct ieee80211_sub_if_data *sdata,
 void ieee80211_update_mu_groups(struct ieee80211_vif *vif,
                                 const u8 *membership, const u8 *position)
 {
-        struct ieee80211_sub_if_data *sdata = vif_to_sdata(vif);
-        struct ieee80211_bss_conf *bss_conf = &sdata->vif.bss_conf;
+        struct ieee80211_bss_conf *bss_conf = &vif->bss_conf;
 
-        if (WARN_ON_ONCE(!(sdata->flags & IEEE80211_SDATA_MU_MIMO_OWNER)))
+        if (WARN_ON_ONCE(!vif->mu_mimo_owner))
                 return;
 
         memcpy(bss_conf->mu_group.membership, membership, WLAN_MEMBERSHIP_LEN);