[PATCH v2] staging: wilc1000: fix static checker warning to unlock mutex in wilc_deinit()

[Ajay Singh]

Fix for static checker warning inconsistent returns of
'hif_deinit_lock'(more details [1]).

"drivers/staging/wilc1000/host_interface.c:3390 wilc_deinit()
              warn: inconsistent returns 'hif_deinit_lock'."

Fixes: ff52a57a7a42 ("staging: wilc1000: move the allocation of cmd out
of wilc_enqueue_cmd()")

[1]. https://www.spinics.net/lists/linux-driver-devel/msg114216.html

Reported-by: Dan Carpenter <dan.carpen...@oracle.com>
Signed-off-by: Ajay Singh <ajay.kat...@microchip.com>
---

Change since v1:
 - added 'Fixes' tag in commit description.

 drivers/staging/wilc1000/host_interface.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/staging/wilc1000/host_interface.c 
b/drivers/staging/wilc1000/host_interface.c
index 52c0c10..918d06e 100644
--- a/drivers/staging/wilc1000/host_interface.c
+++ b/drivers/staging/wilc1000/host_interface.c
@@ -3370,15 +3370,15 @@ int wilc_deinit(struct wilc_vif *vif)
                struct host_if_msg *msg;
 
                msg = wilc_alloc_work(vif, handle_hif_exit_work, true);
-               if (IS_ERR(msg))
-                       return PTR_ERR(msg);
-
-               result = wilc_enqueue_work(msg);
-               if (result)
-                       netdev_err(vif->ndev, "deinit : Error(%d)\n", result);
-               else
-                       wait_for_completion(&msg->work_comp);
-               kfree(msg);
+               if (!IS_ERR(msg)) {
+                       result = wilc_enqueue_work(msg);
+                       if (result)
+                               netdev_err(vif->ndev, "deinit : Error(%d)\n",
+                                          result);
+                       else
+                               wait_for_completion(&msg->work_comp);
+                       kfree(msg);
+               }
                destroy_workqueue(hif_workqueue);
        }
 
-- 
2.7.4