staging: wfx: drop unnecessary condition checks in wfx_upload_ap_templates()

In former code, wfx_upload_ap_templates() was called in more cases
than necessary. Therefore, it tried to not update the frame templates
if it was not necessary.

Now, wfx_upload_ap_templates() is called only if mac80211 asked to
update the templates. In add, it does not hurt to upload template if
they are not used. So, remove unnecessary conditions at beginning of
wfx_upload_ap_templates()

Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Link: https://lore.kernel.org/r/20200410133239.438347-11-Jerome.Pouiller@silabs.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/staging/wfx/sta.c b/drivers/staging/wfx/sta.c
index b0557dab91fdaa2343814f927dcf482613f7cac6..7af7bfa4ac991ff0dfbfed28379881ec615a9cb8 100644 (file)
--- a/drivers/staging/wfx/sta.c
+++ b/drivers/staging/wfx/sta.c
@@ -570,11 +570,6 @@ static int wfx_upload_ap_templates(struct wfx_vif *wvif)
 {
        struct sk_buff *skb;
 
-       if (wvif->vif->type == NL80211_IFTYPE_STATION ||
-           wvif->vif->type == NL80211_IFTYPE_MONITOR ||
-           wvif->vif->type == NL80211_IFTYPE_UNSPECIFIED)
-               return 0;
-
        skb = ieee80211_beacon_get(wvif->wdev->hw, wvif->vif);
        if (!skb)
                return -ENOMEM;