On Fri, 2012-11-23 at 12:09 +0530, Saravana wrote: > > Also, as you're adding new statistics which are only used when debugfs > > is enabled, you might want to wrap them in #ifdefs so that they're > > only compiled when debugfs support is compiled. [...] > Hence we don't need to wrap this up in #ifdef as it is already taken > care during the directory creation. I think Julian is arguing that you should have the *counters* themselves be under ifdef, and I tend to agree. johannes -- 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