[coreboot-gerrit] New patch to review for coreboot: glados: Add ACPI configs for speaker amps and codec

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Thu Jul 23 20:19:35 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11044

-gerrit

commit a0816b41f1c788033f64b934a9c7c7c1dca8013c
Author: Ben Zhang <benzh at chromium.org>
Date:   Fri Jun 26 17:17:53 2015 -0700

    glados: Add ACPI configs for speaker amps and codec
    
    The audio codec nau8825 and two ssm4567 speaker amps are instantiated
    via ACPI.
    
    BUG=chrome-os-partner:41280
    BRANCH=none
    TEST=The devices are instantiated. Speaker/headphone playback works on glados.
    
    Change-Id: I1297c2435b3051dd749ad7de324b64ba1504cf09
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 59e5eb2682a2fc2cb58068dfcb6dd2415d43b286
    Original-Change-Id: Ib7ec8c868251601f67cdf365cd3e935d256c8ac5
    Original-Signed-off-by: Ben Zhang <benzh at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/282364
    Original-Reviewed-by: Duncan Laurie <dlaurie at chromium.org>
---
 src/mainboard/google/glados/acpi/mainboard.asl | 72 ++++++++++++++++++++++++++
 1 file changed, 72 insertions(+)

diff --git a/src/mainboard/google/glados/acpi/mainboard.asl b/src/mainboard/google/glados/acpi/mainboard.asl
index ab40d62..e4b3c59 100644
--- a/src/mainboard/google/glados/acpi/mainboard.asl
+++ b/src/mainboard/google/glados/acpi/mainboard.asl
@@ -93,3 +93,75 @@ Scope (\_SB.PCI0.I2C1)
 		}
 	}
 }
+
+Scope (\_SB.PCI0.I2C4)
+{
+	Device (CODC) // Codec for headset
+	{
+		Name (_HID, "10508825")
+		Name (_DDN, "NAU88L25 Codec")
+		Name (_UID, 1)
+
+		Name (_CRS, ResourceTemplate()
+		{
+			I2cSerialBus (
+				0x1A,                     // SlaveAddress
+				ControllerInitiated,      // SlaveMode
+				400000,                   // ConnectionSpeed
+				AddressingMode7Bit,       // AddressingMode
+				"\\_SB.PCI0.I2C4",        // ResourceSource
+			)
+		})
+
+		Method (_STA)
+		{
+			Return (0xF)
+		}
+	}
+
+	Device (SPK0) // Left speaker Amp
+	{
+		Name (_HID, "INT343B")
+		Name (_DDN, "SSM4567 Speaker Amp")
+		Name (_UID, 0)
+
+		Name (_CRS, ResourceTemplate()
+		{
+			I2cSerialBus (
+				0x34,                     // SlaveAddress
+				ControllerInitiated,      // SlaveMode
+				400000,                   // ConnectionSpeed
+				AddressingMode7Bit,       // AddressingMode
+				"\\_SB.PCI0.I2C4",        // ResourceSource
+			)
+		})
+
+		Method (_STA)
+		{
+			Return (0xF)
+		}
+	}
+
+	Device (SPK1) // Right speaker Amp
+	{
+		Name (_HID, "INT343B")
+		Name (_DDN, "SSM4567 Speaker Amp")
+		Name (_UID, 1)
+
+		Name (_CRS, ResourceTemplate()
+		{
+			I2cSerialBus (
+				0x35,                     // SlaveAddress
+				ControllerInitiated,      // SlaveMode
+				400000,                   // ConnectionSpeed
+				AddressingMode7Bit,       // AddressingMode
+				"\\_SB.PCI0.I2C4",        // ResourceSource
+			)
+		})
+
+		Method (_STA)
+		{
+			Return (0xF)
+		}
+	}
+}



More information about the coreboot-gerrit mailing list