<p>Furquan Shaikh has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/22086">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/skylake: Support logging wake source in SMM<br><br>This change adds support for logging wake source information in gsmi<br>callbacks. With this change, all the elog logging infrastructure can<br>be used for S0ix as well as S3 on skylake.<br><br>BUG=b:67874513<br><br>Change-Id: Ie1f81e956fe0bbe2e5e4c706f27997b7bd30d5e0<br>Signed-off-by: Furquan Shaikh <furquan@chromium.org><br>---<br>M src/soc/intel/skylake/Makefile.inc<br>M src/soc/intel/skylake/elog.c<br>2 files changed, 9 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/86/22086/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/intel/skylake/Makefile.inc b/src/soc/intel/skylake/Makefile.inc<br>index 9a8372b..bc53922 100644<br>--- a/src/soc/intel/skylake/Makefile.inc<br>+++ b/src/soc/intel/skylake/Makefile.inc<br>@@ -72,6 +72,7 @@<br> ramstage-$(CONFIG_UART_DEBUG) += uart_debug.c<br> ramstage-y += vr_config.c<br> <br>+smm-y += elog.c<br> smm-y += gpio.c<br> smm-y += pch.c<br> smm-y += pmutil.c<br>diff --git a/src/soc/intel/skylake/elog.c b/src/soc/intel/skylake/elog.c<br>index c015a78..8240457 100644<br>--- a/src/soc/intel/skylake/elog.c<br>+++ b/src/soc/intel/skylake/elog.c<br>@@ -20,6 +20,7 @@<br> #include <device/pci_ops.h><br> #include <stdint.h><br> #include <elog.h><br>+#include <intelblocks/pmclib.h><br> #include <soc/pci_devs.h><br> #include <soc/pm.h><br> #include <soc/smbus.h><br>@@ -382,3 +383,10 @@<br> }<br> <br> BOOT_STATE_INIT_ENTRY(BS_DEV_INIT, BS_ON_EXIT, pch_log_state, NULL);<br>+<br>+void elog_gsmi_cb_platform_log_wake_source(void)<br>+{<br>+  struct chipset_power_state ps;<br>+       pmc_fill_pm_reg_info(&ps);<br>+       pch_log_wake_source(&ps);<br>+}<br></pre><p>To view, visit <a href="https://review.coreboot.org/22086">change 22086</a>. To unsubscribe, 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/22086"/><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: Ie1f81e956fe0bbe2e5e4c706f27997b7bd30d5e0 </div>
<div style="display:none"> Gerrit-Change-Number: 22086 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Furquan Shaikh <furquan@google.com> </div>