[U-Boot] [PATCH 3/8] USB-CDC: Linux-like debug printout
Wolfgang Denk
wd at denx.de
Fri Aug 13 12:16:48 CEST 2010
Dear Remy Bohmer,
In message <AANLkTi=gnFZr6wT-RJ6CLFEKuGGhhpJUDbj1VftAR9kX at mail.gmail.com> you wrote:
>
> >>> +#define WARN INFO
> >>> +#define ERROR INFO
> >>> +#define DEBUG INFO
> >>
> >> This switches DEBUG logging on by default. This is not wanted.
> >> Can you please change that?
>
> > No, it does not.
>
> Well, I see with this patch much more debug logging then without it...
> Hmm, it seems that you replaced all use of dprintf (which is trashed)
> by DEBUG()...
>
> > Anyway I can change it.
>
> please do, DEBUG logging should not be on by default...
DEBUG is already a well-defined name. Any different use of the same
name will result in the patches being rejected.
Please fix this!
I also object against names like WARN, ERROR and INFO. They are just
too dangerous.
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
You are an excellent tactician, Captain. You let your second in com-
mand attack while you sit and watch for weakness.
-- Khan Noonian Singh, "Space Seed", stardate 3141.9
More information about the U-Boot
mailing list