<p>Richard Spiegel has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21770">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/amd/stoneyridge/southbridge.c: Remove preprocessor #IF<br><br>Replace #if and #endif with runtime <if (condition) {> and <}><br><br>Code Files: southbridge.c<br>BUG=b:62200891<br><br>Change-Id: I69877bf301fa89781381e3eb8e6b4acd7e16b4b4<br>Signed-off-by: Richard Spiegel <richard.spiegel@silverbackltd.com><br>---<br>M src/soc/amd/stoneyridge/southbridge.c<br>1 file changed, 5 insertions(+), 8 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/70/21770/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/amd/stoneyridge/southbridge.c b/src/soc/amd/stoneyridge/southbridge.c<br>index 5e36100..676f971 100644<br>--- a/src/soc/amd/stoneyridge/southbridge.c<br>+++ b/src/soc/amd/stoneyridge/southbridge.c<br>@@ -28,9 +28,7 @@<br> #include <soc/southbridge.h><br> #include <soc/smbus.h><br> #include <soc/smi.h><br>-#if IS_ENABLED(CONFIG_STONEYRIDGE_IMC_FWM)<br> #include <fchec.h><br>-#endif<br> <br> <br> int acpi_get_sleep_type(void)<br>@@ -98,12 +96,11 @@<br> <br> void southbridge_final(void *chip_info)<br> {<br>-#if IS_ENABLED(CONFIG_STONEYRIDGE_IMC_FWM)<br>-       agesawrapper_fchecfancontrolservice();<br>-#if !IS_ENABLED(CONFIG_ACPI_ENABLE_THERMAL_ZONE)<br>-    enable_imc_thermal_zone();<br>-#endif<br>-#endif<br>+ if (IS_ENABLED(CONFIG_STONEYRIDGE_IMC_FWM)) {<br>+                agesawrapper_fchecfancontrolservice();<br>+               if (!IS_ENABLED(CONFIG_ACPI_ENABLE_THERMAL_ZONE))<br>+                    enable_imc_thermal_zone();<br>+   }<br> }<br> <br> /*<br></pre><p>To view, visit <a href="https://review.coreboot.org/21770">change 21770</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/21770"/><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: I69877bf301fa89781381e3eb8e6b4acd7e16b4b4 </div>
<div style="display:none"> Gerrit-Change-Number: 21770 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Richard Spiegel <richard.spiegel@silverbackltd.com> </div>