[coreboot-gerrit] Change in coreboot[master]: ec/h8: Store PWRS and notify CPU on AC power plug/unplug events

Arthur Heymans (Code Review) gerrit at coreboot.org
Mon Dec 18 14:08:38 CET 2017


Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/22931


Change subject: ec/h8: Store PWRS and notify CPU on AC power plug/unplug events
......................................................................

ec/h8: Store PWRS and notify CPU on AC power plug/unplug events

PWRS is is the power source gnvs.

Notifying CPU is needed to change P- and C-states on these events.

Change-Id: I0818d10474523fb14f7ba7cfbf61166b89442083
Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
---
M src/ec/lenovo/h8/acpi/ac.asl
M src/ec/lenovo/h8/acpi/ec.asl
2 files changed, 6 insertions(+), 1 deletion(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/31/22931/1

diff --git a/src/ec/lenovo/h8/acpi/ac.asl b/src/ec/lenovo/h8/acpi/ac.asl
index 91b9d79..43b43d7 100644
--- a/src/ec/lenovo/h8/acpi/ac.asl
+++ b/src/ec/lenovo/h8/acpi/ac.asl
@@ -29,7 +29,10 @@
 
 	Method(_PSR, 0, NotSerialized)
 	{
-		return (HPAC)
+		Store(HPAC, Local0)
+		Store(Local0, \PWRS)
+		\PNOT()
+		return (Local0)
 	}
 
 	Method(_STA, 0, NotSerialized)
diff --git a/src/ec/lenovo/h8/acpi/ec.asl b/src/ec/lenovo/h8/acpi/ec.asl
index 944858f..b6121aa 100644
--- a/src/ec/lenovo/h8/acpi/ec.asl
+++ b/src/ec/lenovo/h8/acpi/ec.asl
@@ -149,6 +149,7 @@
 	Method(_Q26, 0, NotSerialized)
 	{
 		Notify (AC, 0x80)
+ 		\PNOT()
 	}
 
 	/* AC status change: not present */
@@ -156,6 +157,7 @@
 	{
 		Notify (AC, 0x80)
 		Store(0x50, EVNT)
+ 		\PNOT()
 	}
 
 	Method(_Q2A, 0, NotSerialized)

-- 
To view, visit https://review.coreboot.org/22931
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0818d10474523fb14f7ba7cfbf61166b89442083
Gerrit-Change-Number: 22931
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20171218/7b709ae8/attachment.html>


More information about the coreboot-gerrit mailing list