<p>Duncan Laurie has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/29487">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/common: Call EC _PTS/_WAK methods if defined<br><br>Some embedded controllers expect to be sent a command when the OS<br>calls the ACPI \_PTS and \_WAK methods.  For example see the code<br>in ec/google/wilco/acpi/platform.asl that tells the EC when the<br>methods have been executed by the OS.<br><br>Not all ECs may define these methods so this change uses CondRefOf<br>to check if the EC method exists, and if so will call it, providing<br>the same argument that was passed to the system level methods.<br><br>Change-Id: I6bf83509423c0fb07c4890986a189cf54afaed10<br>Signed-off-by: Duncan Laurie <dlaurie@google.com><br>---<br>M src/soc/intel/common/acpi/platform.asl<br>1 file changed, 11 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/87/29487/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/soc/intel/common/acpi/platform.asl b/src/soc/intel/common/acpi/platform.asl</span><br><span>index 1d0294b..6647f77 100644</span><br><span>--- a/src/soc/intel/common/acpi/platform.asl</span><br><span>+++ b/src/soc/intel/common/acpi/platform.asl</span><br><span>@@ -33,6 +33,11 @@</span><br><span> Method (_PTS, 1)</span><br><span> {</span><br><span>         Store (POST_OS_ENTER_PTS, DBG0)</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+     /* Call EC _PTS handler if present */</span><br><span style="color: hsl(120, 100%, 40%);">+ If (CondRefOf (\_SB.PCI0.LPCB.EC0.PTS)) {</span><br><span style="color: hsl(120, 100%, 40%);">+             \_SB.PCI0.LPCB.EC0.PTS (Arg0)</span><br><span style="color: hsl(120, 100%, 40%);">+ }</span><br><span> }</span><br><span> </span><br><span> /* The _WAK method is called on system wakeup */</span><br><span>@@ -40,5 +45,11 @@</span><br><span> Method (_WAK, 1)</span><br><span> {</span><br><span>         Store (POST_OS_ENTER_WAKE, DBG0)</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+    /* Call EC _PTS handler if present */</span><br><span style="color: hsl(120, 100%, 40%);">+ If (CondRefOf (\_SB.PCI0.LPCB.EC0.WAK)) {</span><br><span style="color: hsl(120, 100%, 40%);">+             \_SB.PCI0.LPCB.EC0.WAK (Arg0)</span><br><span style="color: hsl(120, 100%, 40%);">+ }</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>  Return (Package(){0,0})</span><br><span> }</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/29487">change 29487</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/29487"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I6bf83509423c0fb07c4890986a189cf54afaed10 </div>
<div style="display:none"> Gerrit-Change-Number: 29487 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Duncan Laurie <dlaurie@chromium.org> </div>