[coreboot-gerrit] Change in coreboot[master]: siemens/nc_fpga: Add support for lowest FAN speed to FAN controller

[Werner Zeh (Code Review)]

Werner Zeh has uploaded this change for review. ( https://review.coreboot.org/20789


Change subject: siemens/nc_fpga: Add support for lowest FAN speed to FAN controller
......................................................................

siemens/nc_fpga: Add support for lowest FAN speed to FAN controller

The functionality of the FAN controller was extended to enable a
lowest startup speed of the FAN. Add the parameter "fanmin" to
the fan_ctrl_t structure and initialize the value.

Change-Id: Ib2e093ed6f5fc29bbea879779eb4777eb371b937
Signed-off-by: Werner Zeh <werner.zeh at siemens.com>
---
M src/drivers/siemens/nc_fpga/nc_fpga.c
M src/drivers/siemens/nc_fpga/nc_fpga.h
2 files changed, 5 insertions(+), 3 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/20789/1

diff --git a/src/drivers/siemens/nc_fpga/nc_fpga.c b/src/drivers/siemens/nc_fpga/nc_fpga.c
index ab44e23..61d9e08 100644
--- a/src/drivers/siemens/nc_fpga/nc_fpga.c
+++ b/src/drivers/siemens/nc_fpga/nc_fpga.c
@@ -1,7 +1,7 @@
 /*
  * This file is part of the coreboot project.
  *
- * Copyright (C) 2016 Siemens AG.
+ * Copyright (C) 2016-2017 Siemens AG.
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -73,6 +73,7 @@
 	FPGA_SET_PARAM(FANKi, ctrl->ki);
 	FPGA_SET_PARAM(FANKd, ctrl->kd);
 	FPGA_SET_PARAM(FANMaxSpeed, ctrl->fanmax);
+	FPGA_SET_PARAM(FANStartSpeed, ctrl->fanmin);
 	/* Set freeze and FAN configuration. */
 	if ((hwilib_get_field(FF_FanReq, &fan_req, 1) == 1) &&
 	    (hwilib_get_field(FF_FreezeDis, &freeze_disable, 1) == 1)) {
diff --git a/src/drivers/siemens/nc_fpga/nc_fpga.h b/src/drivers/siemens/nc_fpga/nc_fpga.h
index 2cb0200..7bf087b 100644
--- a/src/drivers/siemens/nc_fpga/nc_fpga.h
+++ b/src/drivers/siemens/nc_fpga/nc_fpga.h
@@ -1,7 +1,7 @@
 /*
  * This file is part of the coreboot project.
  *
- * Copyright (C) 2016 Siemens AG.
+ * Copyright (C) 2016-2017 Siemens AG.
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -60,7 +60,8 @@
 	uint16_t	kp;
 	uint16_t	ki;
 	uint16_t	kd;
-	uint16_t	res8[2];
+	uint16_t	fanmin;
+	uint16_t	res8;
 	uint16_t	fanmax;
 	uint16_t	hystval;
 	uint16_t	hystthreshold;

-- 
To view, visit https://review.coreboot.org/20789
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib2e093ed6f5fc29bbea879779eb4777eb371b937
Gerrit-Change-Number: 20789
Gerrit-PatchSet: 1
Gerrit-Owner: Werner Zeh <werner.zeh at siemens.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20170727/4cb07a6f/attachment-0001.html>