On Thursday 10 November 2011 00:43:54 Wolfgang Denk wrote: > Mike Frysinger wrote: > > > #undef VGA_DEBUG > > > #undef VGA_DUMP_REG > > > #ifdef VGA_DEBUG > > > > > > -#define PRINTF(fmt,args...) printf (fmt ,##args) > > > +#undef _DEBUG > > > +#define _DEBUG 1 > > > > > > #else > > > > > > -#define PRINTF(fmt,args...) > > > +#undef _DEBUG > > > +#define _DEBUG 0 > > > > > > #endif > > > > so do you still need this ? since you've converted to debug(), looks > > like this logic can all go away. > > It's needed if I want to keep the original behaviour regarding > VGA_DEBUG and VGA_DUMP_REG which were independent before.
ok, i see that now. i wonder if we need that though. i don't think there's any real difference between doing "#define DEBUG" and "#define VGA_DEBUG". -mike
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot