[coreboot-gerrit] Change in coreboot[master]: ec/starlabs: Write the correct value for KLSE when suspending

[Sean Rhodes (Code Review)]

Sean Rhodes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/62608 )

Change subject: ec/starlabs: Write the correct value for KLSE when suspending
......................................................................

ec/starlabs: Write the correct value for KLSE when suspending

The current code will write the raw value from the CMOS, which doesn't
match the respective setting in EC.

Switch argument will write the correct value, and prevent the setting
being reset.

Signed-off-by: Sean Rhodes 
Change-Id: I93444cdb96eaf729630b48551d0853511b584634
---
M src/ec/starlabs/merlin/acpi/suspend.asl
1 file changed, 15 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/08/62608/1

diff --git a/src/ec/starlabs/merlin/acpi/suspend.asl b/src/ec/starlabs/merlin/acpi/suspend.asl
index cf6741a..438c3a6 100644
--- a/src/ec/starlabs/merlin/acpi/suspend.asl
+++ b/src/ec/starlabs/merlin/acpi/suspend.asl
@@ -90,6 +90,20 @@
    }

_SB.PCI0.LPCB.EC.FLKE = _SB.PCI0.LPCB.FLKC
-	_SB.PCI0.LPCB.EC.KLSE = _SB.PCI0.LPCB.KLSC
+
+	Switch (_SB.PCI0.LPCB.KLSC)
+	{
+			// 0x00 == Disabled == 0x00
+			// 0x01 == Enabled  == 0xdd
+			Case (0x00)
+			{
+				_SB.PCI0.LPCB.EC.KLSE = 0x00
+			)
+			Case (0x01)
+			{
+				_SB.PCI0.LPCB.EC.KLSE = 0xdd
+			}
+	}
+
    _SB.PCI0.LPCB.EC.KLBE = _SB.PCI0.LPCB.KLBC
 }