[coreboot-gerrit] Change in coreboot[master]: ec/google/chromeec: Pass reference of object to BBST() method

[Martin Roth (Code Review)]

Martin Roth has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/34356 )

Change subject: ec/google/chromeec: Pass reference of object to BBST() method
......................................................................

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 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/34356
Tested-by: build bot (Jenkins) 
Reviewed-by: Furquan Shaikh 
Reviewed-by: Martin Roth 
---
M src/ec/google/chromeec/acpi/battery.asl
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Martin Roth: Looks good to me, approved
  Furquan Shaikh: Looks good to me, approved

diff --git a/src/ec/google/chromeec/acpi/battery.asl b/src/ec/google/chromeec/acpi/battery.asl
index 1ff5099..0253395 100644
--- a/src/ec/google/chromeec/acpi/battery.asl
+++ b/src/ec/google/chromeec/acpi/battery.asl
@@ -194,7 +194,7 @@
    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 @@

Method (_BST, 0, Serialized)
    {
-		Return (BBST (0, PBST, BSTP, BFWK))
+		Return (BBST (0, PBST, RefOf (BSTP), BFWK))
    }
 }

@@ -416,7 +416,7 @@

Method (_BST, 0, Serialized)
    {
-		Return (BBST (1, PBST, BSTP, BFWK))
+		Return (BBST (1, PBST, RefOf (BSTP), BFWK))
    }
 }
 #endif