<p>Patrick Rudolph has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/22532">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">cpu/intel/model_206ax: Add ENERGY_PERF_BIAS control<br><br>* Add Kconfig options to set ENGERGY_PERF_BIAS:<br>** Automatic<br>** Power-save<br>** Normal<br>** Performance<br><br>Change-Id: Id9680d8fe52ed3e30bb8b39c4906b593a9e810b4<br>Signed-off-by: Patrick Rudolph <siro@das-labor.org><br>---<br>M src/cpu/intel/model_206ax/Kconfig<br>M src/cpu/intel/model_206ax/model_206ax_init.c<br>2 files changed, 46 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/32/22532/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/cpu/intel/model_206ax/Kconfig b/src/cpu/intel/model_206ax/Kconfig<br>index f16b119..50ed9ed 100644<br>--- a/src/cpu/intel/model_206ax/Kconfig<br>+++ b/src/cpu/intel/model_206ax/Kconfig<br>@@ -35,4 +35,30 @@<br>   hex<br>   default 0x800000<br> <br>+choice<br>+ prompt "Default Energy Perf Bias"<br>+  default CPU_ENGERGY_PERF_BIAS_AUTO<br>+<br>+config CPU_ENGERGY_PERF_BIAS_AUTO<br>+    bool "Automatic selection"<br>+ help<br>+   Let coreboot decide what to do.<br>+<br>+config CPU_ENGERGY_PERF_BIAS_POWERSAVE<br>+        bool "Power-save"<br>+  help<br>+   Select for setting ENERGY_POLICY_POWERSAVE.<br>+<br>+config CPU_ENGERGY_PERF_BIAS_NORMAL<br>+       bool "Normal"<br>+      help<br>+   Select for setting ENERGY_POLICY_NORMAL.<br>+<br>+config CPU_ENGERGY_PERF_BIAS_PERFORMANCE<br>+     bool "Performance"<br>+ help<br>+   Select for setting ENERGY_POLICY_PERFORMANCE.<br>+<br>+endchoice<br>+<br> endif<br>diff --git a/src/cpu/intel/model_206ax/model_206ax_init.c b/src/cpu/intel/model_206ax/model_206ax_init.c<br>index 1385fd6..96b8dc4 100644<br>--- a/src/cpu/intel/model_206ax/model_206ax_init.c<br>+++ b/src/cpu/intel/model_206ax/model_206ax_init.c<br>@@ -501,6 +501,7 @@<br> static void model_206ax_init(struct device *cpu)<br> {<br>        char processor_name[49];<br>+     u8 perf_bias;<br> <br>      /* Turn on caching if we haven't already */<br>       x86_enable_cache();<br>@@ -544,7 +545,25 @@<br>     configure_dca_cap();<br> <br>       /* Set energy policy */<br>-      set_energy_perf_bias(ENERGY_POLICY_NORMAL);<br>+  if (IS_ENABLED(CONFIG_CPU_ENGERGY_PERF_BIAS_AUTO)) {<br>+         switch (get_platform_id()) {<br>+         case 4:<br>+                      perf_bias = ENERGY_POLICY_POWERSAVE;<br>+                 break;<br>+               case 7:<br>+                      perf_bias = ENERGY_POLICY_PERFORMANCE;<br>+                       break;<br>+               default:<br>+                     perf_bias = ENERGY_POLICY_NORMAL;<br>+            }<br>+    } else if (IS_ENABLED(CPU_ENGERGY_PERF_BIAS_POWERSAVE)) {<br>+            perf_bias = ENERGY_POLICY_POWERSAVE;<br>+ } else if (IS_ENABLED(CPU_ENGERGY_PERF_BIAS_NORMAL)) {<br>+               perf_bias = ENERGY_POLICY_NORMAL;<br>+    } else {<br>+             perf_bias = ENERGY_POLICY_PERFORMANCE;<br>+       }<br>+    set_energy_perf_bias(perf_bias);<br> <br>   /* Set Max Ratio */<br>   set_max_ratio();<br></pre><p>To view, visit <a href="https://review.coreboot.org/22532">change 22532</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/22532"/><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: Id9680d8fe52ed3e30bb8b39c4906b593a9e810b4 </div>
<div style="display:none"> Gerrit-Change-Number: 22532 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Patrick Rudolph <siro@das-labor.org> </div>