wil6210: improve debugfs for VRING
authorVladimir Kondratiev <qca_vkondrat@qca.qualcomm.com>
Tue, 23 Dec 2014 07:47:08 +0000 (09:47 +0200)
committerKalle Valo <kvalo@codeaurora.org>
Thu, 15 Jan 2015 12:31:19 +0000 (14:31 +0200)
When printing VRING on debugfs (file "vrings"),
software head & tail indexes were printed in decimal format
while hardware tail in hexadecimal only.

It is not comfortable to compare indexes in different formats;
on the other hand, hexadecimal output useful to see hardware
glitches.

To serve all purposes, print hardware tail in both decimal and
hexadecimal formats.

Signed-off-by: Vladimir Kondratiev <qca_vkondrat@qca.qualcomm.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/ath/wil6210/debugfs.c

index 32df4b7..59bc494 100644 (file)
@@ -50,6 +50,7 @@ static void wil_print_vring(struct seq_file *s, struct wil6210_priv *wil,
                            char _s, char _h)
 {
        void __iomem *x = wmi_addr(wil, vring->hwtail);
+       u32 v;
 
        seq_printf(s, "VRING %s = {\n", name);
        seq_printf(s, "  pa     = %pad\n", &vring->pa);
@@ -58,10 +59,12 @@ static void wil_print_vring(struct seq_file *s, struct wil6210_priv *wil,
        seq_printf(s, "  swtail = %d\n", vring->swtail);
        seq_printf(s, "  swhead = %d\n", vring->swhead);
        seq_printf(s, "  hwtail = [0x%08x] -> ", vring->hwtail);
-       if (x)
-               seq_printf(s, "0x%08x\n", ioread32(x));
-       else
+       if (x) {
+               v = ioread32(x);
+               seq_printf(s, "0x%08x = %d\n", v, v);
+       } else {
                seq_puts(s, "???\n");
+       }
 
        if (vring->va && (vring->size < 1025)) {
                uint i;