[PATCH 02/15] staging: wilc1000: remove conditional lock in wilc_wfi_deinit_mon_interface()

[Ajay.Kathat]

From: Ajay Singh <ajay.kat...@microchip.com>

wilc_wfi_deinit_mon_interface() calls unregister_netdev() which
requires the rtnl lock again. Now move wilc_wfi_deinit_mon_interface()
out of wilc_mac_close(). Also remove explicit call to wilc_mac_close()
because unregister_netdev() takes care of calling wilc_mac_close().

Signed-off-by: Ajay Singh <ajay.kat...@microchip.com>
---
 drivers/staging/wilc1000/linux_mon.c  | 19 +++++--------------
 drivers/staging/wilc1000/linux_wlan.c | 11 +++--------
 2 files changed, 8 insertions(+), 22 deletions(-)

diff --git a/drivers/staging/wilc1000/linux_mon.c 
b/drivers/staging/wilc1000/linux_mon.c
index ed06834..32d0c81 100644
--- a/drivers/staging/wilc1000/linux_mon.c
+++ b/drivers/staging/wilc1000/linux_mon.c
@@ -253,19 +253,10 @@ struct net_device *wilc_wfi_init_mon_interface(struct 
wilc *wl,
 
 void wilc_wfi_deinit_mon_interface(struct wilc *wl)
 {
-       bool rollback_lock = false;
-
-       if (wl->monitor_dev) {
-               if (rtnl_is_locked()) {
-                       rtnl_unlock();
-                       rollback_lock = true;
-               }
-               unregister_netdev(wl->monitor_dev);
+       if (!wl->monitor_dev)
+               return;
 
-               if (rollback_lock) {
-                       rtnl_lock();
-                       rollback_lock = false;
-               }
-               wl->monitor_dev = NULL;
-       }
+       unregister_netdev(wl->monitor_dev);
+       free_netdev(wl->monitor_dev);
+       wl->monitor_dev = NULL;
 }
diff --git a/drivers/staging/wilc1000/linux_wlan.c 
b/drivers/staging/wilc1000/linux_wlan.c
index 1362d8f..b0249d2 100644
--- a/drivers/staging/wilc1000/linux_wlan.c
+++ b/drivers/staging/wilc1000/linux_wlan.c
@@ -920,7 +920,6 @@ static int wilc_mac_close(struct net_device *ndev)
                netdev_dbg(ndev, "Deinitializing wilc1000\n");
                wl->close = 1;
                wilc_wlan_deinitialize(ndev);
-               wilc_wfi_deinit_mon_interface(wl);
        }
 
        vif->mac_opened = 0;
@@ -1006,19 +1005,15 @@ void wilc_netdev_cleanup(struct wilc *wilc)
                wilc->firmware = NULL;
        }
 
-       if (wilc->vif[0]->ndev || wilc->vif[1]->ndev) {
-               for (i = 0; i < WILC_NUM_CONCURRENT_IFC; i++)
-                       if (wilc->vif[i]->ndev)
-                               if (wilc->vif[i]->mac_opened)
-                                       wilc_mac_close(wilc->vif[i]->ndev);
-
-               for (i = 0; i < WILC_NUM_CONCURRENT_IFC; i++) {
+       for (i = 0; i < WILC_NUM_CONCURRENT_IFC; i++) {
+               if (wilc->vif[i] && wilc->vif[i]->ndev) {
                        unregister_netdev(wilc->vif[i]->ndev);
                        wilc_free_wiphy(wilc->vif[i]->ndev);
                        free_netdev(wilc->vif[i]->ndev);
                }
        }
 
+       wilc_wfi_deinit_mon_interface(wilc);
        flush_workqueue(wilc->hif_workqueue);
        destroy_workqueue(wilc->hif_workqueue);
        wilc_wlan_cfg_deinit(wilc);
-- 
2.7.4

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel