[coreboot-gerrit] Patch set updated for coreboot: cyan/strago: disable Ambient Light Sensor device

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Mon Jul 20 23:45:56 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/11005

-gerrit

commit 326ada124a8daa719591df8c98f456d51681ba0d
Author: Jagadish Krishnamoorthy <jagadish.krishnamoorthy at intel.com>
Date:   Thu Jul 16 17:56:18 2015 -0700

    cyan/strago: disable Ambient Light Sensor device
    
    No devices are connected to i2c4 bus on
    both strago and cyan board.
    Hence disabling the ALS platform data.
    This will fix the i2c4 timeout issue and
    also help in boot time optimization.
    
    Removed unused macros.
    
    BUG=None
    BRANCH=chrome-os-partner:41934
    TEST=After booting to kernel, i2c4 timeout
    error message should not appear in dmesg.
    
    Change-Id: Ib7ab4c95b0830a8d4e53c6c0ee919649ad1ed354
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 3c52b64037b46016fe01f1d55c4c58f7684eb778
    Original-Change-Id: Ia7acdcef67a2f2837866f56aa0426a02ee05db46
    Original-Signed-off-by: Jagadish Krishnamoorthy <jagadish.krishnamoorthy at intel.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/283608
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
---
 src/mainboard/google/cyan/acpi/mainboard.asl  | 40 ---------------------------
 src/mainboard/google/cyan/irqroute.h          | 15 ----------
 src/mainboard/google/cyan/onboard.h           |  1 -
 src/mainboard/intel/strago/acpi/mainboard.asl | 39 --------------------------
 src/mainboard/intel/strago/irqroute.h         | 15 ----------
 src/mainboard/intel/strago/onboard.h          |  1 -
 6 files changed, 111 deletions(-)

diff --git a/src/mainboard/google/cyan/acpi/mainboard.asl b/src/mainboard/google/cyan/acpi/mainboard.asl
index 34b345e..9918672 100755
--- a/src/mainboard/google/cyan/acpi/mainboard.asl
+++ b/src/mainboard/google/cyan/acpi/mainboard.asl
@@ -199,46 +199,6 @@ Scope (\_SB.I2C2)
 	}
 }
 
