so move failover.inc after the mmx setup. This is definitely an issue
in the s1850; seperate patch coming.

Can someone test this patch please?

ron
This may not be a problem yet. But failover.inc, when used with the 
right romcc options, will need mmx (and in future SSE) to be set up.
Move it to the right place. 

Can someone please test. 

Signed-off-by: Ronald G. Minnich <[email protected]>
Index: src/mainboard/Makefile.romccboard.inc
===================================================================
--- src/mainboard/Makefile.romccboard.inc	(revision 4735)
+++ src/mainboard/Makefile.romccboard.inc	(working copy)
@@ -24,15 +24,16 @@
 crt0-y += ../../../../src/cpu/x86/16bit/reset16.inc
 crt0-y += ../../../../src/arch/i386/lib/cpu_reset.inc
 crt0-y += ../../../../src/arch/i386/lib/id.inc
-crt0-y += failover.inc
 crt0-y += ../../../../src/cpu/x86/fpu/enable_fpu.inc
 
 ifeq ($(CONFIG_CPU_AMD_GX1),y)
 crt0-y += ../../../../src/cpu/amd/model_gx1/cpu_setup.inc
 crt0-y += ../../../../src/cpu/amd/model_gx1/gx_setup.inc
+crt0-y += failover.inc
 crt0-y += auto.inc
 else
 crt0-y += ../../../../src/cpu/x86/mmx/enable_mmx.inc
+crt0-y += failover.inc
 crt0-y += auto.inc
 crt0-y += ../../../../src/cpu/x86/mmx/disable_mmx.inc
 endif
-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to