<p>Subrata Banik has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21387">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">arch/x86: Make acpi_get_sleep_type function available in romstage<br><br>acpi_is_wakeup_s3() function is available in both romstage and<br>ramstage (include/arch/acpi.h) but helper function<br>acpi_get_sleep_type() is only available during ramstage(ENV_RAMSTAGE).<br><br>This patch to make acpi_get_sleep_type() available in romstage<br>as well.<br><br>Change-Id: Iac8d143903996e5fc1a89d73c6735811a2e7395f<br>Signed-off-by: Subrata Banik <subrata.banik@intel.com><br>---<br>M src/arch/x86/acpi_s3.c<br>1 file changed, 5 insertions(+), 5 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/21387/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/arch/x86/acpi_s3.c b/src/arch/x86/acpi_s3.c<br>index a0a7905..cfca362 100644<br>--- a/src/arch/x86/acpi_s3.c<br>+++ b/src/arch/x86/acpi_s3.c<br>@@ -25,11 +25,6 @@<br> #include <rules.h><br> #include <symbols.h><br> <br>-#if ENV_RAMSTAGE<br>-<br>-/* This is filled with acpi_is_wakeup() call early in ramstage. */<br>-static int acpi_slp_type = -1;<br>-<br> #if IS_ENABLED(CONFIG_EARLY_CBMEM_INIT)<br> int acpi_get_sleep_type(void)<br> {<br>@@ -42,6 +37,11 @@<br> }<br> #endif<br> <br>+#if ENV_RAMSTAGE<br>+<br>+/* This is filled with acpi_is_wakeup() call early in ramstage. */<br>+static int acpi_slp_type = -1;<br>+<br> static void acpi_handoff_wakeup(void)<br> {<br>      if (acpi_slp_type < 0)<br></pre><p>To view, visit <a href="https://review.coreboot.org/21387">change 21387</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/21387"/><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: Iac8d143903996e5fc1a89d73c6735811a2e7395f </div>
<div style="display:none"> Gerrit-Change-Number: 21387 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Subrata Banik <subrata.banik@intel.com> </div>