Hello Srinidhi N Kaushik, Shamile Khan, Bora Guvendik, build bot (Jenkins), Hannah Williams,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/24909
to look at the new patch set (#2).
Change subject: mb/intel/glkrvp: Configure PERST GPIO for PCIE WiFi card
......................................................................
mb/intel/glkrvp: Configure PERST GPIO for PCIE WiFi card
PERST signal is required for WiFi.
When the system resumes from S0ix, WiFi will come out of D3 correctly.
Change-Id: Ic594c5a088af79079a8742f5287ed66dcc0ff067
Signed-off-by: Roy Mingi Park <roy.mingi.park(a)intel.com>
---
M src/mainboard/intel/glkrvp/variants/baseboard/gpio.c
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/09/24909/2
--
To view, visit https://review.coreboot.org/24909
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic594c5a088af79079a8742f5287ed66dcc0ff067
Gerrit-Change-Number: 24909
Gerrit-PatchSet: 2
Gerrit-Owner: Roy Mingi Park <roy.mingi.park(a)intel.com>
Gerrit-Reviewer: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-Reviewer: Hannah Williams <hannah.williams(a)intel.com>
Gerrit-Reviewer: Shamile Khan <shamile.khan(a)intel.com>
Gerrit-Reviewer: Srinidhi N Kaushik <srinidhi.n.kaushik(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Furquan Shaikh <furquan(a)google.com>
Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/24909 )
Change subject: src/mainboard/intel: Configure PERST GPIO for PCIE WiFi card
......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/24909/1//COMMIT_MSG
Commit Message:
https://review.coreboot.org/#/c/24909/1//COMMIT_MSG@7
PS1, Line 7: src/mainboard/intel
mb/intel/glkrvp
https://review.coreboot.org/#/c/24909/1/src/mainboard/intel/glkrvp/variants…
File src/mainboard/intel/glkrvp/variants/baseboard/gpio.c:
https://review.coreboot.org/#/c/24909/1/src/mainboard/intel/glkrvp/variants…
PS1, Line 198: GPIO_163
Is there an external pull-up on this? Else, you might want to do this configuration early in bootblock.
--
To view, visit https://review.coreboot.org/24909
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic594c5a088af79079a8742f5287ed66dcc0ff067
Gerrit-Change-Number: 24909
Gerrit-PatchSet: 1
Gerrit-Owner: Roy Mingi Park <roy.mingi.park(a)intel.com>
Gerrit-Reviewer: Bora Guvendik <bora.guvendik(a)intel.com>
Gerrit-Reviewer: Hannah Williams <hannah.williams(a)intel.com>
Gerrit-Reviewer: Shamile Khan <shamile.khan(a)intel.com>
Gerrit-Reviewer: Srinidhi N Kaushik <srinidhi.n.kaushik(a)intel.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Furquan Shaikh <furquan(a)google.com>
Gerrit-Comment-Date: Tue, 27 Feb 2018 23:09:40 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
Roy Mingi Park has uploaded this change for review. ( https://review.coreboot.org/24909
Change subject: src/mainboard/intel: Configure PERST GPIO for PCIE WiFi card
......................................................................
src/mainboard/intel: Configure PERST GPIO for PCIE WiFi card
PERST signal is required for WiFi.
When the system resumes from S0ix, WiFi will come out of D3 correctly.
Change-Id: Ic594c5a088af79079a8742f5287ed66dcc0ff067
Signed-off-by: Roy Mingi Park <roy.mingi.park(a)intel.com>
---
M src/mainboard/intel/glkrvp/variants/baseboard/gpio.c
1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/09/24909/1
diff --git a/src/mainboard/intel/glkrvp/variants/baseboard/gpio.c b/src/mainboard/intel/glkrvp/variants/baseboard/gpio.c
index 0646bcf..6ea7e4c 100644
--- a/src/mainboard/intel/glkrvp/variants/baseboard/gpio.c
+++ b/src/mainboard/intel/glkrvp/variants/baseboard/gpio.c
@@ -195,7 +195,7 @@
PAD_CFG_GPO_IOSSTATE_IOSTERM(GPIO_160, 0, DEEP, UP_20K, IGNORE, SAME),/*SD_MODE for spk*/
PAD_CFG_GPO_IOSSTATE_IOSTERM(GPIO_161, 1, DEEP, UP_20K, IGNORE, SAME),/*Touch panel reset*/
PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_162, DN_20K, DEEP, NF1, HIZCRx1, SAME),/*AVS_I2S1_BCLK*/
- PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_163, DN_20K, DEEP, NF1, HIZCRx1, SAME),/*AVS_I2S1_WS_SYNC*/
+ PAD_CFG_GPO(GPIO_163, 1, DEEP),/*WIFI RST*/
PAD_CFG_GPO_IOSSTATE_IOSTERM(GPIO_164, 1, DEEP, UP_20K, TxDRxE, ENPD),/*Touch Panel Power Enable*/
PAD_CFG_NF_IOSSTATE_IOSTERM(GPIO_165, DN_20K, DEEP, NF1, HIZCRx1, SAME),/*AVS_I2S1_SDO*/
--
To view, visit https://review.coreboot.org/24909
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic594c5a088af79079a8742f5287ed66dcc0ff067
Gerrit-Change-Number: 24909
Gerrit-PatchSet: 1
Gerrit-Owner: Roy Mingi Park <roy.mingi.park(a)intel.com>