Re: [PATCH] brcmfmac: remove set but not used variable 'dtim_period'

From: Kalle Valo
Date: Tue Aug 06 2019 - 08:35:24 EST


YueHaibing <yuehaibing@xxxxxxxxxx> wrote:

> Fixes gcc '-Wunused-but-set-variable' warning:
>
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c: In function brcmf_update_bss_info:
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c:2962:5: warning: variable dtim_period set but not used [-Wunused-but-set-variable]
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c: In function brcmf_update_bss_info:
> drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c:2961:6: warning: variable beacon_interval set but not used [-Wunused-but-set-variable]
>
> They are never used so can be removed.
>
> Reported-by: Hulk Robot <hulkci@xxxxxxxxxx>
> Signed-off-by: YueHaibing <yuehaibing@xxxxxxxxxx>

Patch applied to wireless-drivers-next.git, thanks.

cddecd92d1ec brcmfmac: remove set but not used variable 'dtim_period'

--
https://patchwork.kernel.org/patch/11056999/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches