On Friday, September 23, 2011 20:40:25 Marek Vasut wrote:
> On Saturday, September 24, 2011 01:04:51 AM Mike Frysinger wrote:
> > On Friday, September 23, 2011 17:52:35 Marek Vasut wrote:
> > > > On Friday, September 23, 2011 15:09:52 Marek Vasut wrote:
> > > > > --- a/include/common.h
> > > > > +++ b/include/common.h
> > > > > 
> > > > > -#define debugX(level,fmt,args...)
> > > > > +static inline void debugX(const char *fmt, ...) {}
> > > > 
> > > > NAK:   you broke debugX()
> > > 
> > > oh, right, good catch. But is this macro even used anyway?
> > 
> > grep says yes, a little
> 
> Maybe we should just squash it and get rid of it altogether ?

i've rarely found a need for this sort of style of debug logging, so i 
wouldn't mind seeing it go
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to