<p>Shamile Khan has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/28328">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mb/google/octopus: Add missing standby settings.<br><br>Also removed internal pull ups for CX_PREQ_L and<br>CX_PREQ_L signals as they have external pull ups.<br><br>BUG=b:110654510<br>TEST=On Yorp Proto 2, flashed image and verified that it boots to OS.<br>Also executed a few suspend resume cycles.<br><br>Change-Id: I0a76cd2a1481c828fc092aaf7e870a411624879c<br>Signed-off-by: Shamile Khan <shamile.khan@intel.com><br>---<br>M src/mainboard/google/octopus/variants/baseboard/gpio.c<br>M src/mainboard/google/octopus/variants/bip/gpio.c<br>M src/mainboard/google/octopus/variants/bobba/gpio.c<br>M src/mainboard/google/octopus/variants/fleex/gpio.c<br>M src/mainboard/google/octopus/variants/phaser/gpio.c<br>5 files changed, 13 insertions(+), 13 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/28/28328/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/mainboard/google/octopus/variants/baseboard/gpio.c b/src/mainboard/google/octopus/variants/baseboard/gpio.c</span><br><span>index 64cfe4d..2942601 100644</span><br><span>--- a/src/mainboard/google/octopus/variants/baseboard/gpio.c</span><br><span>+++ b/src/mainboard/google/octopus/variants/baseboard/gpio.c</span><br><span>@@ -32,8 +32,8 @@</span><br><span>    PAD_CFG_NF_IOSTANDBY_IGNORE(GPIO_3, NONE, DEEP, NF1), /* TDI */</span><br><span>      PAD_CFG_NF_IOSTANDBY_IGNORE(GPIO_4, NONE, DEEP, NF1), /* TDO */</span><br><span>      PAD_NC(GPIO_5, UP_20K), /* JTAGX -- unused */</span><br><span style="color: hsl(0, 100%, 40%);">-   PAD_CFG_NF_IOSTANDBY_IGNORE(GPIO_6, UP_20K, DEEP, NF1), /* CX_PREQ_B */</span><br><span style="color: hsl(0, 100%, 40%);">- PAD_CFG_NF_IOSTANDBY_IGNORE(GPIO_7, UP_20K, DEEP, NF1), /* CX_PRDY_B */</span><br><span style="color: hsl(120, 100%, 40%);">+       PAD_CFG_NF_IOSTANDBY_IGNORE(GPIO_6, NONE, DEEP, NF1), /* CX_PREQ_B */</span><br><span style="color: hsl(120, 100%, 40%);">+ PAD_CFG_NF_IOSTANDBY_IGNORE(GPIO_7, NONE, DEEP, NF1), /* CX_PRDY_B */</span><br><span>        PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_8, DN_20K, DEEP, NF5, HIZCRx0, DISPUPD), /* TRACE_0_CLK_VNN */</span><br><span>      PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_9, DN_20K, DEEP, NF5, HIZCRx0, DISPUPD), /* TRACE_0_DATA0_VNN */</span><br><span>    PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_10, DN_20K, DEEP, NF5, HIZCRx0, DISPUPD), /* TRACE_0_DATA1_VNN */</span><br><span>@@ -117,8 +117,8 @@</span><br><span>     PAD_NC(GPIO_78, UP_20K),/* SVID Clk - unused */</span><br><span> </span><br><span>  /* LPSS */</span><br><span style="color: hsl(0, 100%, 40%);">-      PAD_CFG_NF(GPIO_79, NONE, DEEP, NF1), /* H1_SLAVE_SPI_CLK_R */</span><br><span style="color: hsl(0, 100%, 40%);">-  PAD_CFG_NF(GPIO_80, NONE, DEEP, NF1), /* H1_SLAVE_SPI_CS_L_R */</span><br><span style="color: hsl(120, 100%, 40%);">+       PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_79, NONE, DEEP, NF1, HIZCRx0, DISPUPD), /* H1_SLAVE_SPI_CLK_R */</span><br><span style="color: hsl(120, 100%, 40%);">+     PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_80, NONE, DEEP, NF1, HIZCRx1, DISPUPD), /* H1_SLAVE_SPI_CS_L_R */</span><br><span>   PAD_CFG_GPIO_HI_Z(GPIO_81, UP_20K, DEEP, HIZCRx0, DISPUPD), /* GPIO_81_DEBUG (Boot halt) -- MIPI60 DEBUG */</span><br><span>  PAD_CFG_NF(GPIO_82, NONE, DEEP, NF1), /* H1_SLAVE_SPI_MISO */</span><br><span>        PAD_CFG_NF(GPIO_83, NONE, DEEP, NF1), /* H1_SLAVE_SPI_MOSI_R */</span><br><span>@@ -244,7 +244,7 @@</span><br><span>        /* SCC COMMUNITY GPIOS */</span><br><span>    PAD_NC(GPIO_176, UP_20K), /* SMB_ALERTB -- unused */</span><br><span>         PAD_NC(GPIO_177, UP_20K), /* SMB_CLK -- unused */</span><br><span style="color: hsl(0, 100%, 40%);">-       PAD_CFG_GPO(GPIO_178, 1, DEEP), /* EN_PP3300_WLAN */</span><br><span style="color: hsl(120, 100%, 40%);">+  PAD_CFG_GPO_IOSSTATE_IOSTERM(GPIO_178, 1, DEEP, NONE, Tx1RXDCRx0, DISPUPD), /* EN_PP3300_WLAN */</span><br><span>     PAD_NC(GPIO_179, NONE), /* SDCARD_CLK -- unused */</span><br><span>   PAD_NC(GPIO_180, NONE), /* SDCARD_CMD -- unused */</span><br><span>   PAD_NC(GPIO_181, UP_20K), /* SDCARD_D0 -- unused */</span><br><span>diff --git a/src/mainboard/google/octopus/variants/bip/gpio.c b/src/mainboard/google/octopus/variants/bip/gpio.c</span><br><span>index d929284..647a903 100644</span><br><span>--- a/src/mainboard/google/octopus/variants/bip/gpio.c</span><br><span>+++ b/src/mainboard/google/octopus/variants/bip/gpio.c</span><br><span>@@ -30,8 +30,8 @@</span><br><span>         PAD_CFG_NF_IOSTANDBY_IGNORE(GPIO_3, NONE, DEEP, NF1), /* TDI */</span><br><span>      PAD_CFG_NF_IOSTANDBY_IGNORE(GPIO_4, NONE, DEEP, NF1), /* TDO */</span><br><span>      PAD_NC(GPIO_5, UP_20K), /* JTAGX -- unused */</span><br><span style="color: hsl(0, 100%, 40%);">-   PAD_CFG_NF_IOSTANDBY_IGNORE(GPIO_6, UP_20K, DEEP, NF1), /* CX_PREQ_B */</span><br><span style="color: hsl(0, 100%, 40%);">- PAD_CFG_NF_IOSTANDBY_IGNORE(GPIO_7, UP_20K, DEEP, NF1), /* CX_PRDY_B */</span><br><span style="color: hsl(120, 100%, 40%);">+       PAD_CFG_NF_IOSTANDBY_IGNORE(GPIO_6, NONE, DEEP, NF1), /* CX_PREQ_B */</span><br><span style="color: hsl(120, 100%, 40%);">+ PAD_CFG_NF_IOSTANDBY_IGNORE(GPIO_7, NONE, DEEP, NF1), /* CX_PRDY_B */</span><br><span>        PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_8, DN_20K, DEEP, NF5, HIZCRx0, DISPUPD), /* TRACE_0_CLK_VNN */</span><br><span>      PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_9, DN_20K, DEEP, NF5, HIZCRx0, DISPUPD), /* TRACE_0_DATA0_VNN */</span><br><span>    PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_10, DN_20K, DEEP, NF5, HIZCRx0, DISPUPD), /* TRACE_0_DATA1_VNN */</span><br><span>@@ -115,8 +115,8 @@</span><br><span>     PAD_NC(GPIO_78, UP_20K),/* SVID Clk - unused */</span><br><span> </span><br><span>  /* LPSS */</span><br><span style="color: hsl(0, 100%, 40%);">-      PAD_CFG_NF(GPIO_79, NONE, DEEP, NF1), /* H1_SLAVE_SPI_CLK_R */</span><br><span style="color: hsl(0, 100%, 40%);">-  PAD_CFG_NF(GPIO_80, NONE, DEEP, NF1), /* H1_SLAVE_SPI_CS_L_R */</span><br><span style="color: hsl(120, 100%, 40%);">+       PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_79, NONE, DEEP, NF1, HIZCRx0, DISPUPD), /* H1_SLAVE_SPI_CLK_R */</span><br><span style="color: hsl(120, 100%, 40%);">+     PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_80, NONE, DEEP, NF1, HIZCRx1, DISPUPD), /* H1_SLAVE_SPI_CS_L_R */</span><br><span>   PAD_CFG_GPIO_HI_Z(GPIO_81, UP_20K, DEEP, HIZCRx0, DISPUPD), /* GPIO_81_DEBUG (Boot halt) -- MIPI60 DEBUG */</span><br><span>  PAD_CFG_NF(GPIO_82, NONE, DEEP, NF1), /* H1_SLAVE_SPI_MISO */</span><br><span>        PAD_CFG_NF(GPIO_83, NONE, DEEP, NF1), /* H1_SLAVE_SPI_MOSI_R */</span><br><span>@@ -243,7 +243,7 @@</span><br><span>        /* SCC COMMUNITY GPIOS */</span><br><span>    PAD_NC(GPIO_176, UP_20K), /* SMB_ALERTB -- unused */</span><br><span>         PAD_NC(GPIO_177, UP_20K), /* SMB_CLK -- unused */</span><br><span style="color: hsl(0, 100%, 40%);">-       PAD_CFG_GPO(GPIO_178, 1, DEEP), /* EN_PP3300_WLAN */</span><br><span style="color: hsl(120, 100%, 40%);">+  PAD_CFG_GPO_IOSSTATE_IOSTERM(GPIO_178, 1, DEEP, NONE, Tx1RXDCRx0, DISPUPD), /* EN_PP3300_WLAN */</span><br><span>     PAD_NC(GPIO_179, NONE), /* SDCARD_CLK -- unused */</span><br><span>   PAD_NC(GPIO_180, NONE), /* SDCARD_CMD -- unused */</span><br><span>   PAD_NC(GPIO_181, UP_20K), /* SDCARD_D0 -- unused */</span><br><span>diff --git a/src/mainboard/google/octopus/variants/bobba/gpio.c b/src/mainboard/google/octopus/variants/bobba/gpio.c</span><br><span>index 20e2230..8ce2c51 100644</span><br><span>--- a/src/mainboard/google/octopus/variants/bobba/gpio.c</span><br><span>+++ b/src/mainboard/google/octopus/variants/bobba/gpio.c</span><br><span>@@ -31,7 +31,7 @@</span><br><span>                                      DISPUPD),</span><br><span> </span><br><span>   /* EN_PP3300_WLAN_L */</span><br><span style="color: hsl(0, 100%, 40%);">-  PAD_CFG_GPO(GPIO_178, 0, DEEP),</span><br><span style="color: hsl(120, 100%, 40%);">+       PAD_CFG_GPO_IOSSTATE_IOSTERM(GPIO_178, 0, DEEP, NONE, Tx0RxDCRx0, DISPUPD),</span><br><span> };</span><br><span> </span><br><span> const struct pad_config *variant_override_gpio_table(size_t *num)</span><br><span>diff --git a/src/mainboard/google/octopus/variants/fleex/gpio.c b/src/mainboard/google/octopus/variants/fleex/gpio.c</span><br><span>index ac68569..741eeaf 100644</span><br><span>--- a/src/mainboard/google/octopus/variants/fleex/gpio.c</span><br><span>+++ b/src/mainboard/google/octopus/variants/fleex/gpio.c</span><br><span>@@ -46,7 +46,7 @@</span><br><span>  PAD_NC(GPIO_161, UP_20K),</span><br><span> </span><br><span>        /* EN_PP3300_WLAN_L */</span><br><span style="color: hsl(0, 100%, 40%);">-  PAD_CFG_GPO(GPIO_178, 0, DEEP),</span><br><span style="color: hsl(120, 100%, 40%);">+       PAD_CFG_GPO_IOSSTATE_IOSTERM(GPIO_178, 0, DEEP, NONE, Tx0RxDCRx0, DISPUPD),</span><br><span> </span><br><span>      PAD_NC(GPIO_213, DN_20K),</span><br><span>    PAD_NC(GPIO_214, DN_20K),</span><br><span>diff --git a/src/mainboard/google/octopus/variants/phaser/gpio.c b/src/mainboard/google/octopus/variants/phaser/gpio.c</span><br><span>index 11935d9..fd8777b 100644</span><br><span>--- a/src/mainboard/google/octopus/variants/phaser/gpio.c</span><br><span>+++ b/src/mainboard/google/octopus/variants/phaser/gpio.c</span><br><span>@@ -33,7 +33,7 @@</span><br><span>       PAD_NC(GPIO_161, DN_20K),</span><br><span> </span><br><span>        /* EN_PP3300_WLAN_L */</span><br><span style="color: hsl(0, 100%, 40%);">-  PAD_CFG_GPO(GPIO_178, 0, DEEP),</span><br><span style="color: hsl(120, 100%, 40%);">+       PAD_CFG_GPO_IOSSTATE_IOSTERM(GPIO_178, 0, DEEP, NONE, Tx0RxDCRx0, DISPUPD),</span><br><span> </span><br><span>      PAD_NC(GPIO_213, DN_20K),</span><br><span>    PAD_NC(GPIO_214, DN_20K),</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/28328">change 28328</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/28328"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I0a76cd2a1481c828fc092aaf7e870a411624879c </div>
<div style="display:none"> Gerrit-Change-Number: 28328 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Shamile Khan <shamile.khan@intel.com> </div>