brcmsmac: remove set but not used variables
authorYueHaibing <yuehaibing@huawei.com>
Fri, 1 Nov 2019 13:50:35 +0000 (21:50 +0800)
committerKalle Valo <kvalo@codeaurora.org>
Wed, 6 Nov 2019 17:56:04 +0000 (19:56 +0200)
drivers/net/wireless/broadcom/brcm80211/brcmsmac/main.c:841:7: warning: variable free_pdu set but not used [-Wunused-but-set-variable]
drivers/net/wireless/broadcom/brcm80211/brcmsmac/main.c:842:30: warning: variable tx_rts_count set but not used [-Wunused-but-set-variable]
drivers/net/wireless/broadcom/brcm80211/brcmsmac/main.c:842:6: warning: variable tx_rts set but not used [-Wunused-but-set-variable]
drivers/net/wireless/broadcom/brcm80211/brcmsmac/main.c:843:7: warning: variable totlen set but not used [-Wunused-but-set-variable]

They are never used, so can be removed.

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/broadcom/brcm80211/brcmsmac/main.c

index 6bb34a1..ff39773 100644 (file)
@@ -838,9 +838,8 @@ brcms_c_dotxstatus(struct brcms_c_info *wlc, struct tx_status *txs)
        struct dma_pub *dma = NULL;
        struct d11txh *txh = NULL;
        struct scb *scb = NULL;
-       bool free_pdu;
-       int tx_rts, tx_frame_count, tx_rts_count;
-       uint totlen, supr_status;
+       int tx_frame_count;
+       uint supr_status;
        bool lastframe;
        struct ieee80211_hdr *h;
        u16 mcl;
@@ -917,11 +916,8 @@ brcms_c_dotxstatus(struct brcms_c_info *wlc, struct tx_status *txs)
                             CHSPEC_CHANNEL(wlc->default_bss->chanspec));
        }
 
-       tx_rts = le16_to_cpu(txh->MacTxControlLow) & TXC_SENDRTS;
        tx_frame_count =
            (txs->status & TX_STATUS_FRM_RTX_MASK) >> TX_STATUS_FRM_RTX_SHIFT;
-       tx_rts_count =
-           (txs->status & TX_STATUS_RTS_RTX_MASK) >> TX_STATUS_RTS_RTX_SHIFT;
 
        lastframe = !ieee80211_has_morefrags(h->frame_control);
 
@@ -989,9 +985,6 @@ brcms_c_dotxstatus(struct brcms_c_info *wlc, struct tx_status *txs)
                        tx_info->flags |= IEEE80211_TX_STAT_ACK;
        }
 
-       totlen = p->len;
-       free_pdu = true;
-
        if (lastframe) {
                /* remove PLCP & Broadcom tx descriptor header */
                skb_pull(p, D11_PHY_HDR_LEN);