Re: wifi: ath10k: avoid unnecessary wait for service ready message

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

On 10/16/2025 4:10 AM, Andreas Tobler wrote:
> Dear all,
> 
> this commit (Upstream commit 51a73f1b2e56b0324b4a3bb8cebc4221b5be4c7) makes our WLE600
> Compex wifi cards (qca988x based) unusable. Reverting the commit brings the wifi card back.
> 
> This was discovered on the v6.12.53 from today.
> 
> ath10k messages excerpt:
> --------------
> Oct 15 22:00:13 klog: ath10k_pci 0000:05:00.0: pci irq msi oper_irq_mode 2 irq_mode 0
> reset_mode 0
> Oct 15 22:00:13 klog: ath10k_pci 0000:05:00.0: qca988x hw2.0 target 0x4100016c chip_id
> 0x043222ff sub 0000:0000
> Oct 15 22:00:13 klog: ath10k_pci 0000:05:00.0: kconfig debug 0 debugfs 0 tracing 0 dfs 1
> testmode 0
> Oct 15 22:00:13 klog: ath10k_pci 0000:05:00.0: firmware ver 10.2.4-1.0-00047 api 5
> features no-p2p,raw-mode,mfp,allows-mesh-bcast crc32 35bd9258
> Oct 15 22:00:13 klog: ath10k_pci 0000:05:00.0: board_file api 1 bmi_id N/A crc32 bebc7c08
> Oct 15 22:00:20 klog: ath10k_pci 0000:05:00.0: wmi unified ready event not received
> Oct 15 22:00:21 klog: ath10k_pci 0000:05:00.0: could not init core (-110)
> Oct 15 22:00:21 klog: ath10k_pci 0000:05:00.0: could not probe fw (-110)
> --------------
> 
> Beside reverting, how can we help fixing this?

Thank you Andreas for the report.

If this 100% repro?

Can you try just ignore the err and see what we get? I am suspecting we are missing the
interrupt for unified ready message.

diff --git a/drivers/net/wireless/ath/ath10k/core.c b/drivers/net/wireless/ath/ath10k/core.c
index 6f78f1752cd6..cf4cc44d0a1e 100644
--- a/drivers/net/wireless/ath/ath10k/core.c
+++ b/drivers/net/wireless/ath/ath10k/core.c
@@ -3217,7 +3217,6 @@ int ath10k_core_start(struct ath10k *ar, enum ath10k_firmware_mode mode,
        status = ath10k_wmi_wait_for_unified_ready(ar);
        if (status) {
                ath10k_err(ar, "wmi unified ready event not received\n");
-               goto err_hif_stop;
        }

        status = ath10k_core_compat_services(ar);


Anyway, please help collect verbose ath10k log, you may enable verbose log by

	modprobe ath10k_core debug_mask=0xffffffff
	modprobe ath10k_pci

> Thanks & regards,
> Andreas