> -----Original Message----- > From: Laurent Pinchart [mailto:laurent.pinchart@xxxxxxxxxxxxxxxx] > Sent: Monday, May 23, 2011 12:56 AM > To: Mauro Carvalho Chehab > Cc: Premi, Sanjeev; linux-media@xxxxxxxxxxxxxxx > Subject: Re: [PATCH] omap3: isp: fix compiler warning > > Hi Mauro and Sanjeev, > > On Saturday 21 May 2011 12:55:32 Mauro Carvalho Chehab wrote: > > Em 18-05-2011 13:06, Sanjeev Premi escreveu: > > > This patch fixes this compiler warning: > > > drivers/media/video/omap3isp/isp.c: In function 'isp_isr_dbg': > > > drivers/media/video/omap3isp/isp.c:392:2: warning: zero-length > > > > > > gnu_printf format string > > > > > > Since printk() is used in next few statements, same was used > > > here as well. > > > > > > Signed-off-by: Sanjeev Premi <premi@xxxxxx> > > > Cc: laurent.pinchart@xxxxxxxxxxxxxxxx > > > --- > > > > > > Actually full block can be converted to dev_dbg() > > > as well; but i am not sure about original intent > > > of the mix. > > > > > > Based on comments, i can resubmit with all prints > > > converted to dev_dbg. > > > > It is probably better to convert the full block to dev_dbg. > > You can't insert a KERN_CONT with dev_dbg(). [sp] I did realize that hence changed only the call to dev_dbg. > > > > drivers/media/video/omap3isp/isp.c | 2 +- > > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > > > diff --git a/drivers/media/video/omap3isp/isp.c > > > b/drivers/media/video/omap3isp/isp.c index 503bd79..1d38d96 100644 > > > --- a/drivers/media/video/omap3isp/isp.c > > > +++ b/drivers/media/video/omap3isp/isp.c > > > @@ -387,7 +387,7 @@ static inline void isp_isr_dbg(struct > isp_device > > > *isp, u32 irqstatus) > > > > > > }; > > > int i; > > > > > > - dev_dbg(isp->dev, ""); > > > + printk(KERN_DEBUG "%s:\n", dev_driver_string(isp->dev)); > > The original code doesn't include any \n. Is there a > particular reason why you > want to add one ? [sp] Sorry, that's a mistake out of habit. Another way to fix warning would be to make the string meaningful: - dev_dbg(isp->dev, ""); + dev_dbg (isp->dev, "ISP_IRQ:"); Is this better? ~sanjeev > > > > for (i = 0; i < ARRAY_SIZE(name); i++) { > > > > > > if ((1 << i) & irqstatus) > > -- > Regards, > > Laurent Pinchart > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html