Commit abd984e6 authored by Sujith's avatar Sujith Committed by John W. Linville

ath9k_htc: Use proper station add/remove callbacks

sta_add/sta_remove are the callbacks that can sleep.
Use them instead of sta_notify.
Signed-off-by: default avatarSujith <Sujith.Manoharan@atheros.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 9d1ac34e
...@@ -1516,9 +1516,8 @@ static void ath9k_htc_configure_filter(struct ieee80211_hw *hw, ...@@ -1516,9 +1516,8 @@ static void ath9k_htc_configure_filter(struct ieee80211_hw *hw,
mutex_unlock(&priv->mutex); mutex_unlock(&priv->mutex);
} }
static void ath9k_htc_sta_notify(struct ieee80211_hw *hw, static int ath9k_htc_sta_add(struct ieee80211_hw *hw,
struct ieee80211_vif *vif, struct ieee80211_vif *vif,
enum sta_notify_cmd cmd,
struct ieee80211_sta *sta) struct ieee80211_sta *sta)
{ {
struct ath9k_htc_priv *priv = hw->priv; struct ath9k_htc_priv *priv = hw->priv;
...@@ -1526,22 +1525,29 @@ static void ath9k_htc_sta_notify(struct ieee80211_hw *hw, ...@@ -1526,22 +1525,29 @@ static void ath9k_htc_sta_notify(struct ieee80211_hw *hw,
mutex_lock(&priv->mutex); mutex_lock(&priv->mutex);
ath9k_htc_ps_wakeup(priv); ath9k_htc_ps_wakeup(priv);
switch (cmd) {
case STA_NOTIFY_ADD:
ret = ath9k_htc_add_station(priv, vif, sta); ret = ath9k_htc_add_station(priv, vif, sta);
if (!ret) if (!ret)
ath9k_htc_init_rate(priv, sta); ath9k_htc_init_rate(priv, sta);
break; ath9k_htc_ps_restore(priv);
case STA_NOTIFY_REMOVE: mutex_unlock(&priv->mutex);
ath9k_htc_remove_station(priv, vif, sta);
break; return ret;
default: }
break;
}
static int ath9k_htc_sta_remove(struct ieee80211_hw *hw,
struct ieee80211_vif *vif,
struct ieee80211_sta *sta)
{
struct ath9k_htc_priv *priv = hw->priv;
int ret;
mutex_lock(&priv->mutex);
ath9k_htc_ps_wakeup(priv);
ret = ath9k_htc_remove_station(priv, vif, sta);
ath9k_htc_ps_restore(priv); ath9k_htc_ps_restore(priv);
mutex_unlock(&priv->mutex); mutex_unlock(&priv->mutex);
return ret;
} }
static int ath9k_htc_conf_tx(struct ieee80211_hw *hw, u16 queue, static int ath9k_htc_conf_tx(struct ieee80211_hw *hw, u16 queue,
...@@ -1849,7 +1855,8 @@ struct ieee80211_ops ath9k_htc_ops = { ...@@ -1849,7 +1855,8 @@ struct ieee80211_ops ath9k_htc_ops = {
.remove_interface = ath9k_htc_remove_interface, .remove_interface = ath9k_htc_remove_interface,
.config = ath9k_htc_config, .config = ath9k_htc_config,
.configure_filter = ath9k_htc_configure_filter, .configure_filter = ath9k_htc_configure_filter,
.sta_notify = ath9k_htc_sta_notify, .sta_add = ath9k_htc_sta_add,
.sta_remove = ath9k_htc_sta_remove,
.conf_tx = ath9k_htc_conf_tx, .conf_tx = ath9k_htc_conf_tx,
.bss_info_changed = ath9k_htc_bss_info_changed, .bss_info_changed = ath9k_htc_bss_info_changed,
.set_key = ath9k_htc_set_key, .set_key = ath9k_htc_set_key,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment