* [PATCH] ipw2x00: make ipw_setup_deferred_work() void
@ 2020-04-14 12:02 Jason Yan
2020-04-15 8:50 ` Kalle Valo
0 siblings, 1 reply; 2+ messages in thread
From: Jason Yan @ 2020-04-14 12:02 UTC (permalink / raw)
To: stas.yakovlev, kvalo, davem, linux-wireless, netdev, linux-kernel
Cc: Jason Yan
This function actually needs no return value. So remove the unneeded
variable 'ret' and make it void.
This also fixes the following coccicheck warning:
drivers/net/wireless/intel/ipw2x00/ipw2200.c:10648:5-8: Unneeded
variable: "ret". Return "0" on line 10684
Signed-off-by: Jason Yan <yanaijie@huawei.com>
---
drivers/net/wireless/intel/ipw2x00/ipw2200.c | 12 ++----------
1 file changed, 2 insertions(+), 10 deletions(-)
diff --git a/drivers/net/wireless/intel/ipw2x00/ipw2200.c b/drivers/net/wireless/intel/ipw2x00/ipw2200.c
index 201a1eb0e2f6..923be3781c92 100644
--- a/drivers/net/wireless/intel/ipw2x00/ipw2200.c
+++ b/drivers/net/wireless/intel/ipw2x00/ipw2200.c
@@ -10640,10 +10640,8 @@ static void ipw_bg_link_down(struct work_struct *work)
mutex_unlock(&priv->mutex);
}
-static int ipw_setup_deferred_work(struct ipw_priv *priv)
+static void ipw_setup_deferred_work(struct ipw_priv *priv)
{
- int ret = 0;
-
init_waitqueue_head(&priv->wait_command_queue);
init_waitqueue_head(&priv->wait_state);
@@ -10677,8 +10675,6 @@ static int ipw_setup_deferred_work(struct ipw_priv *priv)
tasklet_init(&priv->irq_tasklet,
ipw_irq_tasklet, (unsigned long)priv);
-
- return ret;
}
static void shim__set_security(struct net_device *dev,
@@ -11659,11 +11655,7 @@ static int ipw_pci_probe(struct pci_dev *pdev,
IPW_DEBUG_INFO("pci_resource_len = 0x%08x\n", length);
IPW_DEBUG_INFO("pci_resource_base = %p\n", base);
- err = ipw_setup_deferred_work(priv);
- if (err) {
- IPW_ERROR("Unable to setup deferred work\n");
- goto out_iounmap;
- }
+ ipw_setup_deferred_work(priv);
ipw_sw_reset(priv, 1);
--
2.21.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] ipw2x00: make ipw_setup_deferred_work() void
2020-04-14 12:02 [PATCH] ipw2x00: make ipw_setup_deferred_work() void Jason Yan
@ 2020-04-15 8:50 ` Kalle Valo
0 siblings, 0 replies; 2+ messages in thread
From: Kalle Valo @ 2020-04-15 8:50 UTC (permalink / raw)
To: Jason Yan
Cc: stas.yakovlev, davem, linux-wireless, netdev, linux-kernel, Jason Yan
Jason Yan <yanaijie@huawei.com> wrote:
> This function actually needs no return value. So remove the unneeded
> variable 'ret' and make it void.
>
> This also fixes the following coccicheck warning:
>
> drivers/net/wireless/intel/ipw2x00/ipw2200.c:10648:5-8: Unneeded
> variable: "ret". Return "0" on line 10684
>
> Signed-off-by: Jason Yan <yanaijie@huawei.com>
Patch applied to wireless-drivers-next.git, thanks.
5a652b49b41b ipw2x00: make ipw_setup_deferred_work() void
--
https://patchwork.kernel.org/patch/11487287/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-04-15 8:50 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-14 12:02 [PATCH] ipw2x00: make ipw_setup_deferred_work() void Jason Yan
2020-04-15 8:50 ` Kalle Valo
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.