[U-Boot] [PATCH v2 6/7] mtd: nand: add driver for diskonchip g4 nand flash
Scott Wood
scottwood at freescale.com
Fri Apr 12 19:34:01 CEST 2013
On 04/12/2013 12:27:09 PM, Marek Vasut wrote:
> Dear Scott Wood,
>
> [...]
> >
> > [snip]
> >
> > > +#ifdef DEBUG_DOCG4
> > > +#define dbg(format, arg...) printf("DEBUG: " format, ##arg)
> > > +#else
> > > +#define dbg(format, arg...) do {} while (0)
> > > +#endif
>
> There already is a debugging facility in include/common.h too. And it
> does the
> same thing too. MTDDEBUG might work as well.
I already pointed out MTDDEBUG, though debug() may be a better fit as
it doesn't require adding debug levels, and since this driver's usage
doesn't correspond to CONFIG_MTD_DEBUG in Linux (which appears to have
been removed from Linux -- removal from U-Boot should likely follow).
Mike said he may just remove the debug messages entirely (maybe also
from the Linux driver?), making it moot.
-Scott
More information about the U-Boot
mailing list