[coreboot-gerrit] Patch set updated for coreboot: ee8f865 Google/Snow: Clean up: remove unnecessary initialization in boot block.

Hung-Te Lin (hungte@chromium.org) gerrit at coreboot.org
Mon Apr 29 16:37:00 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/3147


commit ee8f8656c4bbc7517a3aba196bcde2ce5a8ef683
Author: Hung-Te Lin <hungte at chromium.org>
Date:   Mon Apr 29 22:20:35 2013 +0800

    Google/Snow: Clean up: remove unnecessary initialization in boot block.
    The SPI1 interface is already initialized by iROM.
    The console_init has already printed some initialization message, so the "UART
    initialized" message is also not required.
    Verified by building and booting firmware image on Google/Snow successfully.
    Change-Id: I89390506aa825397c0d7e52ad7503f1cb808f7db
    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..4b3553e 100644
--- a/src/mainboard/google/snow/bootblock.c
+++ b/src/mainboard/google/snow/bootblock.c
@@ -40,12 +40,10 @@ void bootblock_mainboard_init(void)
 			 * We want to do this as early as we can.
-			exynos_pinmux_config(PERIPH_ID_SPI1, PINMUX_FLAG_NONE);
 	exynos_pinmux_config(PERIPH_ID_UART3, PINMUX_FLAG_NONE);
-	printk(BIOS_INFO, "\n\n\n%s: UART initialized\n", __func__);

More information about the coreboot-gerrit mailing list