[coreboot-gerrit] Patch set updated for coreboot: ad14f45 console: Hide global console_loglevel

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Mon Apr 7 10:39:42 CEST 2014


Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5333

-gerrit

commit ad14f45e256cc158631ad91a7b01e00fdbd9650b
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Fri Feb 28 14:37:27 2014 +0200

    console: Hide global console_loglevel
    
    Change-Id: I7bdc468bc3f74516abb2c583bdb5b6d7555d987c
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/arch/x86/lib/romcc_console.c | 21 +++++++--------------
 src/console/init.c               | 10 ++++++++--
 src/console/printk.c             |  3 +--
 src/include/console/console.h    | 10 +---------
 4 files changed, 17 insertions(+), 27 deletions(-)

diff --git a/src/arch/x86/lib/romcc_console.c b/src/arch/x86/lib/romcc_console.c
index 3f8f62f..a6e8ecc 100644
--- a/src/arch/x86/lib/romcc_console.c
+++ b/src/arch/x86/lib/romcc_console.c
@@ -17,18 +17,11 @@
  * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
-#include <stdint.h>
-#include <console/loglevel.h>
-#include <console/post_codes.h>
 #include <console/uart.h>
 #include <console/ne2k.h>
 
-/* __PRE_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.
- */
-#define console_loglevel CONFIG_DEFAULT_CONSOLE_LOGLEVEL
+/* While in romstage, console loglevel is built-time constant. */
+#define console_log_level(msg_level) (CONFIG_DEFAULT_CONSOLE_LOGLEVEL >= msg_level)
 
 #if CONFIG_CONSOLE_SERIAL && CONFIG_DRIVERS_UART_8250IO
 #include "drivers/uart/util.c"
@@ -60,7 +53,7 @@ static void __console_tx_nibble(unsigned nibble)
 
 static void __console_tx_char(int loglevel, unsigned char byte)
 {
-	if (console_loglevel >= loglevel) {
+	if (console_log_level(loglevel)) {
 #if CONFIG_CONSOLE_SERIAL
 		uart_tx_byte(byte);
 #endif
@@ -73,7 +66,7 @@ static void __console_tx_char(int loglevel, unsigned char byte)
 
 static void __console_tx_hex8(int loglevel, unsigned char value)
 {
-	if (console_loglevel >= loglevel) {
+	if (console_log_level(loglevel)) {
 		__console_tx_nibble((value >>  4U) & 0x0fU);
 		__console_tx_nibble(value & 0x0fU);
 	}
@@ -84,7 +77,7 @@ static void __console_tx_hex8(int loglevel, unsigned char value)
 
 static void __console_tx_hex16(int loglevel, unsigned short value)
 {
-	if (console_loglevel >= loglevel) {
+	if (console_log_level(loglevel)) {
 		__console_tx_nibble((value >> 12U) & 0x0fU);
 		__console_tx_nibble((value >>  8U) & 0x0fU);
 		__console_tx_nibble((value >>  4U) & 0x0fU);
@@ -97,7 +90,7 @@ static void __console_tx_hex16(int loglevel, unsigned short value)
 
 static void __console_tx_hex32(int loglevel, unsigned int value)
 {
-	if (console_loglevel >= loglevel) {
+	if (console_log_level(loglevel)) {
 		__console_tx_nibble((value >> 28U) & 0x0fU);
 		__console_tx_nibble((value >> 24U) & 0x0fU);
 		__console_tx_nibble((value >> 20U) & 0x0fU);
@@ -114,7 +107,7 @@ static void __console_tx_hex32(int loglevel, unsigned int value)
 
 static void __console_tx_string(int loglevel, const char *str)
 {
-	if (console_loglevel >= loglevel) {
+	if (console_log_level(loglevel)) {
 		unsigned char ch;
 		while((ch = *str++) != '\0') {
 			if (ch == '\n')
diff --git a/src/console/init.c b/src/console/init.c
index c6ceab5..d7c8cc3 100644
--- a/src/console/init.c
+++ b/src/console/init.c
@@ -29,8 +29,14 @@
 #include <device/pci.h>
 #endif
 
-#if !defined(__PRE_RAM__)
-int console_loglevel = CONFIG_DEFAULT_CONSOLE_LOGLEVEL;
+#if !defined(__ROMCC__)
+/* While in romstage, console loglevel is built-time constant. */
+static ROMSTAGE_CONST int console_loglevel = CONFIG_DEFAULT_CONSOLE_LOGLEVEL;
+
+int console_log_level(int msg_level)
+{
+	return (console_loglevel >= msg_level);
+}
 #endif
 
 void console_init(void)
diff --git a/src/console/printk.c b/src/console/printk.c
index 7ea8c06..fb1194e 100644
--- a/src/console/printk.c
+++ b/src/console/printk.c
@@ -18,9 +18,8 @@ int do_printk(int msg_level, const char *fmt, ...)
 	va_list args;
 	int i;
 
-	if (msg_level > console_loglevel) {
+	if (!console_log_level(msg_level))
 		return 0;
-	}
 
 #if CONFIG_SQUELCH_EARLY_SMP && defined(__PRE_RAM__)
 	if (!boot_cpu())
diff --git a/src/include/console/console.h b/src/include/console/console.h
index 6d1b01a..aa5273f 100644
--- a/src/include/console/console.h
+++ b/src/include/console/console.h
@@ -40,18 +40,10 @@ struct console_driver {
 /* Defined by the linker... */
 extern struct console_driver console_drivers[];
 extern struct console_driver econsole_drivers[];
-
-extern int console_loglevel;
-#else
-/* __PRE_RAM__ */
-/* Using a global variable 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.
- */
-#define console_loglevel CONFIG_DEFAULT_CONSOLE_LOGLEVEL
 #endif
 
 #ifndef __ROMCC__
+int console_log_level(int msg_level);
 void console_init(void);
 void console_hw_init(void);
 void console_tx_byte(unsigned char byte);



More information about the coreboot-gerrit mailing list