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