Commit 1646ff6c authored by Dafna Hirschfeld's avatar Dafna Hirschfeld Committed by Greg Kroah-Hartman

staging: wilc1000: remove 'if' on field address

Remove 'if' statements testing struct's field address.
Since such statements always return true, they are redundant.
Signed-off-by: default avatarDafna Hirschfeld <dafna3@gmail.com>
Acked-by: default avatarJulia Lawall <julia.lawall@lip6.fr>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent e69a0500
...@@ -628,7 +628,6 @@ void wilc1000_wlan_deinit(struct net_device *dev) ...@@ -628,7 +628,6 @@ void wilc1000_wlan_deinit(struct net_device *dev)
wl->hif_func->disable_interrupt(wl); wl->hif_func->disable_interrupt(wl);
mutex_unlock(&wl->hif_cs); mutex_unlock(&wl->hif_cs);
} }
if (&wl->txq_event)
complete(&wl->txq_event); complete(&wl->txq_event);
wlan_deinitialize_threads(dev); wlan_deinitialize_threads(dev);
...@@ -677,10 +676,7 @@ static int wlan_deinit_locks(struct net_device *dev) ...@@ -677,10 +676,7 @@ static int wlan_deinit_locks(struct net_device *dev)
vif = netdev_priv(dev); vif = netdev_priv(dev);
wilc = vif->wilc; wilc = vif->wilc;
if (&wilc->hif_cs)
mutex_destroy(&wilc->hif_cs); mutex_destroy(&wilc->hif_cs);
if (&wilc->rxq_cs)
mutex_destroy(&wilc->rxq_cs); mutex_destroy(&wilc->rxq_cs);
return 0; return 0;
...@@ -716,7 +712,6 @@ static void wlan_deinitialize_threads(struct net_device *dev) ...@@ -716,7 +712,6 @@ static void wlan_deinitialize_threads(struct net_device *dev)
wl->close = 1; wl->close = 1;
if (&wl->txq_event)
complete(&wl->txq_event); complete(&wl->txq_event);
if (wl->txq_thread) { if (wl->txq_thread) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment