RE: [PATCH] omap3: isp: fix compiler warning

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----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


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux