[coreboot] [PATCH]Kconfig: fix QEmu build, reduce bootblock size

Patrick Georgi patrick at georgi-clan.de
Thu Aug 13 17:31:00 CEST 2009


Stefan Reinauer schrieb:
> I think putting "HAVE_INIT_TIMER" in AMD model specific code is
> generally wrong. It's not an AMD specific option. Also enabling options
> per default that are very cpu specific is bad. Is there no better way to
> do this?
>   
The AMD code merely enables it. It's defined in src/Kconfig as disabled,
and then can be enabled wherever appropriate (though I don't know if
_this_ place in the AMD code is appropriate).

When enabling variables, it just has to be conditional on the
responsible component, otherwise it's enabled for everyone.


Patrick




More information about the coreboot mailing list