From ae2cf495084325c6f4a0cb87b96167def77a80cc Mon Sep 17 00:00:00 2001 From: Furquan Shaikh Date: Wed, 21 Nov 2018 14:02:59 -0800 Subject: mb/google/octopus: Update GPIO_178 in early_gpio_table in baseboard This change updates the configuration of GPIO_178 to be active low as per latest revision on different octopus variants. This effectively: 1. Gets rid of early_gpio_table in different variants -- phaser, meep, fleex, bobba. 2. Deprecates board id < 2 for bobba, board id < 1 for fleex and phaser. 3. Adds special early_gpio_table in yorp which has GPIO_178 as an active high signal. BUG=b:119885949 Change-Id: I024199a8f1f96db57f8fa60c4d265789cd3a0493 Signed-off-by: Furquan Shaikh Reviewed-on: https://review.coreboot.org/c/29784 Tested-by: build bot (Jenkins) Reviewed-by: Karthik Ramasubramanian Reviewed-by: Aaron Durbin Reviewed-by: Justin TerAvest --- .../google/octopus/variants/baseboard/gpio.c | 2 +- src/mainboard/google/octopus/variants/bobba/gpio.c | 45 ------------------ src/mainboard/google/octopus/variants/fleex/gpio.c | 45 ------------------ src/mainboard/google/octopus/variants/meep/gpio.c | 36 --------------- .../google/octopus/variants/phaser/gpio.c | 45 ------------------ .../google/octopus/variants/yorp/Makefile.inc | 1 + src/mainboard/google/octopus/variants/yorp/gpio.c | 53 ++++++++++++++++++++++ 7 files changed, 55 insertions(+), 172 deletions(-) create mode 100644 src/mainboard/google/octopus/variants/yorp/Makefile.inc create mode 100644 src/mainboard/google/octopus/variants/yorp/gpio.c (limited to 'src') diff --git a/src/mainboard/google/octopus/variants/baseboard/gpio.c b/src/mainboard/google/octopus/variants/baseboard/gpio.c index 28dfaef25c..877cd767d4 100644 --- a/src/mainboard/google/octopus/variants/baseboard/gpio.c +++ b/src/mainboard/google/octopus/variants/baseboard/gpio.c @@ -308,7 +308,7 @@ static const struct pad_config early_gpio_table[] = { PAD_CFG_NF(GPIO_83, NONE, DEEP, NF1), /* H1_SLAVE_SPI_MOSI_R */ /* Enable power to wifi early in bootblock and de-assert PERST#. */ - PAD_CFG_GPO(GPIO_178, 1, DEEP), /* EN_PP3300_WLAN */ + PAD_CFG_GPO(GPIO_178, 0, DEEP), /* EN_PP3300_WLAN_L */ PAD_CFG_GPO(GPIO_164, 0, DEEP), /* WLAN_PE_RST */ /* diff --git a/src/mainboard/google/octopus/variants/bobba/gpio.c b/src/mainboard/google/octopus/variants/bobba/gpio.c index 6b471e5061..1b5c0bb069 100644 --- a/src/mainboard/google/octopus/variants/bobba/gpio.c +++ b/src/mainboard/google/octopus/variants/bobba/gpio.c @@ -49,48 +49,3 @@ const struct pad_config *variant_override_gpio_table(size_t *num) } return c; } - -/* GPIOs needed prior to ramstage. */ -static const struct pad_config early_gpio_table[] = { - /* PCH_WP_OD */ - PAD_CFG_GPI(GPIO_190, NONE, DEEP), - /* H1_PCH_INT_ODL */ - PAD_CFG_GPI_APIC_IOS(GPIO_63, NONE, DEEP, LEVEL, INVERT, TxDRxE, - DISPUPD), - /* H1_SLAVE_SPI_CLK_R */ - PAD_CFG_NF(GPIO_79, NONE, DEEP, NF1), - /* H1_SLAVE_SPI_CS_L_R */ - PAD_CFG_NF(GPIO_80, NONE, DEEP, NF1), - /* H1_SLAVE_SPI_MISO */ - PAD_CFG_NF(GPIO_82, NONE, DEEP, NF1), - /* H1_SLAVE_SPI_MOSI_R */ - PAD_CFG_NF(GPIO_83, NONE, DEEP, NF1), - - /* WLAN_PE_RST */ - PAD_CFG_GPO(GPIO_164, 0, DEEP), - - /* - * ESPI_IO1 acts as ALERT# (which is open-drain) and requies a weak - * pull-up for proper operation. Since there is no external pull present - * on this platform, configure an internal weak pull-up. - */ - PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_151, UP_20K, DEEP, NF2, HIZCRx1, - ENPU), -}; - -const struct pad_config *variant_early_gpio_table(size_t *num) -{ - /* - * This is a hack to configure EN_PP3300_WLAN based on board id. Once - * board id 0/1 is deprecated, we can get rid of this. - */ - uint32_t bid = board_id(); - - if (bid == UNDEFINED_STRAPPING_ID || bid < 2) - gpio_output(GPIO_178, 1); - else - gpio_output(GPIO_178, 0); - - *num = ARRAY_SIZE(early_gpio_table); - return early_gpio_table; -} diff --git a/src/mainboard/google/octopus/variants/fleex/gpio.c b/src/mainboard/google/octopus/variants/fleex/gpio.c index ea7faf3ae6..920d2c26b0 100644 --- a/src/mainboard/google/octopus/variants/fleex/gpio.c +++ b/src/mainboard/google/octopus/variants/fleex/gpio.c @@ -86,48 +86,3 @@ const struct pad_config *variant_override_gpio_table(size_t *num) return c; } - -/* GPIOs needed prior to ramstage. */ -static const struct pad_config early_gpio_table[] = { - /* PCH_WP_OD */ - PAD_CFG_GPI(GPIO_190, NONE, DEEP), - /* H1_PCH_INT_ODL */ - PAD_CFG_GPI_APIC_IOS(GPIO_63, NONE, DEEP, LEVEL, INVERT, TxDRxE, - DISPUPD), - /* H1_SLAVE_SPI_CLK_R */ - PAD_CFG_NF(GPIO_79, NONE, DEEP, NF1), - /* H1_SLAVE_SPI_CS_L_R */ - PAD_CFG_NF(GPIO_80, NONE, DEEP, NF1), - /* H1_SLAVE_SPI_MISO */ - PAD_CFG_NF(GPIO_82, NONE, DEEP, NF1), - /* H1_SLAVE_SPI_MOSI_R */ - PAD_CFG_NF(GPIO_83, NONE, DEEP, NF1), - - /* WLAN_PE_RST */ - PAD_CFG_GPO(GPIO_164, 0, DEEP), - - /* - * ESPI_IO1 acts as ALERT# (which is open-drain) and requies a weak - * pull-up for proper operation. Since there is no external pull present - * on this platform, configure an internal weak pull-up. - */ - PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_151, UP_20K, DEEP, NF2, HIZCRx1, - ENPU), -}; - -const struct pad_config *variant_early_gpio_table(size_t *num) -{ - /* - * This is a hack to configure EN_PP3300_WLAN based on board id. Once - * board id 0 is deprecated, we can get rid of this. - */ - uint32_t bid = board_id(); - - if (bid == UNDEFINED_STRAPPING_ID || bid < 1) - gpio_output(GPIO_178, 1); - else - gpio_output(GPIO_178, 0); - - *num = ARRAY_SIZE(early_gpio_table); - return early_gpio_table; -} diff --git a/src/mainboard/google/octopus/variants/meep/gpio.c b/src/mainboard/google/octopus/variants/meep/gpio.c index 823224c808..7e572875c3 100644 --- a/src/mainboard/google/octopus/variants/meep/gpio.c +++ b/src/mainboard/google/octopus/variants/meep/gpio.c @@ -38,39 +38,3 @@ const struct pad_config *variant_override_gpio_table(size_t *num) *num = ARRAY_SIZE(default_override_table); return default_override_table; } - -/* GPIOs needed prior to ramstage. */ -static const struct pad_config early_gpio_table[] = { - /* PCH_WP_OD */ - PAD_CFG_GPI(GPIO_190, NONE, DEEP), - /* H1_PCH_INT_ODL */ - PAD_CFG_GPI_APIC_IOS(GPIO_63, NONE, DEEP, LEVEL, INVERT, TxDRxE, - DISPUPD), - /* H1_SLAVE_SPI_CLK_R */ - PAD_CFG_NF(GPIO_79, NONE, DEEP, NF1), - /* H1_SLAVE_SPI_CS_L_R */ - PAD_CFG_NF(GPIO_80, NONE, DEEP, NF1), - /* H1_SLAVE_SPI_MISO */ - PAD_CFG_NF(GPIO_82, NONE, DEEP, NF1), - /* H1_SLAVE_SPI_MOSI_R */ - PAD_CFG_NF(GPIO_83, NONE, DEEP, NF1), - - PAD_CFG_GPO(GPIO_178, 0, DEEP), - - /* WLAN_PE_RST */ - PAD_CFG_GPO(GPIO_164, 0, DEEP), - - /* - * ESPI_IO1 acts as ALERT# (which is open-drain) and requies a weak - * pull-up for proper operation. Since there is no external pull present - * on this platform, configure an internal weak pull-up. - */ - PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_151, UP_20K, DEEP, NF2, HIZCRx1, - ENPU), -}; - -const struct pad_config *variant_early_gpio_table(size_t *num) -{ - *num = ARRAY_SIZE(early_gpio_table); - return early_gpio_table; -} diff --git a/src/mainboard/google/octopus/variants/phaser/gpio.c b/src/mainboard/google/octopus/variants/phaser/gpio.c index 4d3210c6e8..2a5dfbc87d 100644 --- a/src/mainboard/google/octopus/variants/phaser/gpio.c +++ b/src/mainboard/google/octopus/variants/phaser/gpio.c @@ -101,48 +101,3 @@ const struct pad_config *variant_override_gpio_table(size_t *num) } return c; } - -/* GPIOs needed prior to ramstage. */ -static const struct pad_config early_gpio_table[] = { - /* PCH_WP_OD */ - PAD_CFG_GPI(GPIO_190, NONE, DEEP), - /* H1_PCH_INT_ODL */ - PAD_CFG_GPI_APIC_IOS(GPIO_63, NONE, DEEP, LEVEL, INVERT, TxDRxE, - DISPUPD), - /* H1_SLAVE_SPI_CLK_R */ - PAD_CFG_NF(GPIO_79, NONE, DEEP, NF1), - /* H1_SLAVE_SPI_CS_L_R */ - PAD_CFG_NF(GPIO_80, NONE, DEEP, NF1), - /* H1_SLAVE_SPI_MISO */ - PAD_CFG_NF(GPIO_82, NONE, DEEP, NF1), - /* H1_SLAVE_SPI_MOSI_R */ - PAD_CFG_NF(GPIO_83, NONE, DEEP, NF1), - - /* WLAN_PE_RST */ - PAD_CFG_GPO(GPIO_164, 0, DEEP), - - /* - * ESPI_IO1 acts as ALERT# (which is open-drain) and requies a weak - * pull-up for proper operation. Since there is no external pull present - * on this platform, configure an internal weak pull-up. - */ - PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_151, UP_20K, DEEP, NF2, HIZCRx1, - ENPU), -}; - -const struct pad_config *variant_early_gpio_table(size_t *num) -{ - /* - * This is a hack to configure EN_PP3300_WLAN based on board id. Once - * board id 0 is deprecated, we can get rid of this. - */ - uint32_t bid = board_id(); - - if (bid == UNDEFINED_STRAPPING_ID || bid < 1) - gpio_output(GPIO_178, 1); - else - gpio_output(GPIO_178, 0); - - *num = ARRAY_SIZE(early_gpio_table); - return early_gpio_table; -} diff --git a/src/mainboard/google/octopus/variants/yorp/Makefile.inc b/src/mainboard/google/octopus/variants/yorp/Makefile.inc new file mode 100644 index 0000000000..4b48156a52 --- /dev/null +++ b/src/mainboard/google/octopus/variants/yorp/Makefile.inc @@ -0,0 +1 @@ +bootblock-y += gpio.c diff --git a/src/mainboard/google/octopus/variants/yorp/gpio.c b/src/mainboard/google/octopus/variants/yorp/gpio.c new file mode 100644 index 0000000000..571988868f --- /dev/null +++ b/src/mainboard/google/octopus/variants/yorp/gpio.c @@ -0,0 +1,53 @@ +/* + * This file is part of the coreboot project. + * + * Copyright 2018 Google LLC + * + * 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. + */ + +#include +#include +#include +#include + +/* GPIOs needed prior to ramstage. */ +static const struct pad_config early_gpio_table[] = { + PAD_CFG_GPI(GPIO_190, NONE, DEEP), /* PCH_WP_OD */ + /* GSPI0_INT */ + PAD_CFG_GPI_APIC_IOS(GPIO_63, NONE, DEEP, LEVEL, INVERT, TxDRxE, + DISPUPD), /* H1_PCH_INT_ODL */ + /* GSPI0_CLK */ + PAD_CFG_NF(GPIO_79, NONE, DEEP, NF1), /* H1_SLAVE_SPI_CLK_R */ + /* GSPI0_CS# */ + PAD_CFG_NF(GPIO_80, NONE, DEEP, NF1), /* H1_SLAVE_SPI_CS_L_R */ + /* GSPI0_MISO */ + PAD_CFG_NF(GPIO_82, NONE, DEEP, NF1), /* H1_SLAVE_SPI_MISO */ + /* GSPI0_MOSI */ + PAD_CFG_NF(GPIO_83, NONE, DEEP, NF1), /* H1_SLAVE_SPI_MOSI_R */ + + /* Enable power to wifi early in bootblock and de-assert PERST#. */ + PAD_CFG_GPO(GPIO_178, 1, DEEP), /* EN_PP3300_WLAN */ + PAD_CFG_GPO(GPIO_164, 0, DEEP), /* WLAN_PE_RST */ + + /* + * ESPI_IO1 acts as ALERT# (which is open-drain) and requies a weak + * pull-up for proper operation. Since there is no external pull present + * on this platform, configure an internal weak pull-up. + */ + PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_151, UP_20K, DEEP, NF2, HIZCRx1, + ENPU), /* ESPI_IO1 */ +}; + +const struct pad_config *variant_early_gpio_table(size_t *num) +{ + *num = ARRAY_SIZE(early_gpio_table); + return early_gpio_table; +} -- cgit v1.2.3