From bb75effd87f9f86fa612dcfa1aa7b6156d110f01 Mon Sep 17 00:00:00 2001 From: Justin TerAvest Date: Wed, 28 Feb 2018 11:42:38 -0700 Subject: [PATCH] soc/amd/stoneyridge: Remove printk for GPIO The printk() calls in sb_program_gpios() aren't necessary, and incur a 13 second delay if the function is called from bootblock_mainboard_early_init(). This commit removes them so GPIOs can be set up earlier. TEST=call sb_program_gpios from bootblock_mainboard_early_init BUG=b:73898539 Change-Id: I064291decf47d86132e36469e029b3262ec20172 Signed-off-by: Justin TerAvest Reviewed-on: https://review.coreboot.org/24915 Reviewed-by: Aaron Durbin Tested-by: build bot (Jenkins) --- src/soc/amd/stoneyridge/include/soc/southbridge.h | 1 - src/soc/amd/stoneyridge/southbridge.c | 2 -- 2 files changed, 3 deletions(-) diff --git a/src/soc/amd/stoneyridge/include/soc/southbridge.h b/src/soc/amd/stoneyridge/include/soc/southbridge.h index 3489afc0f9..3011af27f1 100644 --- a/src/soc/amd/stoneyridge/include/soc/southbridge.h +++ b/src/soc/amd/stoneyridge/include/soc/southbridge.h @@ -288,7 +288,6 @@ #else #define GPIO_TABLE_BOOTBLOCK 0 #endif -#define STR_GPIO_STAGE ENV_STRING struct soc_amd_stoneyridge_gpio { uint8_t gpio; diff --git a/src/soc/amd/stoneyridge/southbridge.c b/src/soc/amd/stoneyridge/southbridge.c index 5d7b73d1a6..3d0cc74779 100644 --- a/src/soc/amd/stoneyridge/southbridge.c +++ b/src/soc/amd/stoneyridge/southbridge.c @@ -161,7 +161,6 @@ void sb_program_gpios(const struct soc_amd_stoneyridge_gpio *gpio_ptr, void *tmp_ptr; uint8_t control, mux, index; - printk(BIOS_SPEW, "GPIO programming stage %s\n", STR_GPIO_STAGE); for (index = 0; index < size; index++) { mux = gpio_ptr[index].function; control = gpio_ptr[index].control; @@ -176,7 +175,6 @@ void sb_program_gpios(const struct soc_amd_stoneyridge_gpio *gpio_ptr, AMD_GPIO_CONTROL + 2); write8(tmp_ptr, control); } - printk(BIOS_SPEW, "End GPIO programming\n"); } /**