| Message ID | 1543996743-137124-1-git-send-email-yuehaibing@huawei.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 3fe970e76b374bdb4a6232cb8b78945e3d2b478f |
| Delegated to: | Kalle Valo |
| Headers | show |
| Series | wil6210: remove set but not used variable 'wdev' | expand |
On 2018-12-05 09:59, YueHaibing wrote: > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/net/wireless/ath/wil6210/main.c: In function > '_wil6210_disconnect': > drivers/net/wireless/ath/wil6210/main.c:407:23: warning: > variable 'wdev' set but not used [-Wunused-but-set-variable] > > It never used since commit ("e1b43407c034 wil6210: refactor disconnect > flow") > > Signed-off-by: YueHaibing <yuehaibing@huawei.com> > --- > drivers/net/wireless/ath/wil6210/main.c | 2 -- > 1 file changed, 2 deletions(-) Reviewed-by: Maya Erez <merez@codeaurora.org>
YueHaibing <yuehaibing@huawei.com> wrote: > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/net/wireless/ath/wil6210/main.c: In function '_wil6210_disconnect': > drivers/net/wireless/ath/wil6210/main.c:407:23: warning: > variable 'wdev' set but not used [-Wunused-but-set-variable] > > It never used since commit ("e1b43407c034 wil6210: refactor disconnect flow") > > Signed-off-by: YueHaibing <yuehaibing@huawei.com> > Reviewed-by: Maya Erez <merez@codeaurora.org> > Signed-off-by: Kalle Valo <kvalo@codeaurora.org> Patch applied to ath-next branch of ath.git, thanks. 3fe970e76b37 wil6210: remove set but not used variable 'wdev'
diff --git a/drivers/net/wireless/ath/wil6210/main.c b/drivers/net/wireless/ath/wil6210/main.c index ba6a2ee..5b7de00 100644 --- a/drivers/net/wireless/ath/wil6210/main.c +++ b/drivers/net/wireless/ath/wil6210/main.c @@ -404,7 +404,6 @@ static void _wil6210_disconnect(struct wil6210_vif *vif, const u8 *bssid, { struct wil6210_priv *wil; struct net_device *ndev; - struct wireless_dev *wdev; int cid = -ENOENT; if (unlikely(!vif)) @@ -412,7 +411,6 @@ static void _wil6210_disconnect(struct wil6210_vif *vif, const u8 *bssid, wil = vif_to_wil(vif); ndev = vif_to_ndev(vif); - wdev = vif_to_wdev(vif); might_sleep(); wil_info(wil, "disconnect bssid=%pM, reason=%d\n", bssid, reason_code);
Fixes gcc '-Wunused-but-set-variable' warning: drivers/net/wireless/ath/wil6210/main.c: In function '_wil6210_disconnect': drivers/net/wireless/ath/wil6210/main.c:407:23: warning: variable 'wdev' set but not used [-Wunused-but-set-variable] It never used since commit ("e1b43407c034 wil6210: refactor disconnect flow") Signed-off-by: YueHaibing <yuehaibing@huawei.com> --- drivers/net/wireless/ath/wil6210/main.c | 2 -- 1 file changed, 2 deletions(-)