Marc Jones ([email protected]) just uploaded a new patch set to gerrit, which 
you can find at http://review.coreboot.org/908

-gerrit

commit d79f893213490c1cb5bbe7c99a4c1aed11435412
Author: Marc Jones <[email protected]>
Date:   Thu Apr 19 23:44:54 2012 -0600

    Revert wbind added to the reset_vector
    
    This change reverts :
    Change Id I4fdb281b2b684ab5fea999aae28ca08dce24da4d
    
    The wbinvd (or invd) should not be needed at the reset vector. It
    causes problems with some CPUs AP init. If there is a problem with
    a specific CPU and it must be done at this location, it should be
    added conditionally.
    
    Change-Id: I85b71b0a07f039359a4fb889aaa05c75fff619be
    Signed-off-by: Marc Jones <[email protected]>
---
 src/cpu/x86/16bit/reset16.inc |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/src/cpu/x86/16bit/reset16.inc b/src/cpu/x86/16bit/reset16.inc
index 8dba3c8..1be0e3a 100644
--- a/src/cpu/x86/16bit/reset16.inc
+++ b/src/cpu/x86/16bit/reset16.inc
@@ -2,7 +2,6 @@
        .code16
 .globl reset_vector
 reset_vector:
-       wbinvd
        .byte  0xe9
        .int   _start - ( . + 2 )
        /* Note: The above jump is hand coded to work around bugs in binutils.

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

Reply via email to