[coreboot-gerrit] New patch to review for coreboot: glados: Clean up mainboard ACPI devices

[Aaron Durbin]

Aaron Durbin ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11277

-gerrit

commit 13369df16ef0af2e37d499a6ac865b066a54cba1
Author: Duncan Laurie 
Date:   Thu Aug 13 12:18:52 2015 -0700

glados: Clean up mainboard ACPI devices

Clean up the device code for the glados mainboard, using
    the defined values for interrupts by the SOC and moving the
    various codec i2c addresses to the top of the file.

BUG=chrome-os-partner:40635
    BRANCH=none
    TEST=build and boot on glados

Original-Change-Id: Iead1aeb54363b15a6176d4f4a9511674195c0505
    Original-Signed-off-by: Duncan Laurie 
    Original-Reviewed-on: https://chromium-review.googlesource.com/293511
    Original-Reviewed-by: Aaron Durbin

Change-Id: I083c9ef6140e20a433cb2017e4c3cbc7a41e8fed
    Signed-off-by: Duncan Laurie 
---
 src/mainboard/google/glados/acpi/mainboard.asl | 68 ++++++++++++++------------
 1 file changed, 38 insertions(+), 30 deletions(-)

diff --git a/src/mainboard/google/glados/acpi/mainboard.asl b/src/mainboard/google/glados/acpi/mainboard.asl
index f8bc462..f955bb7 100644
--- a/src/mainboard/google/glados/acpi/mainboard.asl
+++ b/src/mainboard/google/glados/acpi/mainboard.asl
@@ -19,8 +19,12 @@

#include <soc/gpio.h>

-#define BOARD_TRACKPAD_I2C_ADDR		0x15
-#define BOARD_TRACKPAD_IRQ		GPP_B3_IRQ
+#define BOARD_TOUCHPAD_I2C_ADDR			0x15
+#define BOARD_TOUCHPAD_IRQ			GPP_B3_IRQ
+
+#define BOARD_HP_MIC_CODEC_I2C_ADDR		0x1a
+#define BOARD_LEFT_SPEAKER_AMP_I2C_ADDR		0x34
+#define BOARD_RIGHT_SPEAKER_AMP_I2C_ADDR	0x35

Scope (_SB)
 {
@@ -68,30 +72,31 @@ Scope (_SB.PCI0.LPCB)
    #include <drivers/pc80/tpm/acpi/tpm.asl>
 }

-/* Trackpad */
 Scope (_SB.PCI0.I2C1)
 {
-	Device (ELAN)
+	/* Touchpad */
+	Device (ELTP)
    {
    	Name (_HID, "ELAN0000")
    	Name (_DDN, "Elan Touchpad")
-		Name (_UID, 3)
+		Name (_UID, 1)
    	Name (_S0W, 4)
-		Name (ISTP, 1) /* TouchPad */
+
    	Name (_CRS, ResourceTemplate()
    	{
    		I2cSerialBus (
-				BOARD_TRACKPAD_I2C_ADDR,	/* SlaveAddress */
-				ControllerInitiated,		/* SlaveMode */
-				400000,				/* ConnectionSpeed */
-				AddressingMode7Bit,		/* AddressingMode */
-				"\_SB.PCI0.I2C1",		/* ResourceSource */
+				BOARD_TOUCHPAD_I2C_ADDR,
+				ControllerInitiated,
+				400000,
+				AddressingMode7Bit,
+				"\_SB.PCI0.I2C1",
    		)
    		Interrupt (ResourceConsumer, Edge, ActiveLow)
    		{
-				BOARD_TRACKPAD_IRQ
+				BOARD_TOUCHPAD_IRQ
    		}
    	})
+
    	Method (_STA)
    	{
    		Return (0xF)
@@ -101,7 +106,8 @@ Scope (_SB.PCI0.I2C1)

Scope (_SB.PCI0.I2C4)
 {
-	Device (CODC) // Codec for headset
+	/* Headphone Codec */
+	Device (HPMC)
    {
    	Name (_HID, "10508825")
    	Name (_DDN, "NAU88L25 Codec")
@@ -110,11 +116,11 @@ Scope (_SB.PCI0.I2C4)
    	Name (_CRS, ResourceTemplate()
    	{
    		I2cSerialBus (
-				0x1A,				/* SlaveAddress */
-				ControllerInitiated,		/* SlaveMode */
-				400000,				/* ConnectionSpeed */
-				AddressingMode7Bit,		/* AddressingMode */
-				"\_SB.PCI0.I2C4",		/* ResourceSource */
+				BOARD_HP_MIC_CODEC_I2C_ADDR,
+				ControllerInitiated,
+				400000,
+				AddressingMode7Bit,
+				"\_SB.PCI0.I2C4",
    		)
    	})

@@ -124,7 +130,8 @@ Scope (_SB.PCI0.I2C4)
    	}
    }

-	Device (SPK0) // Left speaker Amp
+	/* Left Speaker Amp */
+	Device (SPKL)
    {
    	Name (_HID, "INT343B")
    	Name (_DDN, "SSM4567 Speaker Amp")
@@ -133,11 +140,11 @@ Scope (_SB.PCI0.I2C4)
    	Name (_CRS, ResourceTemplate()
    	{
    		I2cSerialBus (
-				0x34,				/* SlaveAddress */
-				ControllerInitiated,		/* SlaveMode */
-				400000,				/* ConnectionSpeed */
-				AddressingMode7Bit,		/* AddressingMode */
-				"\_SB.PCI0.I2C4",		/* ResourceSource */
+				BOARD_LEFT_SPEAKER_AMP_I2C_ADDR,
+				ControllerInitiated,
+				400000,
+				AddressingMode7Bit,
+				"\_SB.PCI0.I2C4",
    		)
    	})

@@ -147,7 +154,8 @@ Scope (_SB.PCI0.I2C4)
    	}
    }

-	Device (SPK1) // Right speaker Amp
+	/* Right Speaker Amp */
+	Device (SPKR)
    {
    	Name (_HID, "INT343B")
    	Name (_DDN, "SSM4567 Speaker Amp")
@@ -156,11 +164,11 @@ Scope (_SB.PCI0.I2C4)
    	Name (_CRS, ResourceTemplate()
    	{
    		I2cSerialBus (
-				0x35,				/* SlaveAddress */
-				ControllerInitiated,		/* SlaveMode */
-				400000,				/* ConnectionSpeed */
-				AddressingMode7Bit,		/* AddressingMode */
-				"\_SB.PCI0.I2C4",		/* ResourceSource */
+				BOARD_RIGHT_SPEAKER_AMP_I2C_ADDR,
+				ControllerInitiated,
+				400000,
+				AddressingMode7Bit,
+				"\_SB.PCI0.I2C4",
    		)
    	})