mt76: mt7603: use params->ssn value directly
authorStanislaw Gruszka <sgruszka@redhat.com>
Fri, 12 Jul 2019 12:07:59 +0000 (14:07 +0200)
committerFelix Fietkau <nbd@nbd.name>
Thu, 5 Sep 2019 15:42:29 +0000 (17:42 +0200)
There is no point to use pointer to params->ssn.

Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
Signed-off-by: Felix Fietkau <nbd@nbd.name>
drivers/net/wireless/mediatek/mt76/mt7603/main.c

index e5d4cb6..d70f42d 100644 (file)
@@ -569,7 +569,7 @@ mt7603_ampdu_action(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
        struct ieee80211_txq *txq = sta->txq[params->tid];
        struct mt7603_sta *msta = (struct mt7603_sta *)sta->drv_priv;
        u16 tid = params->tid;
-       u16 *ssn = &params->ssn;
+       u16 ssn = params->ssn;
        u8 ba_size = params->buf_size;
        struct mt76_txq *mtxq;
 
@@ -580,7 +580,7 @@ mt7603_ampdu_action(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
 
        switch (action) {
        case IEEE80211_AMPDU_RX_START:
-               mt76_rx_aggr_start(&dev->mt76, &msta->wcid, tid, *ssn,
+               mt76_rx_aggr_start(&dev->mt76, &msta->wcid, tid, ssn,
                                   params->buf_size);
                mt7603_mac_rx_ba_reset(dev, sta->addr, tid);
                break;
@@ -599,7 +599,7 @@ mt7603_ampdu_action(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
                mt7603_mac_tx_ba_reset(dev, msta->wcid.idx, tid, -1);
                break;
        case IEEE80211_AMPDU_TX_START:
-               mtxq->agg_ssn = IEEE80211_SN_TO_SEQ(*ssn);
+               mtxq->agg_ssn = IEEE80211_SN_TO_SEQ(ssn);
                ieee80211_start_tx_ba_cb_irqsafe(vif, sta->addr, tid);
                break;
        case IEEE80211_AMPDU_TX_STOP_CONT: