Stefan Reinauer (stefan.reinauer@coreboot.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14917
-gerrit
commit 5a32d0b38add72169cebb648fa871c5cf4a07e9b Author: Akshay Saraswat akshay.s@samsung.com Date: Fri Sep 5 11:27:51 2014 +0530
libpayload: Jazz: Enable serial console
Ramstage should share some data with depthcharge and libpayload. But in case of Jazz we do not wish to share any buffer and just want to enable serial console and for that we ask UART's base address from config.
BUG=None BRANCH=None TEST=Saw prints over console from depthcharge
Change-Id: Iea29572b830bdf617dfbf253151a30bbcbc2a69c Signed-off-by: Akshay Saraswat akshay.s@samsung.com --- payloads/libpayload/drivers/serial/s5p.c | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/payloads/libpayload/drivers/serial/s5p.c b/payloads/libpayload/drivers/serial/s5p.c index 1d23352..cf9a537 100644 --- a/payloads/libpayload/drivers/serial/s5p.c +++ b/payloads/libpayload/drivers/serial/s5p.c @@ -25,9 +25,13 @@ * SUCH DAMAGE. */
+#include <arch/io.h> #include <libpayload.h> +#include <libpayload-config.h> #include <stdint.h>
+#include "config.h" + struct s5p_uart { uint32_t ulcon; // line control @@ -89,10 +93,14 @@ static struct console_input_driver s5p_serial_input =
void serial_init(void) { - if (!lib_sysinfo.serial || !lib_sysinfo.serial->baseaddr) - return; - - uart_regs = (struct s5p_uart *)lib_sysinfo.serial->baseaddr; + if (CONFIG_LP_S5P_SERIAL_CONSOLE_ADDRESS) + uart_regs = (struct s5p_uart *) (uintptr_t) + CONFIG_LP_S5P_SERIAL_CONSOLE_ADDRESS; + else if (lib_sysinfo.serial && lib_sysinfo.serial->baseaddr) + uart_regs = (struct s5p_uart *) (uintptr_t) + lib_sysinfo.serial->baseaddr; + + return; }
void serial_console_init(void)