wcn36xx: Disable bmps when encryption is disabled
authorLoic Poulain <loic.poulain@linaro.org>
Tue, 25 Aug 2020 13:45:27 +0000 (15:45 +0200)
committerKalle Valo <kvalo@codeaurora.org>
Thu, 27 Aug 2020 10:14:54 +0000 (13:14 +0300)
commitc6522a5076e1a65877c51cfee313a74ef61cabf8
tree7d3bfb03c20dab7bcd39f67017b9577df535001d
parent1fcdb567df1b850b751887caf1dc5ab6106e1104
wcn36xx: Disable bmps when encryption is disabled

For whatever reason, when connected to an open/no-security BSS,
the wcn36xx controller in bmps mode does not forward 'wake-up'
beacons despite AP sends DTIM with station AID.

Meaning that AP is not able to wakeup the station and needs to wait
for the station to wakeup by its own (TX data, keep alive pkt...),
causing serious latency issues and unexpected deauth.

When connected to AP with encryption enabled, this issue does not occur.
So a simple workaround is to only enable bmps support in that case.

Ideally, it should be propertly fixed to allow bmps support with open
BSS, whatever the issue is at driver or firmware level.

Tested on wcn3620 and wcn3680.

Signed-off-by: Loic Poulain <loic.poulain@linaro.org>
Tested-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/1598363127-26066-1-git-send-email-loic.poulain@linaro.org
drivers/net/wireless/ath/wcn36xx/main.c
drivers/net/wireless/ath/wcn36xx/pmc.c
drivers/net/wireless/ath/wcn36xx/wcn36xx.h