[U-Boot] [PATCH 01/39] DEBUG: Fix debug macros

Wolfgang Denk wd at denx.de
Tue Oct 25 23:59:55 CEST 2011


Dear =?UTF-8?B?6aas5YWL5rOh?=,

In message <CACCg+XP8zoKGproJ4t=QEQU4ZFYccAPPjsGkz72BKBwowcUQng at mail.gmail.com> you wrote:
>
> > I'm working on a patch for the net code that benefits from the debugX
> > macro.  Especially for the net code, the debug traces can be
> > overwhelming depending on the issue you are looking at.  I recommend
> > we keep it around.
...
> I agreed with Joe,

And I disagree.

After 11 years of U-Boot development, after collecting 2,515 C files,
we have just 2 (in words: TWO) files that use this macro.  And one of
them with a single level only, so it's even useless there.

> I feel some debug code is necessary to keep it into levels since some
> you usually need to split debug level into 2 or 3 to help the debug work.

Usually?  In a single case out of 2.5k files?  That's not usual,
that's a mistake.


I say: we get rid of this!

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
       There is enough for the need of everyone in this world,
       but not for the greed of everyone.     - Mahatma Gandhi


More information about the U-Boot mailing list