[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