summaryrefslogtreecommitdiff
path: root/src/ec/google/chromeec
diff options
context:
space:
mode:
authorMatt DeVillier <matt.devillier@gmail.com>2022-10-18 17:53:08 -0500
committerFelix Held <felix-coreboot@felixheld.de>2023-08-04 14:03:52 +0000
commit3f0de3d3ea57b6ab19d18ef490462d89aaf8aa9d (patch)
tree0ad18522947b7885310d21ec24a959065ea9d782 /src/ec/google/chromeec
parent6ba83484e6d54ddce7c3caccb79d4e7015cfeb37 (diff)
ec/google/chromeec: move TBMC ACPI device under CREC
Tablet motion control is a function of the EC, and under Windows, the TBMC device needs to be initialized after CREC, or driver init will fail. The only way to ensure this happens is for TBMC to be a child device under CREC. TEST=build/boot Win11, Linux on google/eve, verify tablet mode drivers loaded and orientation switching functional under both OSes. Change-Id: I5e9eab9ae277b5a04dc2666960a727e5680bf6f4 Signed-off-by: Matt DeVillier <matt.devillier@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/76792 Reviewed-by: CoolStar <coolstarorganization@gmail.com> Reviewed-by: Caveh Jalali <caveh@chromium.org> Reviewed-by: Felix Held <felix-coreboot@felixheld.de> Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
Diffstat (limited to 'src/ec/google/chromeec')
-rw-r--r--src/ec/google/chromeec/acpi/cros_ec.asl5
-rw-r--r--src/ec/google/chromeec/acpi/ec.asl6
-rw-r--r--src/ec/google/chromeec/acpi/tbmc.asl4
3 files changed, 8 insertions, 7 deletions
diff --git a/src/ec/google/chromeec/acpi/cros_ec.asl b/src/ec/google/chromeec/acpi/cros_ec.asl
index adf8d90d8c..b46c18741a 100644
--- a/src/ec/google/chromeec/acpi/cros_ec.asl
+++ b/src/ec/google/chromeec/acpi/cros_ec.asl
@@ -51,6 +51,11 @@ Device (CREC)
#ifdef EC_ENABLE_PD_MCU_DEVICE
#include "pd.asl"
#endif
+
+#ifdef EC_ENABLE_TBMC_DEVICE
+ #include "tbmc.asl"
+#endif
+
Method(_STA, 0)
{
Return (0xF)
diff --git a/src/ec/google/chromeec/acpi/ec.asl b/src/ec/google/chromeec/acpi/ec.asl
index 3d5961aa1b..d6d33b2a08 100644
--- a/src/ec/google/chromeec/acpi/ec.asl
+++ b/src/ec/google/chromeec/acpi/ec.asl
@@ -457,7 +457,7 @@ Device (EC0)
\_SB.DPTF.TPET()
#endif
#ifdef EC_ENABLE_TBMC_DEVICE
- Notify (TBMC, 0x80)
+ Notify (^CREC.TBMC, 0x80)
#endif
#if CONFIG(SOC_AMD_COMMON_BLOCK_ACPI_DPTC)
If (CondRefOf (\_SB.DPTC)) {
@@ -660,8 +660,4 @@ Device (EC0)
#ifdef EC_ENABLE_KEYBOARD_BACKLIGHT
#include "keyboard_backlight.asl"
#endif
-
-#ifdef EC_ENABLE_TBMC_DEVICE
- #include "tbmc.asl"
-#endif
}
diff --git a/src/ec/google/chromeec/acpi/tbmc.asl b/src/ec/google/chromeec/acpi/tbmc.asl
index 3ecf78ef5b..efc42570df 100644
--- a/src/ec/google/chromeec/acpi/tbmc.asl
+++ b/src/ec/google/chromeec/acpi/tbmc.asl
@@ -7,7 +7,7 @@ Device (TBMC)
Name (_DDN, "Tablet Motion Control")
Method (TBMC)
{
- If (^^RCTM == 1) {
+ If (RCTM == 1) {
Return (0x1)
} Else {
Return (0x0)
@@ -15,7 +15,7 @@ Device (TBMC)
}
Method(_STA, 0)
{
- If (^^MTNS == 1) {
+ If (MTNS == 1) {
Return (0xF)
} Else {
Return (0x0)