<p>Youness Alaoui would like Matt DeVillier to <strong>review</strong> this change.</p><p><a href="https://review.coreboot.org/22045">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/skylake: pass SataSpeedLimit param to FSP2<br><br>The Librem13v2 needs to set this parameter to work around<br>power-related issues with some SATA devices.<br><br>Change-Id: I7fcef36ec8662e18834394b72427a0633c6b7e92<br>Signed-off-by: Matt DeVillier <matt.devillier@gmail.com><br>---<br>M src/soc/intel/skylake/chip.h<br>M src/soc/intel/skylake/chip_fsp20.c<br>2 files changed, 3 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/45/22045/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/intel/skylake/chip.h b/src/soc/intel/skylake/chip.h<br>index 022306c..2c282bc 100644<br>--- a/src/soc/intel/skylake/chip.h<br>+++ b/src/soc/intel/skylake/chip.h<br>@@ -148,6 +148,7 @@<br>    u8 SataSalpSupport;<br>   u8 SataPortsEnable[8];<br>        u8 SataPortsDevSlp[8];<br>+       u8 SataSpeedLimit;<br> <br>         /* Audio related */<br>   u8 EnableAzalia;<br>diff --git a/src/soc/intel/skylake/chip_fsp20.c b/src/soc/intel/skylake/chip_fsp20.c<br>index f9bdf8a..bc0f5a5 100644<br>--- a/src/soc/intel/skylake/chip_fsp20.c<br>+++ b/src/soc/intel/skylake/chip_fsp20.c<br>@@ -206,6 +206,8 @@<br>        params->Device4Enable = config->Device4Enable;<br>  params->SataEnable = config->EnableSata;<br>        params->SataMode = config->SataMode;<br>+   params->SataSpeedLimit = config->SataSpeedLimit;<br>+<br>     tconfig->PchLockDownGlobalSmi = config->LockDownConfigGlobalSmi;<br>        tconfig->PchLockDownRtcLock = config->LockDownConfigRtcLock;<br>    /*<br></pre><p>To view, visit <a href="https://review.coreboot.org/22045">change 22045</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/22045"/><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: I7fcef36ec8662e18834394b72427a0633c6b7e92 </div>
<div style="display:none"> Gerrit-Change-Number: 22045 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Youness Alaoui <snifikino@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Matt DeVillier <matt.devillier@gmail.com> </div>