[coreboot-gerrit] New patch to review for coreboot: 1b6f7c6 lenovo/x60/i915.c: Define `BSM` to fix build error

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Fri Aug 15 11:34:06 CEST 2014


Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/6673

-gerrit

commit 1b6f7c679aec6531de238f651c58850ca3dc745f
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Fri Aug 15 10:32:50 2014 +0200

    lenovo/x60/i915.c: Define `BSM` to fix build error
    
    Commit 8aeab56b (lenovo/x60/i915.c: Use define for `BSM`) [1] breaks the
    build of the Lenovo X60 with native graphics initialization [2].
    
    	    CC         mainboard/lenovo/x60/i915.ramstage.o
    	src/mainboard/lenovo/x60/i915.c: In function ‘gtt_setup’:
    	src/mainboard/lenovo/x60/i915.c:141:69: error: ‘BSM’ undeclared (first use in this function)
    	src/mainboard/lenovo/x60/i915.c:141:69: note: each undeclared identifier is reported only once for each function it appears in
    
    To fix the error, add the define for `BSM` in the file directly as
    including the header `src/northbridge/intel/i945/i945.h` does not work.
    
    [1] http://review.coreboot.org/5932
    [2] This was not caught as only the commit on top was build tested,
        which removes that line where `BSM` is used.
    
    Change-Id: Ic24c66b0175a67ad95064a9840daa615ab88c1ee
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 src/mainboard/lenovo/x60/i915.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/mainboard/lenovo/x60/i915.c b/src/mainboard/lenovo/x60/i915.c
index 8fbd2a1..529b262 100644
--- a/src/mainboard/lenovo/x60/i915.c
+++ b/src/mainboard/lenovo/x60/i915.c
@@ -48,6 +48,7 @@ enum {
 	vmsg = 1, vio = 2, vspin = 4,
 };
 
+#define BSM 0x5c
 #define PGETLB_CTL 0x2020
 
 static int verbose = 0;



More information about the coreboot-gerrit mailing list