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

Reply via email to