<p>Werner Zeh has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20789">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">siemens/nc_fpga: Add support for lowest FAN speed to FAN controller<br><br>The functionality of the FAN controller was extended to enable a<br>lowest startup speed of the FAN. Add the parameter "fanmin" to<br>the fan_ctrl_t structure and initialize the value.<br><br>Change-Id: Ib2e093ed6f5fc29bbea879779eb4777eb371b937<br>Signed-off-by: Werner Zeh <werner.zeh@siemens.com><br>---<br>M src/drivers/siemens/nc_fpga/nc_fpga.c<br>M src/drivers/siemens/nc_fpga/nc_fpga.h<br>2 files changed, 5 insertions(+), 3 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/20789/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/drivers/siemens/nc_fpga/nc_fpga.c b/src/drivers/siemens/nc_fpga/nc_fpga.c<br>index ab44e23..61d9e08 100644<br>--- a/src/drivers/siemens/nc_fpga/nc_fpga.c<br>+++ b/src/drivers/siemens/nc_fpga/nc_fpga.c<br>@@ -1,7 +1,7 @@<br> /*<br>  * This file is part of the coreboot project.<br>  *<br>- * Copyright (C) 2016 Siemens AG.<br>+ * Copyright (C) 2016-2017 Siemens AG.<br>  *<br>  * This program is free software; you can redistribute it and/or modify<br>  * it under the terms of the GNU General Public License as published by<br>@@ -73,6 +73,7 @@<br>     FPGA_SET_PARAM(FANKi, ctrl->ki);<br>   FPGA_SET_PARAM(FANKd, ctrl->kd);<br>   FPGA_SET_PARAM(FANMaxSpeed, ctrl->fanmax);<br>+        FPGA_SET_PARAM(FANStartSpeed, ctrl->fanmin);<br>       /* Set freeze and FAN configuration. */<br>       if ((hwilib_get_field(FF_FanReq, &fan_req, 1) == 1) &&<br>        (hwilib_get_field(FF_FreezeDis, &freeze_disable, 1) == 1)) {<br>diff --git a/src/drivers/siemens/nc_fpga/nc_fpga.h b/src/drivers/siemens/nc_fpga/nc_fpga.h<br>index 2cb0200..7bf087b 100644<br>--- a/src/drivers/siemens/nc_fpga/nc_fpga.h<br>+++ b/src/drivers/siemens/nc_fpga/nc_fpga.h<br>@@ -1,7 +1,7 @@<br> /*<br>  * This file is part of the coreboot project.<br>  *<br>- * Copyright (C) 2016 Siemens AG.<br>+ * Copyright (C) 2016-2017 Siemens AG.<br>  *<br>  * This program is free software; you can redistribute it and/or modify<br>  * it under the terms of the GNU General Public License as published by<br>@@ -60,7 +60,8 @@<br>         uint16_t        kp;<br>   uint16_t        ki;<br>   uint16_t        kd;<br>-  uint16_t        res8[2];<br>+     uint16_t        fanmin;<br>+      uint16_t        res8;<br>         uint16_t        fanmax;<br>       uint16_t        hystval;<br>      uint16_t        hystthreshold;<br></pre><p>To view, visit <a href="https://review.coreboot.org/20789">change 20789</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/20789"/><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: Ib2e093ed6f5fc29bbea879779eb4777eb371b937 </div>
<div style="display:none"> Gerrit-Change-Number: 20789 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Werner Zeh <werner.zeh@siemens.com> </div>