[coreboot-gerrit] Patch set updated for coreboot: a341965 EC H8: remove dependence on IS_X201

Vladimir Serbinenko (phcoder@gmail.com) gerrit at coreboot.org
Tue Nov 26 16:46:50 CET 2013


Vladimir Serbinenko (phcoder at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4282

-gerrit

commit a341965850c2921bf1f706de7a4a39a7e407c74e
Author: Vladimir Serbinenko <phcoder at gmail.com>
Date:   Tue Nov 26 16:37:45 2013 +0100

    EC H8: remove dependence on IS_X201
    
    Instead define brightness up/down function and gfx device and use
    preprocessor magic to glue it together.
    
    Change-Id: I03074ae07b33c1546d229efc3e80606ddbee6300
    Signed-off-by: Vladimir Serbinenko <phcoder at gmail.com>
---
 src/ec/lenovo/h8/acpi/ec.asl      | 20 ++++----------------
 src/mainboard/lenovo/t60/dsdt.asl |  2 ++
 src/mainboard/lenovo/x60/dsdt.asl |  2 ++
 3 files changed, 8 insertions(+), 16 deletions(-)

diff --git a/src/ec/lenovo/h8/acpi/ec.asl b/src/ec/lenovo/h8/acpi/ec.asl
index df19fb8..4b3fb12 100644
--- a/src/ec/lenovo/h8/acpi/ec.asl
+++ b/src/ec/lenovo/h8/acpi/ec.asl
@@ -97,35 +97,23 @@ Device(EC)
 		Notify(^SLPB, 0x80)
 	}
 
-#if IS_ENABLED (CONFIG_BOARD_LENOVO_X201)
 	/* Brightness up GPE */
 	Method(_Q14, 0, NotSerialized)
 	{
-		\_SB.PCI0.GFX0.LCD0.INCB()
+		BRIGHTNESS_UP()
 	}
 
 	/* Brightness down GPE */
 	Method(_Q15, 0, NotSerialized)
 	{
-		\_SB.PCI0.GFX0.LCD0.DECB()
+		BRIGHTNESS_DOWN()
 	}
 
+#ifdef ACPI_VIDEO_DEVICE
 	/* Next display GPE */
 	Method(_Q16, 0, NotSerialized)
 	{
-		Notify (\_SB.PCI0.GFX0, 0x82)
-	}
-#else
-	/* Brightness up GPE */
-	Method(_Q14, 0, NotSerialized)
-	{
-		\DSPC.BRTU ()
-	}
-
-	/* Brightness down GPE */
-	Method(_Q15, 0, NotSerialized)
-	{
-		\DSPC.BRTD()
+		Notify (ACPI_VIDEO_DEVICE, 0x82)
 	}
 #endif
 	/* AC status change: present */
diff --git a/src/mainboard/lenovo/t60/dsdt.asl b/src/mainboard/lenovo/t60/dsdt.asl
index 383bb30..842dbc3 100644
--- a/src/mainboard/lenovo/t60/dsdt.asl
+++ b/src/mainboard/lenovo/t60/dsdt.asl
@@ -20,6 +20,8 @@
  */
 
 #define THINKPAD_EC_GPE 28
+#define BRIGHTNESS_UP \DSPC.BRTU
+#define BRIGHTNESS_DOWN \DSPC.BRTD
 
 DefinitionBlock(
 	"dsdt.aml",
diff --git a/src/mainboard/lenovo/x60/dsdt.asl b/src/mainboard/lenovo/x60/dsdt.asl
index 383bb30..842dbc3 100644
--- a/src/mainboard/lenovo/x60/dsdt.asl
+++ b/src/mainboard/lenovo/x60/dsdt.asl
@@ -20,6 +20,8 @@
  */
 
 #define THINKPAD_EC_GPE 28
+#define BRIGHTNESS_UP \DSPC.BRTU
+#define BRIGHTNESS_DOWN \DSPC.BRTD
 
 DefinitionBlock(
 	"dsdt.aml",



More information about the coreboot-gerrit mailing list