diff options
author | Fred Reitberger <reitbergerfred@gmail.com> | 2023-02-01 16:00:54 -0500 |
---|---|---|
committer | Felix Held <felix-coreboot@felixheld.de> | 2023-02-09 10:01:20 +0000 |
commit | a02176debb88a6e838c3b5f888010ae138b79127 (patch) | |
tree | 3b2df0135653dc16ca2a20e1d5d758d258159f60 | |
parent | d7b7460d6e8a9fbd32c41e91a9d00dc536953310 (diff) |
console: Add SimNow console logging
The AMD SimNow tool supports fast logging through an IO port. Add a new
console to support SimNow logging through port 80.
TEST=observe significant speed improvements on SimNow console log
Signed-off-by: Fred Reitberger <reitbergerfred@gmail.com>
Change-Id: I42a431f48ea14ba4adacbd4a32e15abe7c5e4951
Reviewed-on: https://review.coreboot.org/c/coreboot/+/72751
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
-rw-r--r-- | src/console/Kconfig | 8 | ||||
-rw-r--r-- | src/console/console.c | 3 | ||||
-rw-r--r-- | src/include/console/console.h | 2 | ||||
-rw-r--r-- | src/include/console/simnow.h | 28 | ||||
-rw-r--r-- | src/soc/amd/common/block/simnow/Makefile.inc | 7 | ||||
-rw-r--r-- | src/soc/amd/common/block/simnow/simnow_console.c | 24 |
6 files changed, 71 insertions, 1 deletions
diff --git a/src/console/Kconfig b/src/console/Kconfig index c6aec18b09..721cc60e2b 100644 --- a/src/console/Kconfig +++ b/src/console/Kconfig @@ -367,6 +367,14 @@ config CONSOLE_SYSTEM76_EC help Send coreboot debug output to a System76 embedded controller. +config CONSOLE_AMD_SIMNOW + bool "AMD SimNow console output" + default n + depends on SOC_AMD_COMMON && !POST_IO + select CONSOLE_POST + help + Send coreboot debug output to IO ports for SimNow + config CONSOLE_OVERRIDE_LOGLEVEL bool help diff --git a/src/console/console.c b/src/console/console.c index 499a336efb..9a079ed60a 100644 --- a/src/console/console.c +++ b/src/console/console.c @@ -5,6 +5,7 @@ #include <console/i2c_smbus.h> #include <console/ne2k.h> #include <console/qemu_debugcon.h> +#include <console/simnow.h> #include <console/spi.h> #include <console/spkmodem.h> #include <console/streams.h> @@ -28,6 +29,7 @@ void console_hw_init(void) __flashconsole_init(); __system76_ec_init(); __i2c_smbus_console_init(); + __simnow_console_init(); } void console_interactive_tx_byte(unsigned char byte, void *data_unused) @@ -47,6 +49,7 @@ void console_interactive_tx_byte(unsigned char byte, void *data_unused) __spiconsole_tx_byte(byte); __system76_ec_tx_byte(byte); __i2c_smbus_console_tx_byte(byte); + __simnow_console_tx_byte(byte); } void console_stored_tx_byte(unsigned char byte, void *data_unused) diff --git a/src/include/console/console.h b/src/include/console/console.h index 6f44d7f95c..c69efcde40 100644 --- a/src/include/console/console.h +++ b/src/include/console/console.h @@ -72,7 +72,7 @@ enum { CONSOLE_LOG_NONE = 0, CONSOLE_LOG_FAST, CONSOLE_LOG_ALL }; #define HAS_ONLY_FAST_CONSOLES !(CONFIG(SPKMODEM) || CONFIG(CONSOLE_QEMU_DEBUGCON) || \ CONFIG(CONSOLE_SERIAL) || CONFIG(CONSOLE_NE2K) || CONFIG(CONSOLE_USB) || \ CONFIG(EM100PRO_SPI_CONSOLE) || CONFIG(CONSOLE_SPI_FLASH) || \ - CONFIG(CONSOLE_SYSTEM76_EC)) + CONFIG(CONSOLE_SYSTEM76_EC) || CONFIG(CONSOLE_AMD_SIMNOW)) #else static inline int get_log_level(void) { return -1; } diff --git a/src/include/console/simnow.h b/src/include/console/simnow.h new file mode 100644 index 0000000000..9a9d6792b8 --- /dev/null +++ b/src/include/console/simnow.h @@ -0,0 +1,28 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ + +#ifndef CONSOLE_SIMNOW_H +#define CONSOLE_SIMNOW_H + +#include <stdint.h> + +void simnow_console_init(void); +void simnow_console_tx_byte(unsigned char data); + +#define __SIMNOW_CONSOLE_ENABLE__ CONFIG(CONSOLE_AMD_SIMNOW) + +#if __SIMNOW_CONSOLE_ENABLE__ +static inline void __simnow_console_init(void) +{ + simnow_console_init(); +} + +static inline void __simnow_console_tx_byte(u8 data) +{ + simnow_console_tx_byte(data); +} +#else +static inline void __simnow_console_init(void) {} +static inline void __simnow_console_tx_byte(u8 data) {} +#endif + +#endif /* CONSOLE_SIMNOW_H */ diff --git a/src/soc/amd/common/block/simnow/Makefile.inc b/src/soc/amd/common/block/simnow/Makefile.inc new file mode 100644 index 0000000000..2cd79c899f --- /dev/null +++ b/src/soc/amd/common/block/simnow/Makefile.inc @@ -0,0 +1,7 @@ +## SPDX-License-Identifier: GPL-2.0-only + +ifeq ($(CONFIG_CONSOLE_AMD_SIMNOW),y) +all-y += simnow_console.c + +smm-$(CONFIG_DEBUG_SMI) += simnow_console.c +endif diff --git a/src/soc/amd/common/block/simnow/simnow_console.c b/src/soc/amd/common/block/simnow/simnow_console.c new file mode 100644 index 0000000000..478674ad1f --- /dev/null +++ b/src/soc/amd/common/block/simnow/simnow_console.c @@ -0,0 +1,24 @@ +/* SPDX-License-Identifier: GPL-2.0-only */ + +#include <arch/io.h> +#include <console/console.h> +#include <console/simnow.h> + +#define AMD_SIMNOW_PORT 0x80 +#define AMD_SIMNOW_PORT_DATA_BEGIN 0x5f535452ul +#define AMD_SIMNOW_PORT_DATA_END 0x5f454e44ul + +void simnow_console_init(void) +{ + outl(AMD_SIMNOW_PORT_DATA_BEGIN, AMD_SIMNOW_PORT); +} + +void simnow_console_tx_byte(unsigned char data) +{ + outb(data, AMD_SIMNOW_PORT); + + if (data == '\n') { + outl(AMD_SIMNOW_PORT_DATA_END, AMD_SIMNOW_PORT); + outl(AMD_SIMNOW_PORT_DATA_BEGIN, AMD_SIMNOW_PORT); + } +} |