On Mon, 2017-11-27 at 13:15 +0000, Colin King wrote: > drivers/atm/eni.c Aren't all those ++ variables unused? They seem to be emitted in an #if 0 printk(...) #endif Maybe these should be removed or changed to something like: #if 0 #define DEBUG_INCR(var) do { (var)++ } while (0) #else #define DEBUG_INCR(var) do {} while (0) #endif ... DEBUG_INCR(rx_enqueued); -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html