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?
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to