wil6210: change HALP logging category to IRQ

Change the logging category of HALP functions from
MISC to IRQ, since the HALP mechanism is closely
related to interrupts. Both HALP and IRQ create
a heavy load of logging messages when enabled,
so their logging is typically disabled during normal debug
scenarios. Having them in the same logging category
will make it easier to disable logging for both in one go.

Signed-off-by: Lior David <qca_liord@qca.qualcomm.com>
Signed-off-by: Maya Erez <qca_merez@qca.qualcomm.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>

diff --git a/drivers/net/wireless/ath/wil6210/interrupt.c b/drivers/net/wireless/ath/wil6210/interrupt.c
index f10c47dcbde5bcc89d3019617be8c9823f4052e5..64046e0bd0a228ae724da21aa4592a6cdd8dea4e 100644 (file)
--- a/drivers/net/wireless/ath/wil6210/interrupt.c
+++ b/drivers/net/wireless/ath/wil6210/interrupt.c
@@ -599,7 +599,7 @@ void wil6210_clear_irq(struct wil6210_priv *wil)
 
 void wil6210_set_halp(struct wil6210_priv *wil)
 {
-       wil_dbg_misc(wil, "%s()\n", __func__);
+       wil_dbg_irq(wil, "%s()\n", __func__);
 
        wil_w(wil, RGF_DMA_EP_MISC_ICR + offsetof(struct RGF_ICR, ICS),
              BIT_DMA_EP_MISC_ICR_HALP);
@@ -607,7 +607,7 @@ void wil6210_set_halp(struct wil6210_priv *wil)
 
 void wil6210_clear_halp(struct wil6210_priv *wil)
 {
-       wil_dbg_misc(wil, "%s()\n", __func__);
+       wil_dbg_irq(wil, "%s()\n", __func__);
 
        wil_w(wil, RGF_DMA_EP_MISC_ICR + offsetof(struct RGF_ICR, ICR),
              BIT_DMA_EP_MISC_ICR_HALP);

diff --git a/drivers/net/wireless/ath/wil6210/main.c b/drivers/net/wireless/ath/wil6210/main.c
index 4240e81bcc82962d0b34ff30ce6fb79a98256339..1205d76331e589dd2d504b5b2631ee040077b4ce 100644 (file)
--- a/drivers/net/wireless/ath/wil6210/main.c
+++ b/drivers/net/wireless/ath/wil6210/main.c
@@ -1118,8 +1118,8 @@ void wil_halp_vote(struct wil6210_priv *wil)
 
        mutex_lock(&wil->halp.lock);
 
-       wil_dbg_misc(wil, "%s: start, HALP ref_cnt (%d)\n", __func__,
-                    wil->halp.ref_cnt);
+       wil_dbg_irq(wil, "%s: start, HALP ref_cnt (%d)\n", __func__,
+                   wil->halp.ref_cnt);
 
        if (++wil->halp.ref_cnt == 1) {
                wil6210_set_halp(wil);
@@ -1129,15 +1129,15 @@ void wil_halp_vote(struct wil6210_priv *wil)
                        /* Mask HALP as done in case the interrupt is raised */
                        wil6210_mask_halp(wil);
                } else {
-                       wil_dbg_misc(wil,
-                                    "%s: HALP vote completed after %d ms\n",
-                                    __func__,
-                                    jiffies_to_msecs(to_jiffies - rc));
+                       wil_dbg_irq(wil,
+                                   "%s: HALP vote completed after %d ms\n",
+                                   __func__,
+                                   jiffies_to_msecs(to_jiffies - rc));
                }
        }
 
-       wil_dbg_misc(wil, "%s: end, HALP ref_cnt (%d)\n", __func__,
-                    wil->halp.ref_cnt);
+       wil_dbg_irq(wil, "%s: end, HALP ref_cnt (%d)\n", __func__,
+                   wil->halp.ref_cnt);
 
        mutex_unlock(&wil->halp.lock);
 }
@@ -1148,16 +1148,16 @@ void wil_halp_unvote(struct wil6210_priv *wil)
 
        mutex_lock(&wil->halp.lock);
 
-       wil_dbg_misc(wil, "%s: start, HALP ref_cnt (%d)\n", __func__,
-                    wil->halp.ref_cnt);
+       wil_dbg_irq(wil, "%s: start, HALP ref_cnt (%d)\n", __func__,
+                   wil->halp.ref_cnt);
 
        if (--wil->halp.ref_cnt == 0) {
                wil6210_clear_halp(wil);
-               wil_dbg_misc(wil, "%s: HALP unvote\n", __func__);
+               wil_dbg_irq(wil, "%s: HALP unvote\n", __func__);
        }
 
-       wil_dbg_misc(wil, "%s: end, HALP ref_cnt (%d)\n", __func__,
-                    wil->halp.ref_cnt);
+       wil_dbg_irq(wil, "%s: end, HALP ref_cnt (%d)\n", __func__,
+                   wil->halp.ref_cnt);
 
        mutex_unlock(&wil->halp.lock);
 }