[coreboot-gerrit] Patch set updated for coreboot: google/poppy: Set GPIO GPP_D22 high

Rizwan Qureshi (rizwan.qureshi@intel.com) gerrit at coreboot.org
Fri Feb 3 12:01:57 CET 2017


Rizwan Qureshi (rizwan.qureshi at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18282

-gerrit

commit 87d9f7911ee8c7021d60982b8d8e5deee4bd3e1b
Author: Rizwan Qureshi <rizwan.qureshi at intel.com>
Date:   Thu Feb 2 12:23:37 2017 +0530

    google/poppy: Set GPIO GPP_D22 high
    
    same change as I49935e659bf67225d3f5db1b06acc2cd046dcd74
    this is required for poppy board as well.
    
    GPIO GPP_D22 controls the I2S buffer for isolating the I2S signals
    when doing GPIO-driven I2S.  This needs to be high by default so
    the DSP can drive these signals, instead of low where it is enabled
    for GPIO-driven I2S and the DSP cannot drive these signals.
    
    BUG=None
    BRANCH=None
    TEST=play test sound in OS over internal speaker
    
    Change-Id: I1695e9198f8f78e9c5ad6df6c1ac073ac1762c6b
    Signed-off-by: Rizwan Qureshi <rizwan.qureshi at intel.com>
---
 src/mainboard/google/poppy/gpio.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mainboard/google/poppy/gpio.h b/src/mainboard/google/poppy/gpio.h
index 0a2544c..b3849f1 100644
--- a/src/mainboard/google/poppy/gpio.h
+++ b/src/mainboard/google/poppy/gpio.h
@@ -144,7 +144,7 @@ static const struct pad_config gpio_table[] = {
 /* DMIC_CLK0 */		PAD_CFG_NF(GPP_D19, NONE, DEEP, NF1), /* DMIC_CLK0 */
 /* DMIC_DATA0 */	PAD_CFG_NF(GPP_D20, NONE, DEEP, NF1), /* DMIC_DATA0 */
 /* SPI1_IO2 */		PAD_CFG_NC(GPP_D21),
-/* SPI1_IO3 */		PAD_CFG_GPO(GPP_D22, 0, DEEP), /* I2S2 BUFFER */
+/* SPI1_IO3 */		PAD_CFG_GPO(GPP_D22, 1, DEEP), /* I2S2 BUFFER */
 /* I2S_MCLK */		PAD_CFG_NF(GPP_D23, NONE, DEEP, NF1), /* MCLK */
 
 /* SATAXPCI0 */		PAD_CFG_GPI_APIC_INVERT(GPP_E0, NONE,



More information about the coreboot-gerrit mailing list