-Scope (\_SB.I2C5)
-{
-	Device (ALSI)
-	{
-		/*
-		 * TODO(dlaurie): Need official HID.
-		 *
-		 * The current HID is created from the Intersil PNP
-		 * Vendor ID "LSD" and a shortened device identifier.
-		 */
-		Name (_HID, EisaId ("LSD2918"))
-		Name (_DDN, "Intersil 29018 Ambient Light Sensor")
-		Name (_UID, 1)
-
-		Name (_CRS, ResourceTemplate()
-		{
-			I2cSerialBus (
-				0x44,                     /* SlaveAddress */
-				ControllerInitiated,      /* SlaveMode */
-				400000,                   /* ConnectionSpeed */
-				AddressingMode7Bit,       /* AddressingMode */
-				"\\_SB.I2C5",             /* ResourceSource */
-			)
-			Interrupt (ResourceConsumer, Edge, ActiveLow)
-			{
-				BOARD_ALS_IRQ
-			}
-		})
-
-		Method (_STA)
-		{
-			If (LEqual (\S5EN, 1)) {
-				Return (0xF)
-			} Else {
-				Return (0x0)
-			}
-		}
-	}
-}
-
 Scope (\_SB.I2C6)
 {
 	Device (ETPA)
diff --git a/src/mainboard/google/cyan/irqroute.h b/src/mainboard/google/cyan/irqroute.h
index c670bee..e66c87f 100644
--- a/src/mainboard/google/cyan/irqroute.h
+++ b/src/mainboard/google/cyan/irqroute.h
@@ -47,18 +47,3 @@
 	PIRQ_PIC(G, DISABLE), \
 	PIRQ_PIC(H, DISABLE)
 
-/* CORE bank DIRQs - up to 16 supported */
-#define TPAD_IRQ_OFFSET		0
-#define TOUCH_IRQ_OFFSET	1
-#define I8042_IRQ_OFFSET	2
-#define ALS_IRQ_OFFSET		3
-/* Corresponding SCORE GPIO pins */
-#define TPAD_IRQ_GPIO		55
-#define TOUCH_IRQ_GPIO		72
-#define I8042_IRQ_GPIO		101
-#define ALS_IRQ_GPIO		70
-
-/* SUS bank DIRQs - up to 16 supported */
-#define CODEC_IRQ_OFFSET	0
-/* Corresponding SUS GPIO pins */
-#define CODEC_IRQ_GPIO		9
diff --git a/src/mainboard/google/cyan/onboard.h b/src/mainboard/google/cyan/onboard.h
index eae880b..7e77024 100755
--- a/src/mainboard/google/cyan/onboard.h
+++ b/src/mainboard/google/cyan/onboard.h
@@ -65,7 +65,6 @@
 #define BOARD_TOUCHSCREEN_I2C_BUS       0
 #define BOARD_TOUCHSCREEN_I2C_ADDR      0x4a    /* TODO(shawnn): Check this */
 
-#define BOARD_ALS_IRQ                   GPIO_S0_DED_IRQ(ALS_IRQ_OFFSET)
 
 /* SD CARD gpio */
 #define SDCARD_CD			81
diff --git a/src/mainboard/intel/strago/acpi/mainboard.asl b/src/mainboard/intel/strago/acpi/mainboard.asl
index 1228a73..eefa005 100755
--- a/src/mainboard/intel/strago/acpi/mainboard.asl
+++ b/src/mainboard/intel/strago/acpi/mainboard.asl
@@ -168,45 +168,6 @@ Scope (\_SB.I2C1)
 
 Scope (\_SB.I2C5)
 {
-	Device (ALSI)
-	{
-		/*
-		 * TODO(dlaurie): Need official HID.
-		 *
-		 * The current HID is created from the Intersil PNP
-		 * Vendor ID "LSD" and a shortened device identifier.
-		 */
-		Name (_HID, EisaId ("LSD2918"))
-		Name (_DDN, "Intersil 29018 Ambient Light Sensor")
-		Name (_UID, 1)
-
-		Name (_CRS, ResourceTemplate()
-		{
-			I2cSerialBus (
-				0x44,                     /* SlaveAddress */
-				ControllerInitiated,      /* SlaveMode */
-				400000,                   /* ConnectionSpeed */
-				AddressingMode7Bit,       /* AddressingMode */
-				"\\_SB.I2C5",             /* ResourceSource */
-			)
-			Interrupt (ResourceConsumer, Edge, ActiveLow)
-			{
-				BOARD_ALS_IRQ
-			}
-		})
-
-		Method (_STA)
-		{
-			If (LEqual (\BDID, BOARD_BCRD2)) {
-				Return (0x0)
-			}
-			If (LEqual (\S5EN, 1)) {
-				Return (0xF)
-			} Else {
-				Return (0x0)
-			}
-		}
-	}
 
 	/* Realtek Audio Codec */
 	Device (RTEK)   /* Audio Codec driver I2C */
diff --git a/src/mainboard/intel/strago/irqroute.h b/src/mainboard/intel/strago/irqroute.h
index c670bee..e66c87f 100644
--- a/src/mainboard/intel/strago/irqroute.h
+++ b/src/mainboard/intel/strago/irqroute.h
@@ -47,18 +47,3 @@
 	PIRQ_PIC(G, DISABLE), \
 	PIRQ_PIC(H, DISABLE)
 
-/* CORE bank DIRQs - up to 16 supported */
-#define TPAD_IRQ_OFFSET		0
-#define TOUCH_IRQ_OFFSET	1
-#define I8042_IRQ_OFFSET	2
-#define ALS_IRQ_OFFSET		3
-/* Corresponding SCORE GPIO pins */
-#define TPAD_IRQ_GPIO		55
-#define TOUCH_IRQ_GPIO		72
-#define I8042_IRQ_GPIO		101
-#define ALS_IRQ_GPIO		70
-
-/* SUS bank DIRQs - up to 16 supported */
-#define CODEC_IRQ_OFFSET	0
-/* Corresponding SUS GPIO pins */
-#define CODEC_IRQ_GPIO		9
diff --git a/src/mainboard/intel/strago/onboard.h b/src/mainboard/intel/strago/onboard.h
index ec75578..4b4403c 100755
--- a/src/mainboard/intel/strago/onboard.h
+++ b/src/mainboard/intel/strago/onboard.h
@@ -62,7 +62,6 @@
 #define BOARD_TOUCHSCREEN_I2C_BUS       0
 #define BOARD_TOUCHSCREEN_I2C_ADDR      0x4a    /* TODO(shawnn): Check this */
 
-#define BOARD_ALS_IRQ                   GPIO_S0_DED_IRQ(ALS_IRQ_OFFSET)
 
 
 /* SD CARD gpio */



More information about the coreboot-gerrit mailing list