[coreboot] [commit] r5215 - in trunk: . src/arch/i386/lib src/console src/include/console

repository service svn at coreboot.org
Tue Mar 16 02:02:18 CET 2010


Author: stepan
Date: Tue Mar 16 02:02:18 2010
New Revision: 5215
URL: https://tracker.coreboot.org/trac/coreboot/changeset/5215

Log:
This patch is what I needed to compile coreboot with LLVM.

- call va_* directly if coreboot is running on GCC so we don't need 
  to maintain hacks to get to stdarg.h
- only define LIBGCC_FILE_NAME if it's an absolute path. GCC and LLVM
  just print "libgcc.a" if the file is not there.

Signed-off-by: Stefan Reinauer <stepan at coresystems.de>
Acked-by: Ronald G. Minnich <rminnich at gmail.com>

Modified:
   trunk/Makefile
   trunk/src/arch/i386/lib/printk_init.c
   trunk/src/console/printk.c
   trunk/src/console/vsprintf.c
   trunk/src/console/vtxprintf.c
   trunk/src/include/console/vtxprintf.h

Modified: trunk/Makefile
==============================================================================
--- trunk/Makefile	Tue Mar 16 01:58:36 2010	(r5214)
+++ trunk/Makefile	Tue Mar 16 02:02:18 2010	(r5215)
@@ -63,9 +63,7 @@
 HOSTCXX = g++
 HOSTCFLAGS := -I$(srck) -I$(objk) -g
 HOSTCXXFLAGS := -I$(srck) -I$(objk)
-LIBGCC_FILE_NAME := $(shell $(CC) -print-libgcc-file-name)
-
-DESTDIR = /opt
+LIBGCC_FILE_NAME := $(shell test -r `$(CC) -print-libgcc-file-name` && $(CC) -print-libgcc-file-name)
 
 DOXYGEN := doxygen
 DOXYGEN_OUTPUT_DIR := doxygen
@@ -281,20 +279,20 @@
 	@echo smmobjs:=$(smmobjs)
 	@echo alldirs:=$(alldirs)
 	@echo allsrc=$(allsrc)
+	@echo LIBGCC_FILE_NAME=$(LIBGCC_FILE_NAME)
 
 printcrt0s:
 	@echo $(patsubst $(top)/%,%,$(crt0s))
 
 OBJS     := $(patsubst %,$(obj)/%,$(TARGETS-y))
 INCLUDES := -I$(top)/src -I$(top)/src/include -I$(obj) -I$(top)/src/arch/$(ARCHDIR-y)/include 
-INCLUDES += -I$(shell $(CC) -print-search-dirs | head -n 1 | cut -d' ' -f2)include
 INCLUDES += -I$(top)/util/x86emu/include
 INCLUDES += -include $(obj)/config.h
 
 CFLAGS = $(INCLUDES) -Os -nostdinc
 CFLAGS += -nostdlib -Wall -Wundef -Wstrict-prototypes -Wmissing-prototypes
 CFLAGS += -Wwrite-strings -Wredundant-decls -Wno-trigraphs 
-CFLAGS += -Wstrict-aliasing -Wshadow 
+CFLAGS += -Wstrict-aliasing -Wshadow
 ifeq ($(CONFIG_WARNINGS_ARE_ERRORS),y)
 CFLAGS += -Werror
 endif

Modified: trunk/src/arch/i386/lib/printk_init.c
==============================================================================
--- trunk/src/arch/i386/lib/printk_init.c	Tue Mar 16 01:58:36 2010	(r5214)
+++ trunk/src/arch/i386/lib/printk_init.c	Tue Mar 16 02:02:18 2010	(r5215)
@@ -17,7 +17,6 @@
  * MA 02110-1301 USA
  */
 
-#include <stdarg.h>
 #include <console/console.h>
 #include <console/vtxprintf.h>
 #include <console/loglevel.h>

Modified: trunk/src/console/printk.c
==============================================================================
--- trunk/src/console/printk.c	Tue Mar 16 01:58:36 2010	(r5214)
+++ trunk/src/console/printk.c	Tue Mar 16 02:02:18 2010	(r5215)
@@ -5,19 +5,10 @@
  *
  */
 
-#include <stdarg.h>
 #include <smp/spinlock.h>
 #include <console/vtxprintf.h>
 #include <console/console.h>
 
-/* printk's without a loglevel use this.. */
-#define DEFAULT_MESSAGE_LOGLEVEL 4 /* BIOS_WARNING */
-
-/* We show everything that is MORE important than this.. */
-#define MINIMUM_CONSOLE_LOGLEVEL 1 /* Minimum loglevel we let people use */
-
-/* Keep together for sysctl support */
-
 int console_loglevel = CONFIG_DEFAULT_CONSOLE_LOGLEVEL;
 int default_console_loglevel = CONFIG_DEFAULT_CONSOLE_LOGLEVEL;
 

Modified: trunk/src/console/vsprintf.c
==============================================================================
--- trunk/src/console/vsprintf.c	Tue Mar 16 01:58:36 2010	(r5214)
+++ trunk/src/console/vsprintf.c	Tue Mar 16 02:02:18 2010	(r5215)
@@ -19,7 +19,6 @@
  * MA 02110-1301 USA
  */
 
-#include <stdarg.h>
 #include <string.h>
 #include <smp/spinlock.h>
 #include <console/vtxprintf.h>

Modified: trunk/src/console/vtxprintf.c
==============================================================================
--- trunk/src/console/vtxprintf.c	Tue Mar 16 01:58:36 2010	(r5214)
+++ trunk/src/console/vtxprintf.c	Tue Mar 16 02:02:18 2010	(r5215)
@@ -4,7 +4,6 @@
  *  Copyright (C) 1991, 1992  Linus Torvalds
  */
 
-#include <stdarg.h>
 #include <string.h>
 #include <div64.h>
 #include <console/vtxprintf.h>

Modified: trunk/src/include/console/vtxprintf.h
==============================================================================
--- trunk/src/include/console/vtxprintf.h	Tue Mar 16 01:58:36 2010	(r5214)
+++ trunk/src/include/console/vtxprintf.h	Tue Mar 16 02:02:18 2010	(r5215)
@@ -20,7 +20,19 @@
 #ifndef __CONSOLE_VTXPRINTF_H
 #define __CONSOLE_VTXPRINTF_H
 
+/* With GCC we use -nostdinc -ffreestanding to keep out system includes.
+ * Unfortunately this also gets us rid of the _compiler_ includes, like
+ * stdarg.h. To work around the issue, we define varargs directly here.
+ * On LLVM we can still just include stdarg.h.
+ */
+#ifdef __GNUC__
+#define va_start(v,l)		__builtin_va_start(v,l)
+#define va_end(v)		__builtin_va_end(v)
+#define va_arg(v,l)		__builtin_va_arg(v,l)
+typedef __builtin_va_list	va_list;
+#else
 #include <stdarg.h>
+#endif
 
 int vtxprintf(void (*tx_byte)(unsigned char byte), const char *fmt, va_list args);
 




More information about the coreboot mailing list