iwlwifi: kill iwl_{start,stop}_tx_ba_trans_ready
authorEmmanuel Grumbach <emmanuel.grumbach@intel.com>
Wed, 7 Dec 2011 08:11:00 +0000 (10:11 +0200)
committerWey-Yi Guy <wey-yi.w.guy@intel.com>
Fri, 16 Dec 2011 15:23:34 +0000 (07:23 -0800)
Since my latest patches, the upper layer reports to mac80211 that the
driver is ready to continue the start / stop BA flow as opposed to
the transport layer. Hence, iwl_{start,stop}_tx_ba_trans_ready are
not needed any more.

Signed-off-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Wey-Yi Guy <wey-yi.w.guy@intel.com>
drivers/net/wireless/iwlwifi/iwl-agn-tx.c
drivers/net/wireless/iwlwifi/iwl-core.c
drivers/net/wireless/iwlwifi/iwl-shared.h

index b4f3c0802739365691f8fa5723a075466eb65063..ae35c537b58ced855d6b1de2e2c631c73e025659 100644 (file)
@@ -432,7 +432,6 @@ drop_unlock_priv:
 int iwlagn_tx_agg_stop(struct iwl_priv *priv, struct ieee80211_vif *vif,
                        struct ieee80211_sta *sta, u16 tid)
 {
-       struct iwl_vif_priv *vif_priv = (void *)vif->drv_priv;
        struct iwl_tid_data *tid_data;
        unsigned long flags;
        int sta_id;
@@ -495,7 +494,7 @@ turn_off:
 
        spin_unlock_irqrestore(&priv->shrd->lock, flags);
 
-       iwl_stop_tx_ba_trans_ready(priv, vif_priv->ctx->ctxid, sta_id, tid);
+       ieee80211_stop_tx_ba_cb_irqsafe(vif, sta->addr, tid);
 
        return 0;
 }
@@ -503,7 +502,6 @@ turn_off:
 int iwlagn_tx_agg_start(struct iwl_priv *priv, struct ieee80211_vif *vif,
                        struct ieee80211_sta *sta, u16 tid, u16 *ssn)
 {
-       struct iwl_vif_priv *vif_priv = (void *)vif->drv_priv;
        struct iwl_tid_data *tid_data;
        unsigned long flags;
        int sta_id;
@@ -546,8 +544,7 @@ int iwlagn_tx_agg_start(struct iwl_priv *priv, struct ieee80211_vif *vif,
                IWL_DEBUG_TX_QUEUES(priv, "Can proceed: ssn = next_recl = %d",
                                    tid_data->agg.ssn);
                tid_data->agg.state = IWL_AGG_ON;
-               iwl_start_tx_ba_trans_ready(priv, vif_priv->ctx->ctxid, sta_id,
-                                           tid);
+               ieee80211_start_tx_ba_cb_irqsafe(vif, sta->addr, tid);
        } else {
                IWL_DEBUG_TX_QUEUES(priv, "Can't proceed: ssn %d, "
                                    "next_reclaimed = %d",
@@ -625,9 +622,16 @@ int iwlagn_tx_agg_oper(struct iwl_priv *priv, struct ieee80211_vif *vif,
 static void iwlagn_check_ratid_empty(struct iwl_priv *priv, int sta_id, u8 tid)
 {
        struct iwl_tid_data *tid_data = &priv->tid_data[sta_id][tid];
+       enum iwl_rxon_context_id ctx;
+       struct ieee80211_vif *vif;
+       u8 *addr;
 
        lockdep_assert_held(&priv->shrd->sta_lock);
 
+       addr = priv->stations[sta_id].sta.sta.addr;
+       ctx = priv->stations[sta_id].ctxid;
+       vif = priv->contexts[ctx].vif;
+
        switch (priv->tid_data[sta_id][tid].agg.state) {
        case IWL_EMPTYING_HW_QUEUE_DELBA:
                /* There are no packets for this RA / TID in the HW any more */
@@ -637,9 +641,7 @@ static void iwlagn_check_ratid_empty(struct iwl_priv *priv, int sta_id, u8 tid)
                                " %d", tid_data->next_reclaimed);
                        iwl_trans_tx_agg_disable(trans(priv), sta_id, tid);
                        tid_data->agg.state = IWL_AGG_OFF;
-                       iwl_stop_tx_ba_trans_ready(priv,
-                                                  NUM_IWL_RXON_CTX,
-                                                  sta_id, tid);
+                       ieee80211_stop_tx_ba_cb_irqsafe(vif, addr, tid);
                }
                break;
        case IWL_EMPTYING_HW_QUEUE_ADDBA:
@@ -649,9 +651,7 @@ static void iwlagn_check_ratid_empty(struct iwl_priv *priv, int sta_id, u8 tid)
                                "Can continue ADDBA flow ssn = next_recl ="
                                " %d", tid_data->next_reclaimed);
                        tid_data->agg.state = IWL_AGG_ON;
-                       iwl_start_tx_ba_trans_ready(priv,
-                                                   NUM_IWL_RXON_CTX,
-                                                   sta_id, tid);
+                       ieee80211_start_tx_ba_cb_irqsafe(vif, addr, tid);
                }
                break;
        default:
index d037f69afdc10f72c5097fbf645f79324a6a3ca6..fb690a2700016a1b5d545e8b3fdcdeb04760967a 100644 (file)
@@ -1584,34 +1584,6 @@ __le32 iwl_add_beacon_time(struct iwl_priv *priv, u32 base,
        return cpu_to_le32(res);
 }
 
-void iwl_start_tx_ba_trans_ready(struct iwl_priv *priv,
-                                enum iwl_rxon_context_id ctx,
-                                u8 sta_id, u8 tid)
-{
-       struct ieee80211_vif *vif;
-       u8 *addr = priv->stations[sta_id].sta.sta.addr;
-
-       if (ctx == NUM_IWL_RXON_CTX)
-               ctx = priv->stations[sta_id].ctxid;
-       vif = priv->contexts[ctx].vif;
-
-       ieee80211_start_tx_ba_cb_irqsafe(vif, addr, tid);
-}
-
-void iwl_stop_tx_ba_trans_ready(struct iwl_priv *priv,
-                               enum iwl_rxon_context_id ctx,
-                               u8 sta_id, u8 tid)
-{
-       struct ieee80211_vif *vif;
-       u8 *addr = priv->stations[sta_id].sta.sta.addr;
-
-       if (ctx == NUM_IWL_RXON_CTX)
-               ctx = priv->stations[sta_id].ctxid;
-       vif = priv->contexts[ctx].vif;
-
-       ieee80211_stop_tx_ba_cb_irqsafe(vif, addr, tid);
-}
-
 void iwl_set_hw_rfkill_state(struct iwl_priv *priv, bool state)
 {
        wiphy_rfkill_set_hw_state(priv->hw->wiphy, state);
index c458137f878a0c7cce62d3f4160cc7c1864ae917..aa1ed6549474b4c7166f1383fed41f796deb6643 100644 (file)
@@ -532,12 +532,6 @@ int __must_check iwl_rx_dispatch(struct iwl_priv *priv,
                                 struct iwl_device_cmd *cmd);
 
 int iwlagn_hw_valid_rtc_data_addr(u32 addr);
-void iwl_start_tx_ba_trans_ready(struct iwl_priv *priv,
-                                enum iwl_rxon_context_id ctx,
-                                u8 sta_id, u8 tid);
-void iwl_stop_tx_ba_trans_ready(struct iwl_priv *priv,
-                               enum iwl_rxon_context_id ctx,
-                               u8 sta_id, u8 tid);
 void iwl_set_hw_rfkill_state(struct iwl_priv *priv, bool state);
 void iwl_nic_config(struct iwl_priv *priv);
 void iwl_free_skb(struct iwl_priv *priv, struct sk_buff *skb);