[PATCH 16/16] pcmcia: use dev_dbg and dev_print in pd6729.c

Wolfram Sang w.sang at pengutronix.de
Sat Oct 24 18:50:00 EDT 2009


> > I'd also include dev_err, dev_warn and dev_info :)
> 
> What's their advantage?

They are a bit shorter and much more common AFAICT.

> 
> > >  			if (csc & I365_CSC_DETECT) {
> > >  				events |= SS_DETECT;
> > > -				dprintk("Card detected in socket %i!\n", i);
> > > +				dev_dbg(&socket[i].socket.dev,
> > > +					"Card detected in socket %i!\n", i);
> > 
> > Dunno, in days of dynamic_printk it might be better to skip debug outputs 
> > in ISRs, no?
> 
> Why?

They may not be compiled out although they are not used. Dunno, maybe I'm a bit
biased because I often fight latencies ;)

Regards,

   Wolfram

-- 
Pengutronix e.K.                           | Wolfram Sang                |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-pcmcia/attachments/20091025/298cc771/attachment.sig>


More information about the linux-pcmcia mailing list