On 01/09/2021 07:40, Loic Poulain wrote:
iw wlan0 set power_save off
I do this on wcn3680b and get no loss of signal
If I do this though
diff --git a/drivers/net/wireless/ath/wcn36xx/smd.c
b/drivers/net/wireless/ath/wcn36xx/smd.c
index 03966072f34c..ba613fbb728d 100644
--- a/drivers/net/wireless/ath/wcn36xx/smd.c
+++ b/drivers/net/wireless/ath/wcn36xx/smd.c
@@ -2345,6 +2345,8 @@ int wcn36xx_smd_feature_caps_exchange(struct
wcn36xx *wcn)
set_feat_caps(msg_body.feat_caps, DOT11AC);
set_feat_caps(msg_body.feat_caps,
ANTENNA_DIVERSITY_SELECTION);
}
+ set_feat_caps(msg_body.feat_caps, IBSS_HEARTBEAT_OFFLOAD);
+ set_feat_caps(msg_body.feat_caps, WLANACTIVE_OFFLOAD);
PREPARE_HAL_BUF(wcn->hal_buf, msg_body);
@@ -2589,7 +2591,7 @@ static int wcn36xx_smd_missed_beacon_ind(struct
wcn36xx *wcn,
struct wcn36xx_hal_missed_beacon_ind_msg *rsp = buf;
struct ieee80211_vif *vif = NULL;
struct wcn36xx_vif *tmp;
-
+wcn36xx_info("%s/%d\n", __func__, __LINE__);
/* Old FW does not have bss index */
if (wcn36xx_is_fw_version(wcn, 1, 2, 2, 24)) {
list_for_each_entry(tmp, &wcn->vif_list, list) {
@@ -2608,7 +2610,7 @@ static int wcn36xx_smd_missed_beacon_ind(struct
wcn36xx *wcn,
list_for_each_entry(tmp, &wcn->vif_list, list) {
if (tmp->bss_index == rsp->bss_index) {
- wcn36xx_dbg(WCN36XX_DBG_HAL, "beacon missed
bss_index %d\n",
+ wcn36xx_info("beacon missed bss_index %d\n",
rsp->bss_index);
vif = wcn36xx_priv_to_vif(tmp);
ieee80211_connection_loss(vif);
bingo
root@linaro-developer:~# iw wlan0 set power_save off
# pulls plug on AP
root@linaro-developer:~# [ 83.290987] wcn36xx:
wcn36xx_smd_missed_beacon_ind/2594
[ 83.291070] wcn36xx: beacon missed bss_index 0
[ 83.295403] wlan0: Connection to AP e2:63:da:9c:a4:bd lost
I'm not sure if both flags are required but, this is the behavior we want
