[PATCH v2] wifi: ath12k: Set default beacon mode to burst mode

[Jeff Johnson]

On 9/5/2023 2:39 PM, Sidhanta Sahu wrote:
> Currently, firmware does not like when beacon mode is set as staggered
> mode for more than one beaconing vifs. Beacon mode for multiple
> beaconing (transmitted) vifs are expected to be in burst mode. So set
> beacon mode to burst mode by default.
> 
> Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.0.1-00029-QCAHKSWPL_SILICONZ-1
> 
> Signed-off-by: Sidhanta Sahu <quic_sidhanta at quicinc.com>

Acked-by: Jeff Johnson <quic_jjohnson at quicinc.com>

> ---
> v2: Modified the debug log.
> 
>   drivers/net/wireless/ath/ath12k/mac.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/wireless/ath/ath12k/mac.c b/drivers/net/wireless/ath/ath12k/mac.c
> index 1bb9802ef569..d38b85eaed37 100644
> --- a/drivers/net/wireless/ath/ath12k/mac.c
> +++ b/drivers/net/wireless/ath/ath12k/mac.c
> @@ -2314,7 +2314,7 @@ static void ath12k_mac_op_bss_info_changed(struct ieee80211_hw *hw,
>   
>   	if (changed & BSS_CHANGED_BEACON) {
>   		param_id = WMI_PDEV_PARAM_BEACON_TX_MODE;
> -		param_value = WMI_BEACON_STAGGERED_MODE;
> +		param_value = WMI_BEACON_BURST_MODE;
>   		ret = ath12k_wmi_pdev_set_param(ar, param_id,
>   						param_value, ar->pdev->pdev_id);
>   		if (ret)
> @@ -2322,7 +2322,7 @@ static void ath12k_mac_op_bss_info_changed(struct ieee80211_hw *hw,
>   				    arvif->vdev_id);
>   		else
>   			ath12k_dbg(ar->ab, ATH12K_DBG_MAC,
> -				   "Set staggered beacon mode for VDEV: %d\n",
> +				   "Set burst beacon mode for VDEV: %d\n",
>   				   arvif->vdev_id);
>   
>   		ret = ath12k_mac_setup_bcn_tmpl(arvif);