aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHante Meuleman <[email protected]>2016-09-19 12:09:56 +0100
committerKalle Valo <[email protected]>2016-09-26 20:35:54 +0300
commit835680b82f029818c813324aed3073cdcf63241f (patch)
treea327e409839a4cbef30431aef7d285912c14f7ef
parent8fa5fdec09cd379c9ecb8972f344f8f308e0ccf3 (diff)
brcmfmac: remove unnecessary null pointer check
in the function brcmf_bus_start() in the exception handling a check is made to dermine whether ifp is null, though this is not possible. Removing the unnessary check. Reviewed-by: Arend Van Spriel <[email protected]> Reviewed-by: Franky Lin <[email protected]> Reviewed-by: Pieter-Paul Giesberts <[email protected]> Signed-off-by: Hante Meuleman <[email protected]> Signed-off-by: Arend van Spriel <[email protected]> Signed-off-by: Kalle Valo <[email protected]>
-rw-r--r--drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c
index 3f6a58035077..9a05371453ce 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c
@@ -1048,8 +1048,7 @@ fail:
brcmf_fws_del_interface(ifp);
brcmf_fws_deinit(drvr);
}
- if (ifp)
- brcmf_net_detach(ifp->ndev, false);
+ brcmf_net_detach(ifp->ndev, false);
if (p2p_ifp)
brcmf_net_detach(p2p_ifp->ndev, false);
drvr->iflist[0] = NULL;