diff options
author | Luis R. Rodriguez <lrodriguez@atheros.com> | 2009-07-14 20:14:05 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2009-07-24 15:05:16 -0400 |
commit | 7682a76df8f4e875e4029d95b799c712ee740ddc (patch) | |
tree | 98101f200891a974ae3dda7da59aebd2076a6c3c /drivers/net/wireless/ath/ath9k/rc.c | |
parent | 201c3b414b4b759f399502b059189d7802bbfbb6 (diff) |
ath9k: remove unnecessary IEEE80211_TX_CTL_NO_ACK checks
We check for this condition early on in our mac80211 get_rate()
callback ath_get_rate(), so remove this check later down the path.
Cc: Derek Smithies <derek@indranet.co.nz>
Cc: Chittajit Mitra <Chittajit.Mitra@Atheros.com>
Signed-off-by: Luis R. Rodriguez <lrodriguez@atheros.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/ath/ath9k/rc.c')
-rw-r--r-- | drivers/net/wireless/ath/ath9k/rc.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/net/wireless/ath/ath9k/rc.c b/drivers/net/wireless/ath/ath9k/rc.c index 96d46d697e96..2c72901adbee 100644 --- a/drivers/net/wireless/ath/ath9k/rc.c +++ b/drivers/net/wireless/ath/ath9k/rc.c | |||
@@ -777,7 +777,6 @@ static void ath_rc_rate_set_rtscts(struct ath_softc *sc, | |||
777 | * just CTS. Note that this is only done for OFDM/HT unicast frames. | 777 | * just CTS. Note that this is only done for OFDM/HT unicast frames. |
778 | */ | 778 | */ |
779 | if ((sc->sc_flags & SC_OP_PROTECT_ENABLE) && | 779 | if ((sc->sc_flags & SC_OP_PROTECT_ENABLE) && |
780 | !(tx_info->flags & IEEE80211_TX_CTL_NO_ACK) && | ||
781 | (rate_table->info[rix].phy == WLAN_RC_PHY_OFDM || | 780 | (rate_table->info[rix].phy == WLAN_RC_PHY_OFDM || |
782 | WLAN_RC_PHY_HT(rate_table->info[rix].phy))) { | 781 | WLAN_RC_PHY_HT(rate_table->info[rix].phy))) { |
783 | rates[0].flags |= IEEE80211_TX_RC_USE_CTS_PROTECT; | 782 | rates[0].flags |= IEEE80211_TX_RC_USE_CTS_PROTECT; |
@@ -882,9 +881,8 @@ static void ath_rc_ratefind(struct ath_softc *sc, | |||
882 | * | 881 | * |
883 | * FIXME: Fix duration | 882 | * FIXME: Fix duration |
884 | */ | 883 | */ |
885 | if (!(tx_info->flags & IEEE80211_TX_CTL_NO_ACK) && | 884 | if (ieee80211_has_morefrags(fc) || |
886 | (ieee80211_has_morefrags(fc) || | 885 | (le16_to_cpu(hdr->seq_ctrl) & IEEE80211_SCTL_FRAG)) { |
887 | (le16_to_cpu(hdr->seq_ctrl) & IEEE80211_SCTL_FRAG))) { | ||
888 | rates[1].count = rates[2].count = rates[3].count = 0; | 886 | rates[1].count = rates[2].count = rates[3].count = 0; |
889 | rates[1].idx = rates[2].idx = rates[3].idx = 0; | 887 | rates[1].idx = rates[2].idx = rates[3].idx = 0; |
890 | rates[0].count = ATH_TXMAXTRY; | 888 | rates[0].count = ATH_TXMAXTRY; |