<p>Subrata Banik has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21423">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/skylake: Create acpi_get_sleep_type() to get previous sleep state<br><br>This patch implements soc function to get previous sleep state<br>using chipset_power_state global structure.<br><br>acpi_get_sleep_type is needed in PRE_RAM stage when soc selects<br>CONFIG_EARLY_EBDA_INIT kconfig option.<br><br>Change-Id: I79acbfc09c8d255fbf9d73e49e8c7764f3f3fac6<br>Signed-off-by: Subrata Banik <subrata.banik@intel.com><br>---<br>M src/soc/intel/skylake/romstage/power_state.c<br>1 file changed, 16 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/23/21423/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/intel/skylake/romstage/power_state.c b/src/soc/intel/skylake/romstage/power_state.c<br>index 61851a5..f58ef16 100644<br>--- a/src/soc/intel/skylake/romstage/power_state.c<br>+++ b/src/soc/intel/skylake/romstage/power_state.c<br>@@ -14,6 +14,7 @@<br>  * GNU General Public License for more details.<br>  */<br> <br>+#include <arch/acpi.h><br> #include <arch/early_variables.h><br> #include <arch/io.h><br> #include <cbmem.h><br>@@ -35,12 +36,17 @@<br> <br> static struct chipset_power_state power_state CAR_GLOBAL;<br> <br>+static struct chipset_power_state *get_power_state(void)<br>+{<br>+  return (struct chipset_power_state *)car_get_var_ptr(&power_state);<br>+}<br>+<br> static void migrate_power_state(int is_recovery)<br> {<br>         struct chipset_power_state *ps_cbmem;<br>         struct chipset_power_state *ps_car;<br> <br>-       ps_car = car_get_var_ptr(&power_state);<br>+  ps_car = get_power_state();<br>   ps_cbmem = cbmem_add(CBMEM_ID_POWER_STATE, sizeof(*ps_cbmem));<br> <br>     if (ps_cbmem == NULL) {<br>@@ -129,7 +135,7 @@<br> {<br>      uint16_t tcobase;<br>     uint8_t *pmc;<br>-        struct chipset_power_state *ps = car_get_var_ptr(&power_state);<br>+  struct chipset_power_state *ps = get_power_state();<br> <br>        tcobase = smbus_tco_regs();<br> <br>@@ -161,6 +167,14 @@<br>  return ps;<br> }<br> <br>+int acpi_get_sleep_type(void)<br>+{<br>+        struct chipset_power_state *ps;<br>+<br>+   ps = get_power_state();<br>+      return ps->prev_sleep_state;<br>+}<br>+<br> int vboot_platform_is_resuming(void)<br> {<br>     return acpi_sleep_from_pm1(inl(ACPI_BASE_ADDRESS + PM1_CNT)) == ACPI_S3;<br></pre><p>To view, visit <a href="https://review.coreboot.org/21423">change 21423</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/21423"/><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: I79acbfc09c8d255fbf9d73e49e8c7764f3f3fac6 </div>
<div style="display:none"> Gerrit-Change-Number: 21423 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Subrata Banik <subrata.banik@intel.com> </div>