diff options
author | Sujith <Sujith.Manoharan@atheros.com> | 2009-01-16 11:08:56 -0500 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2009-01-29 16:00:43 -0500 |
commit | c656bbb582cebd988d8c39c4912722dc47578eab (patch) | |
tree | bbe3c1fe48aa27a802dda538a8a87c2a562519b8 /drivers/net/wireless/ath9k/xmit.c | |
parent | d43f301520aa64bb331736a4568d435762f980b0 (diff) |
ath9k: Cleanup buffer type assignment
The buffer state is already cleared in ATH_TXBUF_RESET.
Remove redundant code clearing the type variable.
Signed-off-by: Sujith <Sujith.Manoharan@atheros.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/ath9k/xmit.c')
-rw-r--r-- | drivers/net/wireless/ath9k/xmit.c | 28 |
1 files changed, 11 insertions, 17 deletions
diff --git a/drivers/net/wireless/ath9k/xmit.c b/drivers/net/wireless/ath9k/xmit.c index 0d05a7f8903f..1b83673c3df6 100644 --- a/drivers/net/wireless/ath9k/xmit.c +++ b/drivers/net/wireless/ath9k/xmit.c | |||
@@ -1576,27 +1576,21 @@ static int ath_tx_setup_buffer(struct ath_softc *sc, struct ath_buf *bf, | |||
1576 | 1576 | ||
1577 | bf->bf_frmlen = skb->len + FCS_LEN - (hdrlen & 3); | 1577 | bf->bf_frmlen = skb->len + FCS_LEN - (hdrlen & 3); |
1578 | 1578 | ||
1579 | ieee80211_is_data(fc) ? | 1579 | if (ieee80211_is_data(fc)) |
1580 | (bf->bf_state.bf_type |= BUF_DATA) : | 1580 | bf->bf_state.bf_type |= BUF_DATA; |
1581 | (bf->bf_state.bf_type &= ~BUF_DATA); | 1581 | if (ieee80211_is_back_req(fc)) |
1582 | ieee80211_is_back_req(fc) ? | 1582 | bf->bf_state.bf_type |= BUF_BAR; |
1583 | (bf->bf_state.bf_type |= BUF_BAR) : | 1583 | if (ieee80211_is_pspoll(fc)) |
1584 | (bf->bf_state.bf_type &= ~BUF_BAR); | 1584 | bf->bf_state.bf_type |= BUF_PSPOLL; |
1585 | ieee80211_is_pspoll(fc) ? | 1585 | if (sc->sc_flags & SC_OP_PREAMBLE_SHORT) |
1586 | (bf->bf_state.bf_type |= BUF_PSPOLL) : | 1586 | bf->bf_state.bf_type |= BUF_SHORT_PREAMBLE; |
1587 | (bf->bf_state.bf_type &= ~BUF_PSPOLL); | 1587 | if ((conf_is_ht(&sc->hw->conf) && !is_pae(skb) && |
1588 | (sc->sc_flags & SC_OP_PREAMBLE_SHORT) ? | 1588 | (tx_info->flags & IEEE80211_TX_CTL_AMPDU))) |
1589 | (bf->bf_state.bf_type |= BUF_SHORT_PREAMBLE) : | 1589 | bf->bf_state.bf_type |= BUF_HT; |
1590 | (bf->bf_state.bf_type &= ~BUF_SHORT_PREAMBLE); | ||
1591 | (conf_is_ht(&sc->hw->conf) && !is_pae(skb) && | ||
1592 | (tx_info->flags & IEEE80211_TX_CTL_AMPDU)) ? | ||
1593 | (bf->bf_state.bf_type |= BUF_HT) : | ||
1594 | (bf->bf_state.bf_type &= ~BUF_HT); | ||
1595 | 1590 | ||
1596 | bf->bf_flags = setup_tx_flags(sc, skb, txctl->txq); | 1591 | bf->bf_flags = setup_tx_flags(sc, skb, txctl->txq); |
1597 | 1592 | ||
1598 | bf->bf_keytype = get_hw_crypto_keytype(skb); | 1593 | bf->bf_keytype = get_hw_crypto_keytype(skb); |
1599 | |||
1600 | if (bf->bf_keytype != ATH9K_KEY_TYPE_CLEAR) { | 1594 | if (bf->bf_keytype != ATH9K_KEY_TYPE_CLEAR) { |
1601 | bf->bf_frmlen += tx_info->control.hw_key->icv_len; | 1595 | bf->bf_frmlen += tx_info->control.hw_key->icv_len; |
1602 | bf->bf_keyix = tx_info->control.hw_key->hw_key_idx; | 1596 | bf->bf_keyix = tx_info->control.hw_key->hw_key_idx; |