summaryrefslogtreecommitdiff
path: root/src/ec
diff options
context:
space:
mode:
authorDuncan Laurie <dlaurie@google.com>2019-07-15 17:37:09 -0700
committerMartin Roth <martinroth@google.com>2019-07-18 16:24:44 +0000
commitcacecefb2703e15563616bd83b276dee68130e8a (patch)
tree9e55a2e46767f79a46020ea44cffd2eeb27ef792 /src/ec
parent55471147e57b6f29373edb620b6cb5f2e82cd012 (diff)
downloadcoreboot-cacecefb2703e15563616bd83b276dee68130e8a.tar.xz
ec/google/chromeec: Pass reference of object to BBST() method
The BBST() method writes an updated status flag mask that is intended to be stored back in the battery object. This value needs to be passed as a reference to an object to prevent it from being evaluated at the time the method is loaded or it will not actually update the BSTP value in the battery device. This was tested by instrumenting the _BST method in the primary battery and ensuring the value can be updated by the BBST method. Change-Id: Ia8e207a2990059a60d96d8e0f3ed3c16a55c50f4 Signed-off-by: Duncan Laurie <dlaurie@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/34356 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Furquan Shaikh <furquan@google.com> Reviewed-by: Martin Roth <martinroth@google.com>
Diffstat (limited to 'src/ec')
-rw-r--r--src/ec/google/chromeec/acpi/battery.asl6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/ec/google/chromeec/acpi/battery.asl b/src/ec/google/chromeec/acpi/battery.asl
index 1ff50991c0..025339540f 100644
--- a/src/ec/google/chromeec/acpi/battery.asl
+++ b/src/ec/google/chromeec/acpi/battery.asl
@@ -194,7 +194,7 @@ Method (BBST, 4, Serialized)
Store (Local1, Index (Arg1, 0))
// Notify if battery state has changed since last time
- If (LNotEqual (Local1, Arg2)) {
+ If (LNotEqual (Local1, DeRefOf (Arg2))) {
Store (Local1, Arg2)
If (LEqual(Arg0, 0)) {
Notify (BAT0, 0x80)
@@ -326,7 +326,7 @@ Device (BAT0)
Method (_BST, 0, Serialized)
{
- Return (BBST (0, PBST, BSTP, BFWK))
+ Return (BBST (0, PBST, RefOf (BSTP), BFWK))
}
}
@@ -416,7 +416,7 @@ Device (BAT1)
Method (_BST, 0, Serialized)
{
- Return (BBST (1, PBST, BSTP, BFWK))
+ Return (BBST (1, PBST, RefOf (BSTP), BFWK))
}
}
#endif