[coreboot] [PATCH] Split model_67x CPU support from model_6xx, with microcode / L2 cache code question
Uwe Hermann
uwe at hermann-uwe.de
Wed Oct 13 19:03:20 CEST 2010
On Tue, Oct 12, 2010 at 11:41:24PM -0400, Keith Hui wrote:
> Attached patch adds support for Katmai Slot 1 CPUs (model_67x).
> abuild-tested. Boot tested on P2B-LS.
>
> Signed-off-by: Keith Hui <buurin at gmail.com>
Thanks, r5950 with some small whitespace changes.
I also dropped the name.o from Makefile.inc, the functions provided
there are not (yet?) used in model_67x I think.
Also, you added 671 as CPU ID to model_67x which was not in there yet.
Can you point me to the document where this one is listed, so we can
reference it in a code comment?
> As we already know the L2 patch is a huge chunk of code, so let's
> figure out where best to place that code. It is specific to 63x,65x
> and 67x CPUs. There are some small code path differences between these
> CPU families. I'm trying to minimize duplication;
Yep.
> if these code can be
> put in one .c file and be linked from a few other places, that's how
> I'd prefer it.
Yes, that would be great.
Uwe.
--
http://hermann-uwe.de | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org
More information about the coreboot
mailing list