On Fri, Aug 1, 2008 at 9:50 AM, Marc Jones <[EMAIL PROTECTED]> wrote:

>
>> -#endif                         /* ARCH_X86_MTRR_H */
>> Index: arch/x86/Makefile
>> ===================================================================
>> --- arch/x86/Makefile   (revision 699)
>> +++ arch/x86/Makefile   (working copy)
>> @@ -124,9 +124,12 @@
>>        STAGE0_ARCH_X86_OBJ += geodelx/stage1.o
>>        STAGE0_ARCH_X86_OBJ += ../../northbridge/amd/geodelx/geodelxinit.o
>>  else
>> -       STAGE0_CAR_OBJ = stage0_i586.o
>> +ifeq ($(CONFIG_CPU_AMD_K8),y)
>> +       STAGE0_CAR_OBJ = amd/stage0.o
>> +       STAGE0_ARCH_X86_OBJ += amdk8/stage1.o
>>  endif
>>  endif
>> +endif
>
> Add else and put the stage0_i586 on the end?

I took it out. There should be a valid value for the CPU. anything
else is an error. This mistake bit me so I don't want it getting
anyone else.



>
> I agree with getting the pieces in place as we go.
> The files looked fine to me but I didn't try to build it.....
>
> Acked-by: Marc Jones <[EMAIL PROTECTED]>

Committed revision 702.

I added gigabyte/Kconfig per peter's note.

ron

--
coreboot mailing list
[email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to