lkml.org 
[lkml]   [2011]   [Sep]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] Staging: dt3155v4l: Convert printk's to pr_<level>
On Tue, Sep 06, 2011 at 07:15:38PM -0500, H Hartley Sweeten wrote:
> On Tuesday, September 06, 2011 4:53 PM, Greg KH wrote:
> > On Thu, Sep 01, 2011 at 04:53:55PM -0700, H Hartley Sweeten wrote:
> >> Convert all the printk(KERN_<level>) messages in the driver to pr_<level>()
> >> and use pr_fmt to provide the module name.
> >
> > What's wrong with using dev_* instead of pr_*? Lots of these should be
> > converted to that format instead, right?
>
> Nothing... dev_* should be used when possible.
>
> Actually all of the messages look like they are just plain noise and should
> be removed. Especially the ones in dt3155_{init,exit}_module. A lot of the
> ones in dt3155_probe look like noise also.
>
> The ones in {read,write,wait}_i2c_reg just look messy. Those functions also
> don't have a device pointer to use the dev_* functions.

Really? It should have an i2c device somewhere.

> The ones in dt3155_irq_handler seem troublesome. Isn't it a bad idea to output
> a kernel message in an interrupt handler?

Yes, don't do that.

> dt3155_{open,close} also look like noise. Same with dt3155_init_board.
>
> Maybe I should just submit a patch removing all of them?

Please do :)

thanks,

greg k-h


\
 
 \ /
  Last update: 2011-09-07 18:29    [W:1.485 / U:0.356 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site