mt76: mt76x02: use params->ssn value directly
authorStanislaw Gruszka <sgruszka@redhat.com>
Fri, 12 Jul 2019 12:07:58 +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/mt76x02_util.c

index ad5323447ed429a5c43a2ccc1f0d344deb5aa445..fa45ed280ab1707bc03dfd2f85a1d2cfcacdd207 100644 (file)
@@ -348,7 +348,7 @@ int mt76x02_ampdu_action(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
        struct mt76x02_sta *msta = (struct mt76x02_sta *) sta->drv_priv;
        struct ieee80211_txq *txq = sta->txq[params->tid];
        u16 tid = params->tid;
-       u16 *ssn = &params->ssn;
+       u16 ssn = params->ssn;
        struct mt76_txq *mtxq;
 
        if (!txq)
@@ -359,7 +359,7 @@ int mt76x02_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, params->buf_size);
+                                  ssn, params->buf_size);
                mt76_set(dev, MT_WCID_ADDR(msta->wcid.idx) + 4, BIT(16 + tid));
                break;
        case IEEE80211_AMPDU_RX_STOP:
@@ -378,7 +378,7 @@ int mt76x02_ampdu_action(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
                ieee80211_send_bar(vif, sta->addr, tid, mtxq->agg_ssn);
                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: