ath9k: Nuke struct ath_xmit_status
authorVasanthakumar Thiagarajan <vasanth@atheros.com>
Fri, 20 Mar 2009 09:57:50 +0000 (15:27 +0530)
committerJohn W. Linville <linville@tuxdriver.com>
Sat, 28 Mar 2009 00:13:03 +0000 (20:13 -0400)
Signed-off-by: Vasanthakumar Thiagarajan <vasanth@atheros.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath9k/ath9k.h
drivers/net/wireless/ath9k/xmit.c

index 2b02564551184098ff84cbf825593884f828eb1f..2689a08a2844e045d97b67d2fbcf1f658bfe3e37 100644 (file)
@@ -295,13 +295,9 @@ struct ath_tx_control {
        enum ath9k_internal_frame_type frame_type;
 };
 
-struct ath_xmit_status {
-       int retries;
-       int flags;
 #define ATH_TX_ERROR        0x01
 #define ATH_TX_XRETRY       0x02
 #define ATH_TX_BAR          0x04
-};
 
 /* All RSSI values are noise floor adjusted */
 struct ath_tx_stat {
index 21e90bca35018b97c5c9494f779441a3eb02e6fc..689bdbf78808bc4f32c2a3c316c28ef1c9481c87 100644 (file)
@@ -1750,7 +1750,7 @@ exit:
 /*****************/
 
 static void ath_tx_complete(struct ath_softc *sc, struct sk_buff *skb,
-                           struct ath_xmit_status *tx_status)
+                           int tx_flags)
 {
        struct ieee80211_hw *hw = sc->hw;
        struct ieee80211_tx_info *tx_info = IEEE80211_SKB_CB(skb);
@@ -1771,12 +1771,10 @@ static void ath_tx_complete(struct ath_softc *sc, struct sk_buff *skb,
                tx_info->rate_driver_data[0] = NULL;
        }
 
-       if (tx_status->flags & ATH_TX_BAR) {
+       if (tx_flags & ATH_TX_BAR)
                tx_info->flags |= IEEE80211_TX_STAT_AMPDU_NO_BACK;
-               tx_status->flags &= ~ATH_TX_BAR;
-       }
 
-       if (!(tx_status->flags & (ATH_TX_ERROR | ATH_TX_XRETRY))) {
+       if (!(tx_flags & (ATH_TX_ERROR | ATH_TX_XRETRY))) {
                /* Frame was ACKed */
                tx_info->flags |= IEEE80211_TX_STAT_ACK;
        }
@@ -1803,29 +1801,22 @@ static void ath_tx_complete_buf(struct ath_softc *sc, struct ath_buf *bf,
                                int txok, int sendbar)
 {
        struct sk_buff *skb = bf->bf_mpdu;
-       struct ath_xmit_status tx_status;
        unsigned long flags;
+       int tx_flags = 0;
 
-       /*
-        * Set retry information.
-        * NB: Don't use the information in the descriptor, because the frame
-        * could be software retried.
-        */
-       tx_status.retries = bf->bf_retries;
-       tx_status.flags = 0;
 
        if (sendbar)
-               tx_status.flags = ATH_TX_BAR;
+               tx_flags = ATH_TX_BAR;
 
        if (!txok) {
-               tx_status.flags |= ATH_TX_ERROR;
+               tx_flags |= ATH_TX_ERROR;
 
                if (bf_isxretried(bf))
-                       tx_status.flags |= ATH_TX_XRETRY;
+                       tx_flags |= ATH_TX_XRETRY;
        }
 
        dma_unmap_single(sc->dev, bf->bf_dmacontext, skb->len, DMA_TO_DEVICE);
-       ath_tx_complete(sc, skb, &tx_status);
+       ath_tx_complete(sc, skb, tx_flags);
 
        /*
         * Return the list of ath_buf of this mpdu to free queue