[coreboot-gerrit] Change in coreboot[master]: mb/google/poppy/variants/nocturne: Enable DMIC CLK0/DATA0

Sathyanarayana Nujella (Code Review) gerrit at coreboot.org
Sun Sep 2 18:25:48 CEST 2018


Hello HARSHAPRIYA N,

I'd like you to do a code review. Please visit

    https://review.coreboot.org/28433

to review the following change.


Change subject: mb/google/poppy/variants/nocturne: Enable DMIC CLK0/DATA0
......................................................................

mb/google/poppy/variants/nocturne: Enable DMIC CLK0/DATA0

DMIC's are now connected to DMIC_CLK0/DMIC_DATA0.
So, enable the pins accordingly.

BUG=b:113744731
BRANCH=none
TEST='emerge-nocturne coreboot chromeos-bootimage' builds the image

Change-Id: I48cace3c6099a2853fcb377c695a5e325094baf6
Signed-off-by: Sathyanarayana Nujella <sathyanarayana.nujella at intel.com>
Signed-off-by: Harsha Priya <harshapriya.n at intel.com>
---
M src/mainboard/google/poppy/variants/nocturne/gpio.c
1 file changed, 4 insertions(+), 4 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/33/28433/1

diff --git a/src/mainboard/google/poppy/variants/nocturne/gpio.c b/src/mainboard/google/poppy/variants/nocturne/gpio.c
index 782ce00..08bb801 100644
--- a/src/mainboard/google/poppy/variants/nocturne/gpio.c
+++ b/src/mainboard/google/poppy/variants/nocturne/gpio.c
@@ -199,10 +199,10 @@
 	PAD_CFG_NF(GPP_D17, NONE, DEEP, NF1),
 	/* D18 : DMIC_DATA1 ==> PCH_DMIC_DATA */
 	PAD_CFG_NF(GPP_D18, NONE, DEEP, NF1),
-	/* D19 : DMIC_CLK0 ==> NC */
-	PAD_CFG_NC(GPP_D19),
-	/* D20 : DMIC_DATA0 ==> NC */
-	PAD_CFG_NC(GPP_D20),
+	/* D19 : DMIC_CLK0 ==> PCH_DMIC_CLK_OUT */
+	PAD_CFG_NF(GPP_D19, NONE, DEEP, NF1),
+	/* D20 : DMIC_DATA0 ==> PCH_DMIC_DATA_IN */
+	PAD_CFG_NF(GPP_D20, NONE, DEEP, NF1),
 	/* D21 : SPI1_IO2 ==> NC */
 	PAD_CFG_NC(GPP_D21),
 	/* D22 : SPI1_IO3 ==> NC */

-- 
To view, visit https://review.coreboot.org/28433
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: I48cace3c6099a2853fcb377c695a5e325094baf6
Gerrit-Change-Number: 28433
Gerrit-PatchSet: 1
Gerrit-Owner: Sathyanarayana Nujella <sathyanarayana.nujella at intel.com>
Gerrit-Reviewer: HARSHAPRIYA N <harshapriya.n at intel.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180902/245049fb/attachment.html>


More information about the coreboot-gerrit mailing list