wifi: ath12k: fix debug messages
authorKalle Valo <quic_kvalo@quicinc.com>
Tue, 3 Oct 2023 15:01:32 +0000 (18:01 +0300)
committerKalle Valo <quic_kvalo@quicinc.com>
Thu, 5 Oct 2023 06:24:14 +0000 (09:24 +0300)
In ath12k the debug messages were broken, no matter setting what value to the
debug_mask module parameter would not get the debug messages printed. The issue
is that __ath12k_dbg() uses dev_dbg() to print the debug messages which requires either enabling
CONFIG_DYNAMIC_DEBUG or DEBUG symbol in the driver.

ath12k is supposed to use debug_mask module to control whether debug messages
are printed or not. Using both CONFIG_DYNAMIC_DEBUG and debug_mask parameter
does not make any sense so switch to using dev_printk(), just like ath11k does.
Now it's enough just to debug_mask module parameter to get the debug messages.

Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.0.1-00029-QCAHKSWPL_SILICONZ-1

Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://lore.kernel.org/r/20231003150132.187875-1-kvalo@kernel.org
drivers/net/wireless/ath/ath12k/debug.c

index 67893923e010929a0072575f050ddea3420c989a..45d33279e665deffe018dbaa889f6e1ed2ee1cf0 100644 (file)
@@ -64,7 +64,7 @@ void __ath12k_dbg(struct ath12k_base *ab, enum ath12k_debug_mask mask,
        vaf.va = &args;
 
        if (ath12k_debug_mask & mask)
-               dev_dbg(ab->dev, "%pV", &vaf);
+               dev_printk(KERN_DEBUG, ab->dev, "%pV", &vaf);
 
        /* TODO: trace log */