commit 0ce353ec8de4cd7b7dc48474b0c0ff8894816316
author William A. Kennington III <wak@google.com> Tue Oct 30 18:30:36 2018 -0700
committer William A. Kennington III <wak@google.com> Wed Oct 31 12:28:27 2018 -0700
tree 6f3b3ac451cc21fd2e2fd3ef8028dcaf3866433a
parent 1cfc2f11b13412a15f8478cebc35e50e6feb13a2

control: Alias TimerType locally

This is needed for a future commit that removes the local
phosphor::util::Timer implementation.

Tested:
    Built and run through unit tests.

Change-Id: I8ea3399fec8761055d4a0d94fb9ea91fd34040ea
Signed-off-by: William A. Kennington III <wak@google.com>

control/gen-fan-zone-defs.py

diff --git a/control/gen-fan-zone-defs.py b/control/gen-fan-zone-defs.py
index e5aa39b..0ddfb63 100755
--- a/control/gen-fan-zone-defs.py
+++ b/control/gen-fan-zone-defs.py
@@ -127,7 +127,7 @@
                         param = (
                             "Timer{static_cast<std::chrono::seconds>(" +
                             str(eActions[p]['delay']) + "), " +
-                            "util::Timer::TimerType::" +
+                            "TimerType::" +
                             str(eActions[p]['type']) + "}")
                     else:
                         param += (str(eActions[p]['type']).lower() + ">(")
@@ -257,7 +257,7 @@
     else:
         timer['interval'] = (interval +
                              "(" + str(0) + ")")
-    timer['type'] = "util::Timer::TimerType::repeating"
+    timer['type'] = "TimerType::repeating"
     event['timer'] = timer
 
     return event
@@ -409,7 +409,7 @@
     else:
         timer['interval'] = (interval +
                              "(" + str(0) + ")")
-    timer['type'] = "util::Timer::TimerType::repeating"
+    timer['type'] = "TimerType::repeating"
     precond['pctime'] = timer
 
     return precond

control/types.hpp

diff --git a/control/types.hpp b/control/types.hpp
index bb9367b..f565333 100644
--- a/control/types.hpp
+++ b/control/types.hpp
@@ -64,8 +64,9 @@
 
 constexpr auto intervalPos = 0;
 constexpr auto typePos = 1;
+using TimerType = phosphor::fan::util::Timer::TimerType;
 using Timer = std::tuple<std::chrono::seconds,
-                         util::Timer::TimerType>;
+                         TimerType>;
 
 constexpr auto sigMatchPos = 0;
 constexpr auto sigHandlerPos = 1;

control/zone.cpp

diff --git a/control/zone.cpp b/control/zone.cpp
index 0050899..11246db 100644
--- a/control/zone.cpp
+++ b/control/zone.cpp
@@ -73,8 +73,7 @@
         // Start timer for fan speed decreases
         if (!_decTimer.running() && _decInterval != seconds::zero())
         {
-            _decTimer.start(_decInterval,
-                            util::Timer::TimerType::repeating);
+            _decTimer.start(_decInterval, TimerType::repeating);
         }
     }
 }
@@ -245,8 +244,7 @@
         }
         setSpeed(requestTarget);
         // Start timer countdown for fan speed increase
-        _incTimer.start(_incDelay,
-                        util::Timer::TimerType::oneshot);
+        _incTimer.start(_incDelay, TimerType::oneshot);
     }
 }