Re: [PATCH -next] staging: wfx: remove set but not used variable 'tx_priv'
On Tuesday 11 February 2020 16:21:41 CET Jérôme Pouiller wrote: > On Tuesday 11 February 2020 15:03:34 CET YueHaibing wrote: > > drivers/staging/wfx/queue.c: In function wfx_tx_queues_get: > > drivers/staging/wfx/queue.c:484:28: warning: variable tx_priv set but not > > used [-Wunused-but-set-variable] > > > > commit 2e57865e79cf ("staging: wfx: pspoll_mask make no sense") > > left behind this unused variable. > > > > Reported-by: Hulk Robot > > Signed-off-by: YueHaibing Maybe it could make sens to add a Fixes tag with the commit id that introduce the warning? -- Jérôme Pouiller ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [PATCH -next] staging: wfx: remove set but not used variable 'tx_priv'
On Tuesday 11 February 2020 15:03:34 CET YueHaibing wrote: > drivers/staging/wfx/queue.c: In function wfx_tx_queues_get: > drivers/staging/wfx/queue.c:484:28: warning: variable tx_priv set but not > used [-Wunused-but-set-variable] > > commit 2e57865e79cf ("staging: wfx: pspoll_mask make no sense") > left behind this unused variable. > > Reported-by: Hulk Robot > Signed-off-by: YueHaibing > --- > drivers/staging/wfx/queue.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/staging/wfx/queue.c b/drivers/staging/wfx/queue.c > index 0bcc61f..c73d158 100644 > --- a/drivers/staging/wfx/queue.c > +++ b/drivers/staging/wfx/queue.c > @@ -481,7 +481,6 @@ struct hif_msg *wfx_tx_queues_get(struct wfx_dev *wdev) > struct wfx_queue *vif_queue = NULL; > u32 tx_allowed_mask = 0; > u32 vif_tx_allowed_mask = 0; > - const struct wfx_tx_priv *tx_priv = NULL; > struct wfx_vif *wvif; > int not_found; > int burst; > @@ -541,7 +540,6 @@ struct hif_msg *wfx_tx_queues_get(struct wfx_dev *wdev) > skb = wfx_tx_queue_get(wdev, queue, tx_allowed_mask); > if (!skb) > continue; > - tx_priv = wfx_skb_tx_priv(skb); > hif = (struct hif_msg *) skb->data; > wvif = wdev_to_wvif(wdev, hif->interface); > WARN_ON(!wvif); > -- > 2.7.4 Reviewed-by: Jérôme Pouiller -- Jérôme Pouiller ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
[PATCH -next] staging: wfx: remove set but not used variable 'tx_priv'
drivers/staging/wfx/queue.c: In function wfx_tx_queues_get: drivers/staging/wfx/queue.c:484:28: warning: variable tx_priv set but not used [-Wunused-but-set-variable] commit 2e57865e79cf ("staging: wfx: pspoll_mask make no sense") left behind this unused variable. Reported-by: Hulk Robot Signed-off-by: YueHaibing --- drivers/staging/wfx/queue.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/wfx/queue.c b/drivers/staging/wfx/queue.c index 0bcc61f..c73d158 100644 --- a/drivers/staging/wfx/queue.c +++ b/drivers/staging/wfx/queue.c @@ -481,7 +481,6 @@ struct hif_msg *wfx_tx_queues_get(struct wfx_dev *wdev) struct wfx_queue *vif_queue = NULL; u32 tx_allowed_mask = 0; u32 vif_tx_allowed_mask = 0; - const struct wfx_tx_priv *tx_priv = NULL; struct wfx_vif *wvif; int not_found; int burst; @@ -541,7 +540,6 @@ struct hif_msg *wfx_tx_queues_get(struct wfx_dev *wdev) skb = wfx_tx_queue_get(wdev, queue, tx_allowed_mask); if (!skb) continue; - tx_priv = wfx_skb_tx_priv(skb); hif = (struct hif_msg *) skb->data; wvif = wdev_to_wvif(wdev, hif->interface); WARN_ON(!wvif); -- 2.7.4 ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel