On Tue, May 30, 2017 at 02:50:53PM +0200, Simon Horman wrote: > Trivial updates to improve checkpatch cleanness. > > Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx> ... > diff --git a/drivers/mmc/host/tmio_mmc_core.c b/drivers/mmc/host/tmio_mmc_core.c > index 70516abd50a1..07905c05f356 100644 > --- a/drivers/mmc/host/tmio_mmc_core.c > +++ b/drivers/mmc/host/tmio_mmc_core.c > @@ -127,16 +127,17 @@ static int tmio_mmc_next_sg(struct tmio_mmc_host *host) > > #define STATUS_TO_TEXT(a, status, i) \ > do { \ > - if (status & TMIO_STAT_##a) { \ > - if (i++) \ > - printk(" | "); \ > - printk(#a); \ > + if ((status) & TMIO_STAT_##a) { \ > + if ((i)++) \ > + printk(KERN_DEBUG, " | "); \ > + printk(KERN_DEBUG, #a); \ Ooops, 0-day tells me that there should not be a comma after KERN_DEBUG. I'll fix that in v2. ...