aboutsummaryrefslogtreecommitdiff
path: root/src/include/console
diff options
context:
space:
mode:
authorStefan Reinauer <reinauer@chromium.org>2015-01-05 13:12:38 -0800
committerKyösti Mälkki <kyosti.malkki@gmail.com>2015-01-09 06:12:22 +0100
commitd6865222c84c9da02cb515329af6a5384638d521 (patch)
tree813e9b2d76cb4d17d68acb505abac4279efbc3ad /src/include/console
parentc62ee70b6ef651288c4c4710319ee313b28fd520 (diff)
misc: Drop print_ implementation from non-romcc boards
Because we had no stack on romcc boards, we had a separate, not as powerful clone of printk: print_*. Back in the day, like more than half a decade ago, we migrated a lot of boards to printk, but we never cleaned up the existing code to be consistent. Instead, we worked around the problem with a very messy console.h (nowadays the mess is hidden in romstage_console.c and early_print.h) This patch cleans up the generic code pieces to use printk() on all non-ROMCC boards. Our two remaining ROMCC boards are fixed up in this commit: bifferos/bifferboard and dmp/vortex86ex. Change-Id: I16676eeabe5c892c8e3c9f3c0cd3bae2e8fd74b6 Signed-off-by: Stefan Reinauer <stefan.reinauer@coreboot.org> Reviewed-on: http://review.coreboot.org/8115 Tested-by: build bot (Jenkins) Reviewed-by: Edward O'Callaghan <eocallaghan@alterapraxis.com> Reviewed-by: Andrew Wu <arw@dmp.com.tw> Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Diffstat (limited to 'src/include/console')
-rw-r--r--src/include/console/console.h12
-rw-r--r--src/include/console/early_print.h54
-rw-r--r--src/include/console/streams.h7
3 files changed, 2 insertions, 71 deletions
diff --git a/src/include/console/console.h b/src/include/console/console.h
index 9e98bfc59c..38e05b3fb1 100644
--- a/src/include/console/console.h
+++ b/src/include/console/console.h
@@ -23,6 +23,7 @@
#include <stdint.h>
#include <rules.h>
#include <console/post_codes.h>
+#include <console/loglevel.h>
#ifndef __ROMCC__
void post_code(u8 value);
@@ -70,15 +71,6 @@ static inline void do_vtxprintf(const char *fmt, va_list args) {};
#endif
#endif
-/* A lot of code still uses print_debug() et al. while use of printk()
- * would be preferred.
- */
-#include <console/early_print.h>
-
-#else /* __ROMCC__ */
-
-#include "arch/x86/lib/romcc_console.c"
-
-#endif /* __ROMCC__ */
+#endif /* !__ROMCC__ */
#endif /* CONSOLE_CONSOLE_H_ */
diff --git a/src/include/console/early_print.h b/src/include/console/early_print.h
index 8bc1489dd4..cbc9b4616a 100644
--- a/src/include/console/early_print.h
+++ b/src/include/console/early_print.h
@@ -92,58 +92,4 @@
#define print_debug_hex32(HEX) __console_tx_hex32(BIOS_DEBUG, HEX)
#define print_spew_hex32(HEX) __console_tx_hex32(BIOS_SPEW, HEX)
-#else
-
-#define print_emerg(STR) printk(BIOS_EMERG, "%s", (STR))
-#define print_alert(STR) printk(BIOS_ALERT, "%s", (STR))
-#define print_crit(STR) printk(BIOS_CRIT, "%s", (STR))
-#define print_err(STR) printk(BIOS_ERR, "%s", (STR))
-#define print_warning(STR) printk(BIOS_WARNING,"%s", (STR))
-#define print_notice(STR) printk(BIOS_NOTICE, "%s", (STR))
-#define print_info(STR) printk(BIOS_INFO, "%s", (STR))
-#define print_debug(STR) printk(BIOS_DEBUG, "%s", (STR))
-#define print_spew(STR) printk(BIOS_SPEW, "%s", (STR))
-
-#define print_emerg_char(CH) printk(BIOS_EMERG, "%c", (CH))
-#define print_alert_char(CH) printk(BIOS_ALERT, "%c", (CH))
-#define print_crit_char(CH) printk(BIOS_CRIT, "%c", (CH))
-#define print_err_char(CH) printk(BIOS_ERR, "%c", (CH))
-#define print_warning_char(CH) printk(BIOS_WARNING,"%c", (CH))
-#define print_notice_char(CH) printk(BIOS_NOTICE, "%c", (CH))
-#define print_info_char(CH) printk(BIOS_INFO, "%c", (CH))
-#define print_debug_char(CH) printk(BIOS_DEBUG, "%c", (CH))
-#define print_spew_char(CH) printk(BIOS_SPEW, "%c", (CH))
-
-#define print_emerg_hex8(HEX) printk(BIOS_EMERG, "%02x", (HEX))
-#define print_alert_hex8(HEX) printk(BIOS_ALERT, "%02x", (HEX))
-#define print_crit_hex8(HEX) printk(BIOS_CRIT, "%02x", (HEX))
-#define print_err_hex8(HEX) printk(BIOS_ERR, "%02x", (HEX))
-#define print_warning_hex8(HEX) printk(BIOS_WARNING,"%02x", (HEX))
-#define print_notice_hex8(HEX) printk(BIOS_NOTICE, "%02x", (HEX))
-#define print_info_hex8(HEX) printk(BIOS_INFO, "%02x", (HEX))
-#define print_debug_hex8(HEX) printk(BIOS_DEBUG, "%02x", (HEX))
-#define print_spew_hex8(HEX) printk(BIOS_SPEW, "%02x", (HEX))
-
-#define print_emerg_hex16(HEX) printk(BIOS_EMERG, "%04x", (HEX))
-#define print_alert_hex16(HEX) printk(BIOS_ALERT, "%04x", (HEX))
-#define print_crit_hex16(HEX) printk(BIOS_CRIT, "%04x", (HEX))
-#define print_err_hex16(HEX) printk(BIOS_ERR, "%04x", (HEX))
-#define print_warning_hex16(HEX) printk(BIOS_WARNING,"%04x", (HEX))
-#define print_notice_hex16(HEX) printk(BIOS_NOTICE, "%04x", (HEX))
-#define print_info_hex16(HEX) printk(BIOS_INFO, "%04x", (HEX))
-#define print_debug_hex16(HEX) printk(BIOS_DEBUG, "%04x", (HEX))
-#define print_spew_hex16(HEX) printk(BIOS_SPEW, "%04x", (HEX))
-
-#define print_emerg_hex32(HEX) printk(BIOS_EMERG, "%08x", (HEX))
-#define print_alert_hex32(HEX) printk(BIOS_ALERT, "%08x", (HEX))
-#define print_crit_hex32(HEX) printk(BIOS_CRIT, "%08x", (HEX))
-#define print_err_hex32(HEX) printk(BIOS_ERR, "%08x", (HEX))
-#define print_warning_hex32(HEX) printk(BIOS_WARNING,"%08x", (HEX))
-#define print_notice_hex32(HEX) printk(BIOS_NOTICE, "%08x", (HEX))
-#define print_info_hex32(HEX) printk(BIOS_INFO, "%08x", (HEX))
-#define print_debug_hex32(HEX) printk(BIOS_DEBUG, "%08x", (HEX))
-#define print_spew_hex32(HEX) printk(BIOS_SPEW, "%08x", (HEX))
-
-#endif
-
#endif /* __CONSOLE_EARLY_PRINT_H_ */
diff --git a/src/include/console/streams.h b/src/include/console/streams.h
index fb168da905..892ae40f1c 100644
--- a/src/include/console/streams.h
+++ b/src/include/console/streams.h
@@ -28,11 +28,4 @@ void gdb_tx_byte(unsigned char byte);
void gdb_tx_flush(void);
unsigned char gdb_rx_byte(void);
-/* Helpers for ROMCC console. */
-void console_tx_nibble(unsigned nibble);
-void console_tx_hex8(unsigned char value);
-void console_tx_hex16(unsigned short value);
-void console_tx_hex32(unsigned int value);
-void console_tx_string(const char *str);
-
#endif /* _CONSOLE_STREAMS_H_ */