brcmsmac: remove set but not used variables (55653311) · Commits · 戴 / test

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

+2 −9

Original line number	Diff line number	Diff line
		@@ -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);

Admin message