[coreboot-gerrit] Patch set updated for coreboot: 998c0d7 samus: Set codec PDM clock output to 3MHz

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Fri Apr 3 18:55:33 CEST 2015


Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/9277

-gerrit

commit 998c0d7fd0b79830979d3f9feeaef223a50135a6
Author: Ben Zhang <benzh at chromium.org>
Date:   Mon Nov 17 17:21:09 2014 -0800

    samus: Set codec PDM clock output to 3MHz
    
    Currently the rt5677 codec outputs 6MHz PDM clock which is
    out-of-spec for the speaker amp SSM2537. The amp's GAIN_FS
    pin is pulled down to PGND with a 47k resistor, so the
    expected PDM clock is 64*FS (~3MHz) according to its datasheet.
    
    The corresponding kernel patch that adds the PDM clock config
    option is https://chromium-review.googlesource.com/#/c/230303/
    
    BUG=chrome-os-partner:33303
    BRANCH=samus
    TEST=flash coreboot with this patch and see PDM CLK went
    from 6MHz to 3MHz on samus with a scope.
    
    Change-Id: Icf2c61930175bede1ee8ebc2b0fb17c2938b806c
    Signed-off-by: Stefan Reinauer <reinauer at chromium.org>
    Original-Commit-Id: b9ba4597515b2fbcc72fa22e296357c454175648
    Original-Change-Id: I09acdf47bab4f641981491a84197de234918435e
    Original-Signed-off-by: Ben Zhang <benzh at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/230344
    Original-Reviewed-by: Dylan Reid <dgreid at chromium.org>
    Original-Reviewed-by: Duncan Laurie <dlaurie at chromium.org>
---
 src/mainboard/google/samus/acpi/mainboard.asl | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/mainboard/google/samus/acpi/mainboard.asl b/src/mainboard/google/samus/acpi/mainboard.asl
index 39797e5..2b6cea0 100644
--- a/src/mainboard/google/samus/acpi/mainboard.asl
+++ b/src/mainboard/google/samus/acpi/mainboard.asl
@@ -170,6 +170,7 @@ Scope (\_SB.PCI0.I2C0)
 		Name (WAKE, 45) /* DSP_INT (use as codec wake) */
 
 		Name (DCLK, 0)  /* RT5677_DMIC_CLK1 */
+		Name (PCLK, 1)  /* RT5677_PDM_CLK_DIV2 (~3MHz) */
 		Name (IN1, 1)   /* IN1 differential */
 		Name (IN2, 0)   /* IN2 not differential */
 		Name (OUT1, 1)  /* LOUT1 differential */



More information about the coreboot-gerrit mailing list