[U-Boot] [PATCH 07/11] blackfin: bf609: add board and headers files to support bf609
Wolfgang Denk
wd at denx.de
Sun Feb 17 21:13:57 CET 2013
Dear Sonic Zhang,
In message <CAJxxZ0MbkmPgGjFrQR3Uq0M=V-67gBryF1oTQd6_ayv8pZh2Dw at mail.gmail.com> you wrote:
> On Thu, Feb 7, 2013 at 6:26 PM, Wolfgang Denk <wd at denx.de> wrote:
> > Dear Sonic Zhang,
> >
> > In message <1360223258-6945-8-git-send-email-sonic.adi at gmail.com> you wrote:
> >> From: Sonic Zhang <sonic.zhang at analog.com>
> >>
> >> Board and config header files for bf609-ezkit support.
> >
> > Checkpatch errors, please fix.
> >
> >
> >> +#define ADI_CMDS_NETWORK 1
> >> +#define CONFIG_NETCONSOLE 1
> >> +#define CONFIG_NET_MULTI 1
> >
> > Please do not define values for variables that don't actually have
> > one. Please fix globally.
>
> The values of these 3 variables are used in
> include/configs/bfin_adi_common.h. They should be defined values,
> otherwise the compilation fails.
This is a bug in include/configs/bfin_adi_common.h then, which needs
to be fixed there.
It is wrong to use #if on these defines.
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
How much net work could a network work, if a network could net work?
More information about the U-Boot
mailing list