[U-Boot] [PATCH v5 2/9] x86: Allow excluding reset vector code from u-boot

Wolfgang Denk wd at denx.de
Tue Nov 27 14:41:10 CET 2012


Dear Simon Glass,

In message <CAPnjgZ2EjgHHnNj-0dyHgMMHhomYLuVJE=KF7pFCpwRSbDgY8g at mail.gmail.com> you wrote:
> 
> > CONFIG_SYS_X86_RESET_VECTOR - is this just another new, undocumented
> > variable?
> 
> Yes it is new and undocumented, because it is internal to the x86
> implementation and I don't want people to set it.

Then document it as internal and not to be touched, but documented it
must be.

> Well there are now two options:
> 
> 1. The user-facing CONFIG_NO_RESET_CODE option which, if not defined,
> asserts CONFIG_SYS_X86_RESET_VECTOR
> 2. CONFIG_SYS_X86_RESET_VECTOR which is used in the Makefiles
> 
> I would rather have a single positive option
> (CONFIG_SYS_X86_RESET_VECTOR, as series v4). Failing that I would
> rather have ifneq in the Makefile (and just use CONFIG_NO_RESET_CODE).
> I am not sure how to have a negative option without an ifneq in the
> Makefile. I looked pretty hard but could not find an example in
> U-Boot.

First, you can #define CONFIG_SYS_X86_RESET_VECTOR in some global
header file, and in the boards that don't want it add an #undef in the
board config file.

Second, you can have some

	RESET_OBJS-$(CONFIG_SYS_X86_RESET_VECTOR) = <your_list>

in your Makefile, and then use
	
	COBJS   := $(sort $(COBJS-y) $(RESET_OBJS-))

which will include <your_list> only if CONFIG_SYS_X86_RESET_VECTOR is
_not_ set or empty.

etc.

> This is possible a minor point since I think Graeme said that the only
> other x86 board will move to Coreboot soon.

...for the time being, maybe.  But we should not bar other options
without serious need.

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
Doubt is a pain too lonely to know that faith is his twin brother.
- Kahlil Gibran


More information about the U-Boot mailing list