[coreboot-gerrit] New patch to review for coreboot: 6a0cc96 Google/Snow: Remove unnecessary serial console init code.

Hung-Te Lin (hungte@chromium.org) gerrit at coreboot.org
Tue Apr 30 09:37:36 CEST 2013


Hung-Te Lin (hungte at chromium.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3150

-gerrit

commit 6a0cc96e22c3e4a99e1c1cc19d46344fa48cf453
Author: Hung-Te Lin <hungte at chromium.org>
Date:   Tue Apr 30 15:31:48 2013 +0800

    Google/Snow: Remove unnecessary serial console init code.
    
    The "console_init" does initialize UART driver (which will setup peripheral and
    pinmux) and print starting message. Duplicated initialization can be removed.
    
    Verified by building and booting on Google/Snow.
    
    Change-Id: I0c6b4d4eb1a4e81af0f65bcb032978dfb945c63d
    Signed-off-by: Hung-Te Lin <hungte at chromium.org>
---
 src/mainboard/google/snow/bootblock.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/mainboard/google/snow/bootblock.c b/src/mainboard/google/snow/bootblock.c
index 09edbad..22d775c 100644
--- a/src/mainboard/google/snow/bootblock.c
+++ b/src/mainboard/google/snow/bootblock.c
@@ -44,8 +44,6 @@ void bootblock_mainboard_init(void)
 			break;
 	}
 #if CONFIG_EARLY_CONSOLE
-	exynos_pinmux_config(PERIPH_ID_UART3, PINMUX_FLAG_NONE);
 	console_init();
-	printk(BIOS_INFO, "\n\n\n%s: UART initialized\n", __func__);
 #endif
 }



More information about the coreboot-gerrit mailing list