Re: [PATCH ath-next] wifi: ath12k: Set up MLO after SSR

[Baochen Qiang <baochen.qiang@oss.qualcomm.com>]

On 2/27/2026 12:11 PM, Roopni Devanathan wrote:
> From: Ramya Gnanasekar <ramya.gnanasekar@oss.qualcomm.com>
> 
> During recovery of an MLO setup from a core reset,
> ATH12K_GROUP_FLAG_REGISTERED is set because ath12k_mac_unregister is not
> called during core reset. So, when an MLO setup is recovering from a core
> reset, ath12k_core_mlo_setup() is skipped. Hence, the firmware will not
> have information about partner links. This makes MLO association fail
> after recovery.
> 
> To resolve this, call ath12k_core_mlo_setup() during recovery, to set up
> MLO. Also, if MLO setup fails during recovery, call
> ath12k_mac_unregister() and ath12k_mac_destroy() to unregister mac and
> then tear down the mac structures.
> 
> Also, initiate MLO teardown in the hardware group stop sequence to align
> with the hardware group start sequence.
> 
> Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.6-01181-QCAHKSWPL_SILICONZ-1
> 
> Signed-off-by: Ramya Gnanasekar <ramya.gnanasekar@oss.qualcomm.com>
> Signed-off-by: Roopni Devanathan <roopni.devanathan@oss.qualcomm.com>
> ---
>  drivers/net/wireless/ath/ath12k/core.c | 10 +++++++++-
>  1 file changed, 9 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/net/wireless/ath/ath12k/core.c b/drivers/net/wireless/ath/ath12k/core.c
> index 9dca1a0af73e..218c0a0c9699 100644
> --- a/drivers/net/wireless/ath/ath12k/core.c
> +++ b/drivers/net/wireless/ath/ath12k/core.c
> @@ -1017,6 +1017,8 @@ static void ath12k_core_hw_group_stop(struct ath12k_hw_group *ag)
>  
>  	ath12k_mac_unregister(ag);
>  
> +	ath12k_mac_mlo_teardown(ag);
> +
>  	for (i = ag->num_devices - 1; i >= 0; i--) {
>  		ab = ag->ab[i];
>  		if (!ab)
> @@ -1134,8 +1136,14 @@ static int ath12k_core_hw_group_start(struct ath12k_hw_group *ag)
>  
>  	lockdep_assert_held(&ag->mutex);
>  
> -	if (test_bit(ATH12K_GROUP_FLAG_REGISTERED, &ag->flags))
> +	if (test_bit(ATH12K_GROUP_FLAG_REGISTERED, &ag->flags)) {
> +		ret = ath12k_core_mlo_setup(ag);
> +		if (WARN_ON(ret)) {
> +			ath12k_mac_unregister(ag);
> +			goto err_mac_destroy;
> +		}
>  		goto core_pdev_create;
> +	}
>  
>  	ret = ath12k_mac_allocate(ag);
>  	if (WARN_ON(ret))
> 
> base-commit: 20ad0d58517073b3b683ff786c65dd3142321707

Reviewed-by: Baochen Qiang <baochen.qiang@oss.qualcomm.com>