commit 85ba457daedd7fcac26177f2bdd0e358b3bf837b
author James Feist <james.feist@linux.intel.com> Mon Feb 04 12:59:58 2019 -0800
committer James Feist <james.feist@linux.intel.com> Tue Feb 05 19:49:40 2019 +0000
tree 7a489e20febf6923bb9cca0f73d5db9cc599a1ec
parent bea722b6f172d8ab83ad24a080d354d0dada6293

Back out multiply by 100 for pid-control

This change
https://gerrit.openbmc-project.xyz/#/c/openbmc/phosphor-pid-control/+/17952/

Backs out the divide by 100 to make the numbers less
confusing in the configuration. This change is the other
half of that to keep the configuration the same.

Tested-by: Tested boosting fans on WFP platform and
idle speeds worked correctly

Change-Id: Ib51fe119f50b66682a1eb49ba05a552fb1e3ea3f
Signed-off-by: James Feist <james.feist@linux.intel.com>

configurations/WFT Baseboard.json

diff --git a/configurations/WFT Baseboard.json b/configurations/WFT Baseboard.json
index d23d545..879cd5d 100644
--- a/configurations/WFT Baseboard.json
+++ b/configurations/WFT Baseboard.json
@@ -856,19 +856,19 @@
             "Class": "temp",
             "FFGainCoefficient": 0.0,
             "FFOffCoefficient": 0.0,
-            "ICoefficient": -20.0,
-            "ILimitMax": 7000,
-            "ILimitMin": 3000,
+            "ICoefficient": -0.2,
+            "ILimitMax": 100,
+            "ILimitMin": 30,
             "Inputs": [
                 "Core 0 CPU0"
             ],
             "Name": "Core 0 CPU0",
-            "OutLimitMax": 7000,
-            "OutLimitMin": 3000,
+            "OutLimitMax": 100,
+            "OutLimitMin": 30,
             "Outputs": [],
-            "PCoefficient": -300.0,
+            "PCoefficient": -3.0,
             "SetPoint": 45.0,
-            "SlewNeg": -100,
+            "SlewNeg": -1,
             "SlewPos": 0.0,
             "Type": "Pid",
             "Zones": [
@@ -1345,4 +1345,4 @@
         "PartNumber": "$PRODUCT_PART_NUMBER",
         "SerialNumber": "$PRODUCT_SERIAL_NUMBER"
     }
-}
\ No newline at end of file
+}