2015-01-16 14:09:37

by Michal Kazior

[permalink] [raw]
Subject: [PATCH] ath10k: fix dtim period with beacon templates

Firmware supporting beacon templates (i.e. wmi-tlv
for qca6174) doesn't implicitly take dtim period
from the template. Instead it requires vdev param
to be set accordingly.

This fixes dtim period being stuck at 3.

Signed-off-by: Michal Kazior <[email protected]>
---
drivers/net/wireless/ath/ath10k/mac.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/ath/ath10k/mac.c b/drivers/net/wireless/ath/ath10k/mac.c
index 9524bc5..99af537 100644
--- a/drivers/net/wireless/ath/ath10k/mac.c
+++ b/drivers/net/wireless/ath/ath10k/mac.c
@@ -3435,7 +3435,7 @@ static void ath10k_bss_info_changed(struct ieee80211_hw *hw,
arvif->vdev_id, ret);
}

- if (changed & BSS_CHANGED_BEACON_INFO) {
+ if (changed & (BSS_CHANGED_BEACON_INFO | BSS_CHANGED_BEACON)) {
arvif->dtim_period = info->dtim_period;

ath10k_dbg(ar, ATH10K_DBG_MAC,
--
1.8.5.3



2015-01-27 14:20:29

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH] ath10k: fix dtim period with beacon templates

Michal Kazior <[email protected]> writes:

> Firmware supporting beacon templates (i.e. wmi-tlv
> for qca6174) doesn't implicitly take dtim period
> from the template. Instead it requires vdev param
> to be set accordingly.
>
> This fixes dtim period being stuck at 3.
>
> Signed-off-by: Michal Kazior <[email protected]>

Thanks, applied to ath.git.

--
Kalle Valo