monitor: Move timer ops into TachSensor
Upcoming commits will touch the TachSensor timer
outside of the Fan class, and it will be cleaner to
encapsulate the operations in TachSensor.
Change-Id: I8584c44eb5dbe6bb93191a006d20dfc37596eeb3
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
diff --git a/monitor/fan.cpp b/monitor/fan.cpp
index 054c67d..a2e0386 100644
--- a/monitor/fan.cpp
+++ b/monitor/fan.cpp
@@ -27,7 +27,6 @@
{
using namespace phosphor::logging;
-using TimerType = phosphor::fan::util::Timer::TimerType;
constexpr auto INVENTORY_PATH = "/xyz/openbmc_project/inventory";
constexpr auto INVENTORY_INTF = "xyz.openbmc_project.Inventory.Manager";
@@ -90,8 +89,7 @@
void Fan::tachChanged(TachSensor& sensor)
{
- auto& timer = sensor.getTimer();
- auto running = timer.running();
+ auto running = sensor.timerRunning();
//If this sensor is out of range at this moment, start
//its timer, at the end of which the inventory
@@ -103,7 +101,7 @@
{
if (sensor.functional() && !running)
{
- timer.start(sensor.getTimeout(), TimerType::oneshot);
+ sensor.startTimer();
}
}
else
@@ -115,7 +113,7 @@
if (running)
{
- timer.stop();
+ sensor.stopTimer();
}
//If the fan was nonfunctional and enough sensors are now OK,