<p>Mario Scheithauer has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20759">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/apollolake: Make usage of RAPL selectable<br><br>Apollo Lake SoC supports configuration of Running Average Power Limits<br>(RAPL) for package domain. This feature is not required for all APL<br>mainboards. According to the APL SoC EDS Vol 4 chapter 18.4 Power<br>Limiting Control it is not necessary to enable the RAPL algorithm per<br>default. For that reason make the RAPL configuration selectable.<br><br>Change-Id: Ib737b162f72b76c15e5768859f9099e2e7ef6426<br>Signed-off-by: Mario Scheithauer <mario.scheithauer@siemens.com><br>---<br>M src/soc/intel/apollolake/Kconfig<br>M src/soc/intel/apollolake/chip.c<br>2 files changed, 13 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/59/20759/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/intel/apollolake/Kconfig b/src/soc/intel/apollolake/Kconfig<br>index a9774dc..a5438df 100644<br>--- a/src/soc/intel/apollolake/Kconfig<br>+++ b/src/soc/intel/apollolake/Kconfig<br>@@ -342,4 +342,12 @@<br>   int<br>   default 100<br> <br>+config APL_SKIP_SET_POWER_LIMITS<br>+    bool<br>+ default n<br>+    help<br>+   Some Apollo Lake mainboards do not need the Running Average Power<br>+    Limits (RAPL) algorithm for a constant power management.<br>+     Set this config option to disable for skip the configuration.<br>+<br> endif<br>diff --git a/src/soc/intel/apollolake/chip.c b/src/soc/intel/apollolake/chip.c<br>index f72173d..ec4662c 100644<br>--- a/src/soc/intel/apollolake/chip.c<br>+++ b/src/soc/intel/apollolake/chip.c<br>@@ -223,6 +223,11 @@<br>         uint32_t tdp, min_power, max_power;<br>   uint32_t pl2_val;<br> <br>+ if (IS_ENABLED(CONFIG_APL_SKIP_SET_POWER_LIMITS)) {<br>+          printk(BIOS_INFO, "Skip the RAPL settings.\n");<br>+            return;<br>+      }<br>+<br>  if (!dev || !dev->chip_info) {<br>             printk(BIOS_ERR, "BUG! Could not find SOC devicetree config\n");<br>            return;<br></pre><p>To view, visit <a href="https://review.coreboot.org/20759">change 20759</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/20759"/><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: Ib737b162f72b76c15e5768859f9099e2e7ef6426 </div>
<div style="display:none"> Gerrit-Change-Number: 20759 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Mario Scheithauer <mario.scheithauer@siemens.com> </div>