On Friday, October 14, 2016 2:45:49 PM CEST Mauro Carvalho Chehab wrote: > > -#define dprintk if (debug) printk > +#define dprintk(fmt, arg...) do { \ > + if (debug) \ > + printk(KERN_DEBUG pr_fmt("%s: " fmt), \ > + __func__, ##arg); \ > +} while (0) > Why not just use pr_debug() or dev_dbg() here? They already have a way to control output at runtime (CONFIG_DYNAMIC_DEBUG). Arnd