summaryrefslogtreecommitdiffstats
path: root/monitor/fan.cpp
diff options
context:
space:
mode:
authorMatthew Barth <msbarth@us.ibm.com>2018-01-15 16:22:04 -0600
committerMatthew Barth <msbarth@us.ibm.com>2018-01-18 11:16:23 -0600
commitf552ea5ce23889905ce15c0ffcfc73fa1245a423 (patch)
treed3621f920bebf9fd9744aeb61d2a5fb54a2f419e /monitor/fan.cpp
parente4338cdb856db2b71f8adf42c50603d816980f8a (diff)
downloadphosphor-fan-presence-f552ea5ce23889905ce15c0ffcfc73fa1245a423.tar.gz
phosphor-fan-presence-f552ea5ce23889905ce15c0ffcfc73fa1245a423.zip
All sensors should return a target speed value
All tach sensors associated with a fan should return a target speed sensor from its getTarget function. In the case where a target speed sensor does not exist for the tach sensor, it retrieves and returns the target speed value from the fan where the fan finds the target speed value from a tach sensor the fan contains that provides it. Resolves openbmc/openbmc#2784 Change-Id: Iea5561b0aad6942be52af262c7255c60e5e75c7a Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
Diffstat (limited to 'monitor/fan.cpp')
-rw-r--r--monitor/fan.cpp36
1 files changed, 17 insertions, 19 deletions
diff --git a/monitor/fan.cpp b/monitor/fan.cpp
index a6af74a..f24312d 100644
--- a/monitor/fan.cpp
+++ b/monitor/fan.cpp
@@ -134,34 +134,32 @@ void Fan::tachChanged(TachSensor& sensor)
}
-uint64_t Fan::getTargetSpeed(const TachSensor& sensor)
+uint64_t Fan::findTargetSpeed()
{
uint64_t target = 0;
-
- if (sensor.hasTarget())
- {
- target = sensor.getTarget();
- }
- else
+ //The sensor doesn't support a target,
+ //so get it from another sensor.
+ auto s = std::find_if(_sensors.begin(), _sensors.end(),
+ [](const auto& s)
+ {
+ return s->hasTarget();
+ });
+
+ if (s != _sensors.end())
{
- //The sensor doesn't support a target,
- //so get it from another sensor.
- auto s = std::find_if(_sensors.begin(), _sensors.end(),
- [](const auto& s)
- {
- return s->hasTarget();
- });
-
- if (s != _sensors.end())
- {
- target = (*s)->getTarget();
- }
+ target = (*s)->getTarget();
}
return target;
}
+uint64_t Fan::getTargetSpeed(const TachSensor& sensor)
+{
+ return sensor.getTarget();
+}
+
+
bool Fan::tooManySensorsNonfunctional()
{
size_t numFailed = std::count_if(_sensors.begin(), _sensors.end(),
OpenPOWER on IntegriCloud