[coreboot-gerrit] New patch to review for coreboot: google/lars: Add support for MAX98357A audio amplifier

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Thu Jan 14 11:06:41 CET 2016


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

-gerrit

commit 0a7aeb37c616b9e71f08aeae55f29abea9acb995
Author: Subrata Banik <subrata.banik at intel.com>
Date:   Fri Nov 27 11:17:35 2015 +0530

    google/lars: Add support for MAX98357A audio amplifier
    
    Adding support for Maxim 98357A audio amplifier.
    Removed SSM4567 support from LARs.
    
    BUG=chrome-os-partner:44481
    BRANCH=None
    TEST=Build & boot on LARs.
    Verify audio playback works using MAXIM amplifiers.
    
    Change-Id: I2cd8b20e936319b434017b6dd73d4739684d21d3
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 76cbc20826c884194a144f6b6bc644900e5d475d
    Original-Change-Id: I1156096b6aa367c0b8d8e3952d92f0eb5cf2820f
    Original-Signed-off-by: Subrata Banik <subrata.banik at intel.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/314543
    Original-Commit-Ready: Nicolas Boichat <drinkcat at chromium.org>
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
---
 src/mainboard/google/lars/acpi/mainboard.asl | 88 +++++++++++++---------------
 src/mainboard/google/lars/gpio.h             |  3 +
 2 files changed, 43 insertions(+), 48 deletions(-)

diff --git a/src/mainboard/google/lars/acpi/mainboard.asl b/src/mainboard/google/lars/acpi/mainboard.asl
index 73d43bc..e8397a1 100644
--- a/src/mainboard/google/lars/acpi/mainboard.asl
+++ b/src/mainboard/google/lars/acpi/mainboard.asl
@@ -44,6 +44,46 @@ Scope (\_SB)
 	{
 		Name (_HID, EisaId ("PNP0C0C"))
 	}
+
+	Device (MAXM)
+	{
+		Name (_HID, "MX98357A")
+		Name (_DDN, "Maxim Integrated 98357A Amplifier")
+		Name (_UID, 1)
+		Name (_CRS, ResourceTemplate()
+		{
+			GpioIo (Exclusive, PullDefault, 0x0000, 0x0000,
+				IoRestrictionNone,
+				"\\_SB.PCI0.GPIO", 0x00, ResourceConsumer,,)
+			{
+				GPIO_SPEAKER_MAXIM_AMP_SDMODE,
+			}
+		})
+		Name (_DSD, Package ()
+		{
+			ToUUID ("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"),
+			Package ()
+			{
+				Package () {
+					/*
+					 * Create a named GPIO
+					 * "sdmode-gpio" for the
+					 * kernel codec driver
+					 * to use.
+					 */
+
+					"sdmode-gpio", Package () {
+								^MAXM, 0, 0, 0
+							}
+				},
+			}
+		})
+
+		Method (_STA)
+		{
+			Return (0xF)
+		}
+	}
 }
 
 /*
@@ -201,52 +241,4 @@ Scope (\_SB.PCI0.I2C4)
 			Return (0xF)
 		}
 	}
-
-	/* Left Speaker Amp */
-	Device (SPKL)
-	{
-		Name (_HID, "INT343B")
-		Name (_DDN, "SSM4567 Speaker Amp")
-		Name (_UID, 0)
-
-		Name (_CRS, ResourceTemplate()
-		{
-			I2cSerialBus (
-				BOARD_LEFT_SPEAKER_AMP_I2C_ADDR,
-				ControllerInitiated,
-				400000,
-				AddressingMode7Bit,
-				"\\_SB.PCI0.I2C4",
-			)
-		})
-
-		Method (_STA)
-		{
-			Return (0xF)
-		}
-	}
-
-	/* Right Speaker Amp */
-	Device (SPKR)
-	{
-		Name (_HID, "INT343B")
-		Name (_DDN, "SSM4567 Speaker Amp")
-		Name (_UID, 1)
-
-		Name (_CRS, ResourceTemplate()
-		{
-			I2cSerialBus (
-				BOARD_RIGHT_SPEAKER_AMP_I2C_ADDR,
-				ControllerInitiated,
-				400000,
-				AddressingMode7Bit,
-				"\\_SB.PCI0.I2C4",
-			)
-		})
-
-		Method (_STA)
-		{
-			Return (0xF)
-		}
-	}
 }
diff --git a/src/mainboard/google/lars/gpio.h b/src/mainboard/google/lars/gpio.h
index 0c2ef32..3aebebc 100755
--- a/src/mainboard/google/lars/gpio.h
+++ b/src/mainboard/google/lars/gpio.h
@@ -20,6 +20,9 @@
 #include <soc/gpe.h>
 #include <soc/gpio.h>
 
+/* MAX98357A SD_MODE */
+#define GPIO_SPEAKER_MAXIM_AMP_SDMODE	GPP_B2
+
 /* EC in RW */
 #define GPIO_EC_IN_RW		GPP_C6
 



More information about the coreboot-gerrit mailing list