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

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/62607 )

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

ec/starlabs: Write the correct value for TPLE 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: Iabeec47bf492b698f95d86aa2d08ba9caedd75f6
Reviewed-on: https://review.coreboot.org/c/coreboot/+/62607
Tested-by: build bot (Jenkins) 
Reviewed-by: Angel Pons 
---
M src/ec/starlabs/merlin/acpi/suspend.asl
1 file changed, 15 insertions(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Angel Pons: Looks good to me, approved

diff --git a/src/ec/starlabs/merlin/acpi/suspend.asl b/src/ec/starlabs/merlin/acpi/suspend.asl
index 81ae8d1..0f1c27c 100644
--- a/src/ec/starlabs/merlin/acpi/suspend.asl
+++ b/src/ec/starlabs/merlin/acpi/suspend.asl
@@ -82,7 +82,21 @@
    _SB.PCI0.LPCB.EC.OSFG = 0x01

/* Restore EC settings from CMOS */
-	_SB.PCI0.LPCB.EC.TPLE = _SB.PCI0.LPCB.TPLC
+	Switch (ToInteger (_SB.PCI0.LPCB.TPLC))
+	{
+		// 0x00 == Enabled    == 0x00
+		// 0x00 == Re-enabled == 0x11
+		// 0x01 == Disabled   == 0x22
+		Case (0x00)
+		{
+			_SB.PCI0.LPCB.EC.TPLE = 0x00
+		}
+		Case (0x01)
+		{
+			_SB.PCI0.LPCB.EC.TPLE = 0x22
+		}
+	}
+
    _SB.PCI0.LPCB.EC.FLKE = _SB.PCI0.LPCB.FLKC
    _SB.PCI0.LPCB.EC.KLSE = _SB.PCI0.LPCB.KLSC
    _SB.PCI0.LPCB.EC.KLBE = _SB.PCI0.LPCB.KLBC