Keith Hui wrote:
Attached patch moves support for Deschutes Slot 1 CPUs (model_65x) into its own home.
Then I guess the patch should also delete some lines in existing
files?
Yes. I knew I must be missing something. (Oops) See attached patch.
Signed-off-by: Keith Hui buurin@gmail.com
This is similar to the Katmai patch I submitted before, but without microcode this time. With 17 of them they will make the patch too large. I think I'll defer them for later or someone else*, hehe. :D
abuild-tested.
It doesn't look like the new code is used anywhere? Should it be?
It should be used from cpu/intel/slot_1.
- also because I like to keep all microcodes for the same family
within one file instead of one microcode per file.
We want microcode updates to be as close as possible to upstream
How upstream are we talking about? :P These updates come from Intel as one big .h file. ;-)
format. Eventually I think we want to put them verbatim into cbfs.
This I agree wholeheartedly. Did I ever mention that Award also stuff their microcode updates this way into the firmware image? And hey, I might as well take a crack at this!
But a bit of scheduling headache then awaits. How much CPU init do we need to do before we start drawing things in from the cbfs? Is there anything that cannot be done until microcode updates are loaded? Is cpu_dev_ops.init() an appropriate place to access cbfs? If not, from where?
Cheers Keith