[U-Boot] [PATCH 5/9] ARM: Rename CONFIG_SKIP_LOWLEVEL_INIT to CONFIG_SKIP_CPU_INIT_CRIT

Wolfgang Denk
2011-04-24T23:53:53Z

Dear Michael Spang,

In message <1300391223-11879-6-git-send-email-mspang@csclub.uwaterloo.ca> you wrote:
> For ARM, the CONFIG_SKIP_LOWLEVEL_INIT option implies that the
> cpu_init_crit and lowlevel_init assembly functions are skipped. We may
> want to skip only lowlevel_init, so rename the option that skips both
> to CONFIG_SKIP_CPU_INIT_CRIT. The MIPS option of the same name
> is not renamed.

This is an inconsistency between architectures which I dislike.

Also, what in case we should want to skip only cpu_init_crit and not
lowlevel_init? If we need to handle these separately, then we need 2
CONFIG options.

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
"Plan to throw one away. You will anyway."
- Fred Brooks, "The Mythical Man Month"