On 04/03/2012 02:02 PM, Joe Perches wrote:
On Tue, 2012-04-03 at 13:22 -0700, greearb@xxxxxxxxxxxxxxx wrote:
From: Ben Greear<greearb@xxxxxxxxxxxxxxx>
Seems like a useful stat to report (when debugfs
is enabled).
[]
diff --git a/drivers/net/wireless/ath/ath9k/debug.c b/drivers/net/wireless/ath/ath9k/debug.c
[]
@@ -385,57 +385,61 @@ static ssize_t read_file_interrupt(struct file *file, char __user *user_buf,
size_t count, loff_t *ppos)
{
struct ath_softc *sc = file->private_data;
- char buf[512];
+ char buf[800];
800 seems excessive.
Maybe it's better to kmalloc this.
Ok.
unsigned int len = 0;
if (sc->sc_ah->caps.hw_caps& ATH9K_HW_CAP_EDMA) {
len += snprintf(buf + len, sizeof(buf) - len,
- "%8s: %10u\n", "RXLP", sc->debug.stats.istats.rxlp);
+ "%11s: %10u\n", "RXLP", sc->debug.stats.istats.rxlp);
why change the size from 8 to 11?
To make the columns line up properly.
+#define ATH_INCR_IRQ_STAT_COMMON(c, s) \
+ do { \
+ struct ath_softc *sc = (struct ath_softc *) (c)->priv; \
Isn't priv void *?
I think you don't need the cast.
Dunno..just copied existing code. I'll see if it compiles w/out
the explicit cast.
+ sc->debug.stats.istats.s++; \
+ } while (0);
sb: "while (0)" without the ;
Ok.
Thanks,
Ben
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.com
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html