From 52fc6b12cb458a93ba1eeac4082f4e7f574cdafc Mon Sep 17 00:00:00 2001 From: Stefan Reinauer Date: Sat, 24 Oct 2009 13:06:04 +0000 Subject: [PATCH] re-order console output functions, add proper prototypes, drop claim that our files were blatantly copied, because they have been rewritten a very long time ago. Signed-off-by: Stefan Reinauer Acked-by: Carl-Daniel Hailfinger git-svn-id: svn://svn.coreboot.org/coreboot/trunk@4837 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1 --- src/arch/i386/include/stdint.h | 2 +- src/arch/i386/lib/console.c | 13 ++----- src/arch/i386/lib/console_print.c | 4 +++ src/arch/i386/lib/printk_init.c | 35 ++++++++++++------- src/arch/ppc/lib/printk_init.c | 28 +++++++++------ src/console/Makefile.inc | 1 - src/console/printk.c | 4 +-- src/console/vsprintf.c | 58 ++++++++++++++++++++----------- src/console/vtxprintf.c | 39 +-------------------- src/include/console/vtxprintf.h | 29 ++++++++++++++++ 10 files changed, 116 insertions(+), 97 deletions(-) create mode 100644 src/include/console/vtxprintf.h diff --git a/src/arch/i386/include/stdint.h b/src/arch/i386/include/stdint.h index 7f6b94037f..a015a84b2a 100644 --- a/src/arch/i386/include/stdint.h +++ b/src/arch/i386/include/stdint.h @@ -1,7 +1,7 @@ #ifndef I386_STDINT_H #define I386_STDINT_H -#if defined(__GNUC__) && !defined(__ROMCC__) +#if defined(__GNUC__) #define __HAVE_LONG_LONG__ 1 #else #define __HAVE_LONG_LONG__ 0 diff --git a/src/arch/i386/lib/console.c b/src/arch/i386/lib/console.c index 47015bc75c..2bab6032d3 100644 --- a/src/arch/i386/lib/console.c +++ b/src/arch/i386/lib/console.c @@ -1,18 +1,9 @@ #include #if CONFIG_USE_PRINTK_IN_CAR == 0 -static void __console_tx_byte(unsigned char byte) -{ - uart_tx_byte(byte); -} - #include "console_print.c" - -#else -/* CONFIG_USE_PRINTK_IN_CAR == 1 */ - +#else /* CONFIG_USE_PRINTK_IN_CAR == 1 */ #include "console_printk.c" - #endif /* CONFIG_USE_PRINTK_IN_CAR */ #ifndef COREBOOT_EXTRA_VERSION @@ -32,7 +23,7 @@ static void console_init(void) } -static void die(const char *str) +void die(const char *str) { print_emerg(str); do { diff --git a/src/arch/i386/lib/console_print.c b/src/arch/i386/lib/console_print.c index 93886cc34d..ed807e6064 100644 --- a/src/arch/i386/lib/console_print.c +++ b/src/arch/i386/lib/console_print.c @@ -1,3 +1,7 @@ +static void __console_tx_byte(unsigned char byte) +{ + uart_tx_byte(byte); +} static void __console_tx_nibble(unsigned nibble) { diff --git a/src/arch/i386/lib/printk_init.c b/src/arch/i386/lib/printk_init.c index 6163acc2ae..7e5dfa03dc 100644 --- a/src/arch/i386/lib/printk_init.c +++ b/src/arch/i386/lib/printk_init.c @@ -1,31 +1,42 @@ /* - * blantantly copied from linux/kernel/printk.c + * This file is part of the coreboot project. + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; version 2 of + * the License. * - * Copyright (C) 1991, 1992 Linus Torvalds + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. * - * by yhlu moved from arch/ppc/lib/printk_init.c, removed the global variable console_loglevel + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, + * MA 02110-1301 USA */ + #include +#include #include +#include -/* printk's without a loglevel use this.. */ -#define DEFAULT_MESSAGE_LOGLEVEL 4 /* BIOS_WARNING */ - -/* Keep together for sysctl support */ -/* Using an global varible can cause problem when we reset the stack from cache as ram to ram*/ +/* Using a global varible can cause problems when we reset the stack + * from cache as ram to ram. If we make this a define USE_SHARED_STACK + * we could use the same code on all architectures. + */ #if 0 int console_loglevel = CONFIG_DEFAULT_CONSOLE_LOGLEVEL; #else -#define console_loglevel ASM_CONSOLE_LOGLEVEL +#define console_loglevel CONFIG_DEFAULT_CONSOLE_LOGLEVEL #endif -extern int vtxprintf(void (*)(unsigned char), const char *, va_list); -extern void uart8250_tx_byte(unsigned, unsigned char); - void console_tx_byte(unsigned char byte) { if (byte == '\n') uart8250_tx_byte(CONFIG_TTYS0_BASE, '\r'); + uart8250_tx_byte(CONFIG_TTYS0_BASE, byte); } diff --git a/src/arch/ppc/lib/printk_init.c b/src/arch/ppc/lib/printk_init.c index 98d78b3fc4..d9e6bacfb3 100644 --- a/src/arch/ppc/lib/printk_init.c +++ b/src/arch/ppc/lib/printk_init.c @@ -1,26 +1,34 @@ /* - * blantantly copied from linux/kernel/printk.c + * This file is part of the coreboot project. + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; version 2 of + * the License. * - * Copyright (C) 1991, 1992 Linus Torvalds + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, + * MA 02110-1301 USA */ + #include +#include #include - -/* printk's without a loglevel use this.. */ -#define DEFAULT_MESSAGE_LOGLEVEL 4 /* BIOS_WARNING */ - -/* Keep together for sysctl support */ +#include int console_loglevel = CONFIG_DEFAULT_CONSOLE_LOGLEVEL; -extern int vtxprintf(void (*)(unsigned char), const char *, va_list); -extern void uart8250_tx_byte(unsigned, unsigned char); - void console_tx_byte(unsigned char byte) { if (byte == '\n') uart8250_tx_byte(CONFIG_TTYS0_BASE, '\r'); + uart8250_tx_byte(CONFIG_TTYS0_BASE, byte); } diff --git a/src/console/Makefile.inc b/src/console/Makefile.inc index 4545ce5a37..d721fb4e33 100644 --- a/src/console/Makefile.inc +++ b/src/console/Makefile.inc @@ -4,7 +4,6 @@ obj-y += vtxprintf.o obj-y += vsprintf.o initobj-y += vtxprintf.o -initobj-y += vsprintf.o driver-$(CONFIG_CONSOLE_SERIAL8250) += uart8250_console.o driver-$(CONFIG_USBDEBUG_DIRECT) += usbdebug_direct_console.o diff --git a/src/console/printk.c b/src/console/printk.c index f4157a7b00..3b1ae24ec1 100644 --- a/src/console/printk.c +++ b/src/console/printk.c @@ -7,6 +7,7 @@ #include #include +#include #include /* printk's without a loglevel use this.. */ @@ -22,9 +23,6 @@ int default_message_loglevel = DEFAULT_MESSAGE_LOGLEVEL; int minimum_console_loglevel = MINIMUM_CONSOLE_LOGLEVEL; int default_console_loglevel = CONFIG_DEFAULT_CONSOLE_LOGLEVEL; -void display(char*); -extern int vtxprintf(void (*)(unsigned char), const char *, va_list); - static spinlock_t console_lock = SPIN_LOCK_UNLOCKED; int do_printk(int msg_level, const char *fmt, ...) diff --git a/src/console/vsprintf.c b/src/console/vsprintf.c index bfa6ffebab..9507e8ea54 100644 --- a/src/console/vsprintf.c +++ b/src/console/vsprintf.c @@ -1,45 +1,61 @@ /* - * linux/lib/vsprintf.c + * This file is part of the coreboot project. + * + * Copyright (C) 2009 coresystems GmbH * - * Copyright (C) 1991, 1992 Linus Torvalds + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; version 2 of + * the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, + * MA 02110-1301 USA */ -/* vsprintf.c -- Lars Wirzenius & Linus Torvalds. */ -/* - * Wirzenius wrote this portably, Torvalds fucked it up :-) - */ #include -#include +#include +#include -int vtxprintf(void (*tx_byte)(unsigned char byte), const char *fmt, va_list args); +static spinlock_t vsprintf_lock = SPIN_LOCK_UNLOCKED; -int vsprintf(char * buf, const char *fmt, va_list args) +static char *str_buf; + +static void str_tx_byte(unsigned char byte) { - char *str_buf; - - /* this function is only used by vsprintf. - To keep str_buf local (for reentrancy - and to avoid .bss use, nest it */ - void str_tx_byte(unsigned char byte) - { - *str_buf = byte; - str_buf++; - } + *str_buf = byte; + str_buf++; +} +static int vsprintf(char *buf, const char *fmt, va_list args) +{ int i; + + spin_lock(&vsprintf_lock); + str_buf = buf; i = vtxprintf(str_tx_byte, fmt, args); *str_buf = '\0'; + + spin_unlock(&vsprintf_lock); + return i; } -int sprintf(char * buf, const char *fmt, ...) +int sprintf(char *buf, const char *fmt, ...) { va_list args; int i; va_start(args, fmt); - i=vsprintf(buf,fmt,args); + i = vsprintf(buf, fmt, args); va_end(args); + return i; } diff --git a/src/console/vtxprintf.c b/src/console/vtxprintf.c index 72bbac5fb0..b1b53b6a39 100644 --- a/src/console/vtxprintf.c +++ b/src/console/vtxprintf.c @@ -7,50 +7,13 @@ #include #include #include +#include /* haha, don't need ctype.c */ #define isdigit(c) ((c) >= '0' && (c) <= '9') #define is_digit isdigit #define isxdigit(c) (((c) >= '0' && (c) <= '9') || ((c) >= 'a' && (c) <= 'f') || ((c) >= 'A' && (c) <= 'F')) -#if 0 -/* We are using number() instead. So this code is obsoleted and should - * probably go away. - */ - -static unsigned long simple_strtoul(const char *cp,char **endp,unsigned int base) -{ - unsigned long result = 0,value; - - if (!base) { - base = 10; - if (*cp == '0') { - base = 8; - cp++; - if ((*cp == 'x') && isxdigit(cp[1])) { - cp++; - base = 16; - } - } - } - while (isxdigit(*cp) && (value = isdigit(*cp) ? *cp-'0' : (islower(*cp) - ? toupper(*cp) : *cp)-'A'+10) < base) { - result = result*base + value; - cp++; - } - if (endp) - *endp = (char *)cp; - return result; -} - -static long simple_strtol(const char *cp,char **endp,unsigned int base) -{ - if(*cp=='-') - return -simple_strtoul(cp+1,endp,base); - return simple_strtoul(cp,endp,base); -} -#endif - static int skip_atoi(const char **s) { int i=0; diff --git a/src/include/console/vtxprintf.h b/src/include/console/vtxprintf.h new file mode 100644 index 0000000000..4c7c51c375 --- /dev/null +++ b/src/include/console/vtxprintf.h @@ -0,0 +1,29 @@ +/* + * This file is part of the coreboot project. + * + * Copyright (C) 2009 coresystems GmbH + * + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public License as + * published by the Free Software Foundation; version 2 of + * the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, + * MA 02110-1301 USA + */ + +#ifndef __CONSOLE_VTXPRINTF_H +#define __CONSOLE_VTXPRINTF_H + +#include + +int vtxprintf(void (*tx_byte)(unsigned char byte), const char *fmt, va_list args); + +#endif