[U-Boot] [PATCH] p1022ds: use weak CFI flash accessors when DIU is enabled
Wolfgang Denk
wd at denx.de
Thu Sep 16 23:29:35 CEST 2010
Dear Timur Tabi,
In message <4C928998.4040805 at freescale.com> you wrote:
>
> >> +/**
> >> + * set_mux_to_lbc - disable the DIU so that we can read/write to elbc
> >
> > Incorrect multiline comment format. Please check and fix globally.
>
> Can you be more specific? Is it the "**"?
Yes.
> > "#ifdef CONFIG_CFI_FLASH_USE_WEAK_ACCESSORS" / "#endif" arount the
> > #undef ?
>
> Why? CONFIG_CFI_FLASH_USE_WEAK_ACCESSORS is only defined if CONFIG_VIDEO is
> defined. It's just one block of macros.
OK, if you like it that way. OTOH, the #ifdef would not hurt and keep
the logic in place in case you ever define
CONFIG_CFI_FLASH_USE_WEAK_ACCESSORS for other reasons as well.
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
"...and the fully armed nuclear warheads, are, of course, merely a
courtesy detail."
More information about the U-Boot
mailing list