commit 5008dafb53c29599dd85ad7cc383a7f2ab319fdd
author Matthew Barth <msbarth@us.ibm.com> Mon Jan 15 16:38:24 2018 -0600
committer Matthew Barth <msbarth@us.ibm.com> Tue Jan 23 13:07:36 2018 -0600
tree 8401d3f65ff1b3553bc58df94621e441f68eb3c1
parent f552ea5ce23889905ce15c0ffcfc73fa1245a423

Remove getTargetSpeed

The getTargetSpeed function is no longer needed. The tach sensors are
used to get the target speed.

Change-Id: I5f82b0c3e8104203e2700b22a5488cf63673d181
Signed-off-by: Matthew Barth <msbarth@us.ibm.com>

monitor/fan.cpp

diff --git a/monitor/fan.cpp b/monitor/fan.cpp
index f24312d..9d2593f 100644
--- a/monitor/fan.cpp
+++ b/monitor/fan.cpp
@@ -154,12 +154,6 @@
 }
 
 
-uint64_t Fan::getTargetSpeed(const TachSensor& sensor)
-{
-    return sensor.getTarget();
-}
-
-
 bool Fan::tooManySensorsNonfunctional()
 {
     size_t numFailed =  std::count_if(_sensors.begin(), _sensors.end(),
@@ -175,7 +169,7 @@
 bool Fan::outOfRange(const TachSensor& sensor)
 {
     auto actual = static_cast<uint64_t>(sensor.getInput());
-    auto target = getTargetSpeed(sensor);
+    auto target = sensor.getTarget();
 
     uint64_t min = target * (100 - _deviation) / 100;
     uint64_t max = target * (100 + _deviation) / 100;
@@ -203,7 +197,7 @@
                 entry("FAN=%s", _name.c_str()),
                 entry("TACH_SENSOR=%s", sensor.name().c_str()),
                 entry("ACTUAL_SPEED=%lld", sensor.getInput()),
-                entry("TARGET_SPEED=%lld", getTargetSpeed(sensor)));
+                entry("TARGET_SPEED=%lld", sensor.getTarget()));
 
         updateInventory(false);
     }