summaryrefslogtreecommitdiff
path: root/src/ec/google
diff options
context:
space:
mode:
authorMatt DeVillier <matt.devillier@gmail.com>2023-01-25 14:27:24 -0600
committerMartin L Roth <gaumless@gmail.com>2023-08-20 18:28:48 +0000
commit0ace876a741b063a7c3745f2ff96d61787927114 (patch)
tree51dcbcb54a12001866b8ebc5e8ff5fbe65626997 /src/ec/google
parent095043fccac525bff3ba7f8eda713a766828899f (diff)
ec/google/wilco: Fix ACPI EC RAM read/write ops
While debugging lack of battery status under Windows, it was discovered that the read/write flags in the args to the EC RAM 'ECRW' method were not being correctly identified. Force set them from the R() and W() methods which call ECRW() so those calls are processed properly. TEST=build/boot Windows on google/drallion, verify battery status, charging, etc are all reported properly. Change-Id: I2a40b8d50ba65213813c781e53b56cc1a8b8debf Signed-off-by: Coolstar <coolstarorganization@gmail.com> Signed-off-by: Matt DeVillier <matt.devillier@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/72472 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')
-rw-r--r--src/ec/google/wilco/acpi/ec.asl7
-rw-r--r--src/ec/google/wilco/acpi/ec_ram.asl3
2 files changed, 7 insertions, 3 deletions
diff --git a/src/ec/google/wilco/acpi/ec.asl b/src/ec/google/wilco/acpi/ec.asl
index 3a9256eb09..7b41ef2e3f 100644
--- a/src/ec/google/wilco/acpi/ec.asl
+++ b/src/ec/google/wilco/acpi/ec.asl
@@ -1,5 +1,8 @@
/* SPDX-License-Identifier: GPL-2.0-only */
+Name (RD, 0)
+Name (WR, 1)
+
Device (EC0)
{
Name (_HID, EisaId ("PNP0C09"))
@@ -124,6 +127,8 @@ Device (EC0)
*/
Method (R, 1, Serialized, 2)
{
+ /* Set read operation */
+ Arg0[2] = RD
Return (ECRW (Arg0, 0))
}
@@ -134,6 +139,8 @@ Device (EC0)
*/
Method (W, 2, Serialized, 2)
{
+ /* Set write operation */
+ Arg0[2] = WR
Return (ECRW (Arg0, Arg1))
}
diff --git a/src/ec/google/wilco/acpi/ec_ram.asl b/src/ec/google/wilco/acpi/ec_ram.asl
index 9329ee7e02..665fa1b481 100644
--- a/src/ec/google/wilco/acpi/ec_ram.asl
+++ b/src/ec/google/wilco/acpi/ec_ram.asl
@@ -1,8 +1,5 @@
/* SPDX-License-Identifier: GPL-2.0-only */
-Name (RD, 0)
-Name (WR, 1)
-
/*
* EC RAM READ
*/