[coreboot] New patch to review for coreboot: 4ba39bc libpayload: Remove unnecessary include of arch/msr.h

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Tue Mar 12 02:21:38 CET 2013

Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/2656


commit 4ba39bc7acb0ccf3c510735464959fc58cb99dd4
Author: Gabe Black <gabeblack at google.com>
Date:   Fri Jan 18 15:14:03 2013 -0800

    libpayload: Remove unnecessary include of arch/msr.h
    The functions defined in that header aren't used anywhere in the actual code,
    and that include breaks things on ARM.
    Built for ARM with COREBOOT_VIDEO_CONSOLE turned on and saw compiler
    errors go away.
    Change-Id: I56d6fe5e00c8fccda6e31ef8752326bd36398e74
    Signed-off-by: Gabe Black <gabeblack at google.com>
 payloads/libpayload/drivers/video/corebootfb.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/payloads/libpayload/drivers/video/corebootfb.c b/payloads/libpayload/drivers/video/corebootfb.c
index 5672d1f..faf9e2c 100644
--- a/payloads/libpayload/drivers/video/corebootfb.c
+++ b/payloads/libpayload/drivers/video/corebootfb.c
@@ -32,7 +32,6 @@
 #include <coreboot_tables.h>
 #include <pci.h>
 #include <video_console.h>
-#include <arch/msr.h>
 #include "font8x16.h"
 struct video_console coreboot_video_console;

More information about the coreboot mailing list