[coreboot-gerrit] Patch set updated for coreboot: 47f2787 src/console/post.c: Sanitize headers from preprocessor abuse

Edward O'Callaghan (eocallaghan@alterapraxis.com) gerrit at coreboot.org
Tue Jun 24 06:06:45 CEST 2014


Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/6091

-gerrit

commit 47f2787078c280817d2bb7db8c3b16f35aedb50a
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date:   Sat Jun 21 21:42:25 2014 +1000

    src/console/post.c: Sanitize headers from preprocessor abuse
    
    Some headers could not be taken care of here yet due to deeper issues.
    
    Change-Id: I524713b21684f6fa99355614a1ab38aee9975790
    Signed-off-by: Edward O'Callaghan <eocallaghan at alterapraxis.com>
---
 src/console/post.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/console/post.c b/src/console/post.c
index ef1a3a5..6b51abc 100644
--- a/src/console/post.c
+++ b/src/console/post.c
@@ -20,15 +20,17 @@
  */
 
 #include <arch/io.h>
+#include <elog.h>
 #include <console/console.h>
+#include <smp/spinlock.h>
+
+/* FIXME: unwrapping these header guards breaks builds due to deeper issues */
 #if CONFIG_CMOS_POST
 #include <pc80/mc146818rtc.h>
-#include <smp/spinlock.h>
 #endif
 #if CONFIG_CMOS_POST_EXTRA
 #include <device/device.h>
 #endif
-#include <elog.h>
 
 /* Write POST information */
 



More information about the coreboot-gerrit mailing list