mt76: mt7615: use params->ssn value directly
authorStanislaw Gruszka <sgruszka@redhat.com>
Fri, 12 Jul 2019 12:08:00 +0000 (14:08 +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/mt7615/main.c

index 3f5f355..2c702b3 100644 (file)
@@ -443,7 +443,7 @@ mt7615_ampdu_action(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
        struct ieee80211_txq *txq = sta->txq[params->tid];
        struct mt7615_sta *msta = (struct mt7615_sta *)sta->drv_priv;
        u16 tid = params->tid;
-       u16 *ssn = &params->ssn;
+       u16 ssn = params->ssn;
        struct mt76_txq *mtxq;
 
        if (!txq)
@@ -453,7 +453,7 @@ mt7615_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);
                mt7615_mcu_set_rx_ba(dev, params, 1);
                break;
@@ -473,7 +473,7 @@ mt7615_ampdu_action(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
                mt7615_mcu_set_tx_ba(dev, params, 0);
                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: