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 <[email protected]>
---
 drivers/net/wireless/ath/wil6210/debugfs.c | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/drivers/net/wireless/ath/wil6210/debugfs.c 
b/drivers/net/wireless/ath/wil6210/debugfs.c
index 5117a02..fedfa64 100644
--- a/drivers/net/wireless/ath/wil6210/debugfs.c
+++ b/drivers/net/wireless/ath/wil6210/debugfs.c
@@ -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;
-- 
2.1.0

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to