Re: [staging-next 02/12] staging/easycap: prefer printk over SAY in module entry functions

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

 



On Wed, Feb 09, 2011 at 09:11:49AM +0200, Winkler, Tomas wrote:
> 
> 
> > -----Original Message-----
> > From: Greg KH [mailto:gregkh@xxxxxxx]
> > Sent: Wednesday, February 09, 2011 7:11 AM
> > To: Winkler, Tomas
> > Cc: Dan Carpenter; devel@xxxxxxxxxxxxxxxxxxxxxx; Mike Thomas
> > Subject: Re: [staging-next 02/12] staging/easycap: prefer printk over SAY in
> > module entry functions
> > 
> > On Wed, Feb 09, 2011 at 07:03:22AM +0200, Winkler, Tomas wrote:
> > >
> > > > -----Original Message-----
> > > > From: Dan Carpenter [mailto:error27@xxxxxxxxx]
> > > > Sent: Wednesday, February 09, 2011 1:34 AM
> > > > To: Winkler, Tomas
> > > > Cc: gregkh@xxxxxxx; devel@xxxxxxxxxxxxxxxxxxxxxx; Mike Thomas
> > > > Subject: Re: [staging-next 02/12] staging/easycap: prefer printk over
> > SAY in
> > > > module entry functions
> > > >
> > > > On Wed, Feb 09, 2011 at 01:12:41AM +0200, Tomas Winkler wrote:
> > > > > +	printk(KERN_INFO "Easycap version: "EASYCAP_DRIVER_VERSION
> > "\n");
> > > > > +
> > > >
> > > > These should just be deleted.  The user can use `modinfo` if they care.
> > >
> > > It is very common and I think also good practices to notify user when a
> > driver is loaded.
> > 
> > No, not at all.  Do you really want to see this for EVERY driver your
> > system loads?
> > 
> > We are removing these slowly, as it makes no sense at all.  Please don't
> > add to it.  Distros patch this kind of thing away to make for faster
> > boot time and cleaner load times, don't add extra work.
> 
> Fast boot is a good reason. I will rework the patch

Just remove the lines in a later patch, I'll take this as-is because it
is a good fix.

thanks,

greg k-h
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux