wcn36xx: Advertise beacon filtering support in bmps
authorLoic Poulain <loic.poulain@linaro.org>
Thu, 18 Jun 2020 09:17:43 +0000 (11:17 +0200)
committerKalle Valo <kvalo@codeaurora.org>
Thu, 1 Oct 2020 19:34:04 +0000 (22:34 +0300)
In bmps mode, beacons are filtered, and firmware is in charge
of monitoring the beacons and report changes or loss.

mac80211 must be advertised about such change to prevent it's
internal timer based beacon monitor to report beacon loss.

Fix that by setting/clearing the IEEE80211_VIF_BEACON_FILTER
vif flag on bmps entry/exit.

Signed-off-by: Loic Poulain <loic.poulain@linaro.org>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/1592471863-31402-2-git-send-email-loic.poulain@linaro.org
drivers/net/wireless/ath/wcn36xx/pmc.c

index 8441031..2d0780f 100644 (file)
@@ -31,6 +31,7 @@ int wcn36xx_pmc_enter_bmps_state(struct wcn36xx *wcn,
        if (!ret) {
                wcn36xx_dbg(WCN36XX_DBG_PMC, "Entered BMPS\n");
                vif_priv->pw_state = WCN36XX_BMPS;
+               vif->driver_flags |= IEEE80211_VIF_BEACON_FILTER;
        } else {
                /*
                 * One of the reasons why HW will not enter BMPS is because
@@ -55,6 +56,7 @@ int wcn36xx_pmc_exit_bmps_state(struct wcn36xx *wcn,
        }
        wcn36xx_smd_exit_bmps(wcn, vif);
        vif_priv->pw_state = WCN36XX_FULL_POWER;
+       vif->driver_flags &= ~IEEE80211_VIF_BEACON_FILTER;
        return 0;
 }