ath9k: Fix TX status reporting for retries and MCS index
authorJouni Malinen <jouni.malinen@atheros.com>
Mon, 15 Dec 2008 13:44:53 +0000 (15:44 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 19 Dec 2008 20:23:23 +0000 (15:23 -0500)
The count field in struct ieee80211_tx_rate does not include the final
successful attempt, so only report retries here.

Fix the struct ieee80211_tx_rate::idx field when MCS was used. It is
supposed to be the MCS index, not an internal index to the rate
control algorithm table.

Signed-off-by: Jouni Malinen <jouni.malinen@atheros.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/ath9k/xmit.c

index f9c309e..d6d402c 100644 (file)
@@ -125,7 +125,15 @@ static void ath_tx_complete(struct ath_softc *sc, struct sk_buff *skb,
                tx_info->flags |= IEEE80211_TX_STAT_ACK;
        }
 
-       tx_info->status.rates[0].count = tx_status->retries + 1;
+       tx_info->status.rates[0].count = tx_status->retries;
+       if (tx_info->status.rates[0].flags & IEEE80211_TX_RC_MCS) {
+               /* Change idx from internal table index to MCS index */
+               int idx = tx_info->status.rates[0].idx;
+               struct ath_rate_table *rate_table = sc->cur_rate_table;
+               if (idx >= 0 && idx < rate_table->rate_cnt)
+                       tx_info->status.rates[0].idx =
+                               rate_table->info[idx].ratecode & 0x7f;
+       }
 
        ieee80211_tx_status(hw, skb);
 }