[coreboot-gerrit] New patch to review for coreboot: libpayload: Jazz: Enable serial console

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Thu May 19 20:39:06 CEST 2016


Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14917

-gerrit

commit a2e89483249cef724e8512c4293d22ed727c7717
Author: Akshay Saraswat <akshay.s at 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 at 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)



More information about the coreboot-gerrit mailing list