On Fri, Nov 20, 2015 at 03:14:19PM +0000, Marciniszyn, Mike wrote:
> > > + dd_dev_info(dd, "%s: program XMT margin, CcePcieCtrl 0x%llx\n",
> > > +             fname, pcie_ctrl);
> > 
> > Why spam the log with this all the time?  Shouldn't this be a debug line
> > instead?
> 
> The current implementation is layered on dev_info().
> 
> Are you saying dev_dbg() is a better choice?

Yes, why would a user ever want to see that above line?  What can they
do with it?
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to