[coreboot] [PATCH] model_65x CPU support

Keith Hui buurin at gmail.com
Sun Oct 17 02:48:15 CEST 2010


>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 at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: model_65x_partb.patch
Type: application/octet-stream
Size: 906 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20101016/d588c009/attachment.obj>


More information about the coreboot mailing list