[coreboot-gerrit] Patch set updated for coreboot: df82ada cygnus: enable serial driver for depthcharge

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Mon Apr 20 15:40:58 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/9844

-gerrit

commit df82ada80b4355ec5cb310078e52ba6fdc534fbb
Author: Daisuke Nojiri <dnojiri at chromium.org>
Date:   Wed Feb 18 14:57:34 2015 -0800

    cygnus: enable serial driver for depthcharge
    
    BUG=chrome-os-partner:35807
    BRANCH=broadcom-firmware
    TEST=tested input and output by the cli in depthcharge
    
    Change-Id: Icd517a366a3fe5bdcf9e9aeb4284d7e4378ee3f2
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 02c3b6c82d9d71beb19a0879591c754fa7685362
    Original-Signed-off-by: Daisuke Nojiri <dnojiri at chromium.org>
    Original-Reviewed-on: https://chrome-internal-review.googlesource.com/200568
    Original-Reviewed-by: Julius Werner <jwerner at chromium.org>
    Original-Reviewed-by: Scott Branden <sbranden at broadcom.com>
    Original-Commit-Queue: Daisuke Nojiri <dnojiri at google.com>
    Original-Tested-by: Daisuke Nojiri <dnojiri at google.com>
    Original-Change-Id: I97144fc6f19969b8cf3872939c9c74efaef3604b
    Original-Reviewed-on: https://chromium-review.googlesource.com/256413
---
 payloads/libpayload/Config.in               |  5 ---
 payloads/libpayload/configs/config.purin    |  2 +-
 payloads/libpayload/drivers/Makefile.inc    |  1 -
 payloads/libpayload/drivers/serial/cygnus.c | 49 -----------------------------
 4 files changed, 1 insertion(+), 56 deletions(-)

diff --git a/payloads/libpayload/Config.in b/payloads/libpayload/Config.in
index ed6d91c..455d058 100644
--- a/payloads/libpayload/Config.in
+++ b/payloads/libpayload/Config.in
@@ -214,11 +214,6 @@ config BG4CD_SERIAL_CONSOLE
 	depends on SERIAL_CONSOLE
 	default n
 
-config CYGNUS_SERIAL_CONSOLE
-	bool "Serial port driver for Cygnus"
-	depends on SERIAL_CONSOLE
-	default n
-
 config PL011_SERIAL_CONSOLE
 	bool "PL011 compatible serial port driver"
 	depends on 8250_SERIAL_CONSOLE
diff --git a/payloads/libpayload/configs/config.purin b/payloads/libpayload/configs/config.purin
index 9482eb6..e2d0d7c 100644
--- a/payloads/libpayload/configs/config.purin
+++ b/payloads/libpayload/configs/config.purin
@@ -32,7 +32,7 @@ CONFIG_LP_CBMEM_CONSOLE=y
 CONFIG_LP_SERIAL_CONSOLE=y
 # CONFIG_LP_8250_SERIAL_CONSOLE is not set
 # CONFIG_LP_S5P_SERIAL_CONSOLE is not set
-# CONFIG_LP_8250_MMIO32_SERIAL_CONSOLE is not set
+CONFIG_LP_8250_MMIO32_SERIAL_CONSOLE=y
 # CONFIG_LP_IPQ806X_SERIAL_CONSOLE is not set
 # CONFIG_LP_BG4CD_SERIAL_CONSOLE is not set
 # CONFIG_LP_SERIAL_SET_SPEED is not set
diff --git a/payloads/libpayload/drivers/Makefile.inc b/payloads/libpayload/drivers/Makefile.inc
index b5f6c16..61b0034 100644
--- a/payloads/libpayload/drivers/Makefile.inc
+++ b/payloads/libpayload/drivers/Makefile.inc
@@ -38,7 +38,6 @@ libc-$(CONFIG_LP_S5P_SERIAL_CONSOLE) += serial/s5p.c
 libc-$(CONFIG_LP_8250_MMIO32_SERIAL_CONSOLE) += serial/8250_mmio32.c
 libc-$(CONFIG_LP_IPQ806X_SERIAL_CONSOLE) += serial/ipq806x.c
 libc-$(CONFIG_LP_BG4CD_SERIAL_CONSOLE) += serial/bg4cd.c
-libc-$(CONFIG_LP_CYGNUS_SERIAL_CONSOLE) += serial/cygnus.c
 libc-$(CONFIG_LP_PC_KEYBOARD) += keyboard.c
 
 libc-$(CONFIG_LP_CBMEM_CONSOLE) += cbmem_console.c
diff --git a/payloads/libpayload/drivers/serial/cygnus.c b/payloads/libpayload/drivers/serial/cygnus.c
deleted file mode 100644
index d822f10..0000000
--- a/payloads/libpayload/drivers/serial/cygnus.c
+++ /dev/null
@@ -1,49 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright (C) 2015 Google Electronics
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
- */
-#include <libpayload-config.h>
-#include <libpayload.h>
-
-void serial_putchar(unsigned int c)
-{
-}
-
-int serial_havechar(void)
-{
-	return 0;
-}
-
-int serial_getchar(void)
-{
-	return 0;
-}
-
-static struct console_input_driver consin = {
-	.havekey = &serial_havechar,
-	.getchar = &serial_getchar
-};
-
-static struct console_output_driver consout = {
-	.putchar = &serial_putchar
-};
-
-void serial_console_init(void)
-{
-	console_add_input_driver(&consin);
-	console_add_output_driver(&consout);
-}



More information about the coreboot-gerrit mailing list