From: Jérôme Pouiller Date: Mon, 20 Apr 2020 16:03:08 +0000 (+0200) Subject: staging: wfx: drop useless checks in wfx_do_unjoin() X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=06efad5808aa0db3528eca4b8faee4013ffd787b;p=linux.git staging: wfx: drop useless checks in wfx_do_unjoin() The callers of wfx_do_unjoin() already take care of vif state. Therefore, it is not necessary to take care of the status of the interface. Signed-off-by: Jérôme Pouiller Link: https://lore.kernel.org/r/20200420160311.57323-14-Jerome.Pouiller@silabs.com Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/wfx/sta.c b/drivers/staging/wfx/sta.c index 56cb6fff4a06a..bc891b6e4392c 100644 --- a/drivers/staging/wfx/sta.c +++ b/drivers/staging/wfx/sta.c @@ -286,12 +286,6 @@ void wfx_set_default_unicast_key(struct ieee80211_hw *hw, // Call it with wdev->conf_mutex locked static void wfx_do_unjoin(struct wfx_vif *wvif) { - if (!wvif->state) - return; - - if (wvif->state == WFX_STATE_AP) - return; - wvif->state = WFX_STATE_PASSIVE; /* Unjoin is a reset. */