diff options
author | Rajkumar Manoharan <rmanohar@qca.qualcomm.com> | 2011-09-26 12:18:39 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2011-09-30 15:56:57 -0400 |
commit | 185d1589ccc9f49afcdaede480523df2bfec7c01 (patch) | |
tree | 62268a4f05c7a469e7e9893ec1b972434d68e4ba | |
parent | 89888e368eebb8d5c3dbf58425b95fc773aee511 (diff) |
ath9k: Remove unnecessary AMPDU check at tx status
Fill the ampdu_[ack]_len for both aggregation and normal frames.
So that we could avoid unnecesary conditional at tx status.
Signed-off-by: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r-- | drivers/net/wireless/ath/ath9k/rc.c | 6 | ||||
-rw-r--r-- | drivers/net/wireless/ath/ath9k/xmit.c | 5 |
2 files changed, 2 insertions, 9 deletions
diff --git a/drivers/net/wireless/ath/ath9k/rc.c b/drivers/net/wireless/ath/ath9k/rc.c index 4f1301881137..8448281dd069 100644 --- a/drivers/net/wireless/ath/ath9k/rc.c +++ b/drivers/net/wireless/ath/ath9k/rc.c | |||
@@ -1362,12 +1362,6 @@ static void ath_tx_status(void *priv, struct ieee80211_supported_band *sband, | |||
1362 | if (tx_info->flags & IEEE80211_TX_STAT_TX_FILTERED) | 1362 | if (tx_info->flags & IEEE80211_TX_STAT_TX_FILTERED) |
1363 | return; | 1363 | return; |
1364 | 1364 | ||
1365 | if (!(tx_info->flags & IEEE80211_TX_STAT_AMPDU)) { | ||
1366 | tx_info->status.ampdu_ack_len = | ||
1367 | (tx_info->flags & IEEE80211_TX_STAT_ACK ? 1 : 0); | ||
1368 | tx_info->status.ampdu_len = 1; | ||
1369 | } | ||
1370 | |||
1371 | if (!(tx_info->flags & IEEE80211_TX_STAT_ACK)) | 1365 | if (!(tx_info->flags & IEEE80211_TX_STAT_ACK)) |
1372 | tx_status = 1; | 1366 | tx_status = 1; |
1373 | 1367 | ||
diff --git a/drivers/net/wireless/ath/ath9k/xmit.c b/drivers/net/wireless/ath/ath9k/xmit.c index fa3dcfdf7174..f5d4764888b9 100644 --- a/drivers/net/wireless/ath/ath9k/xmit.c +++ b/drivers/net/wireless/ath/ath9k/xmit.c | |||
@@ -2043,10 +2043,9 @@ static void ath_tx_rc_status(struct ath_softc *sc, struct ath_buf *bf, | |||
2043 | tx_info->flags |= IEEE80211_TX_STAT_AMPDU; | 2043 | tx_info->flags |= IEEE80211_TX_STAT_AMPDU; |
2044 | 2044 | ||
2045 | BUG_ON(nbad > nframes); | 2045 | BUG_ON(nbad > nframes); |
2046 | |||
2047 | tx_info->status.ampdu_len = nframes; | ||
2048 | tx_info->status.ampdu_ack_len = nframes - nbad; | ||
2049 | } | 2046 | } |
2047 | tx_info->status.ampdu_len = nframes; | ||
2048 | tx_info->status.ampdu_ack_len = nframes - nbad; | ||
2050 | 2049 | ||
2051 | if ((ts->ts_status & ATH9K_TXERR_FILT) == 0 && | 2050 | if ((ts->ts_status & ATH9K_TXERR_FILT) == 0 && |
2052 | (tx_info->flags & IEEE80211_TX_CTL_NO_ACK) == 0) { | 2051 | (tx_info->flags & IEEE80211_TX_CTL_NO_ACK) == 0) { |