[PATCH ath-next] wifi: ath12k: Fix scan initiation failure handling

[Jeff Johnson]

On 5/11/2025 11:58 PM, Vignesh C wrote:
> During a code review, it is observed that in the scenario
> where scan initiation fails, the current code schedules the
> ar->scan.timeout workqueue which is not necessary.

Are there any user-visible effects?

> 
> In the scan initiation failure scenario, exit the code instead
> of scheduling the ar->scan.timeout workqueue.
> 
> Compile tested only.
> 
> Fixes: d889913205cf ("wifi: ath12k: driver for Qualcomm Wi-Fi 7 devices")

If there are no user-visible effects then I'll drop the Fixes tag since there
is no reason to backport this to stable kernel versions.

> Signed-off-by: Vignesh C <quic_vignc at quicinc.com>
> ---
>  drivers/net/wireless/ath/ath12k/mac.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/net/wireless/ath/ath12k/mac.c b/drivers/net/wireless/ath/ath12k/mac.c
> index 4dae941c9615..0cf2b3d06c3b 100644
> --- a/drivers/net/wireless/ath/ath12k/mac.c
> +++ b/drivers/net/wireless/ath/ath12k/mac.c
> @@ -4647,6 +4647,7 @@ static int ath12k_mac_op_hw_scan(struct ieee80211_hw *hw,
>  		spin_lock_bh(&ar->data_lock);
>  		ar->scan.state = ATH12K_SCAN_IDLE;
>  		spin_unlock_bh(&ar->data_lock);
> +		goto exit;
>  	}
>  
>  	ath12k_dbg(ar->ab, ATH12K_DBG_MAC, "mac scan started");
> 
> base-commit: 3a64d6def8733e75909abcfab983efae92dc4102