commit 6fd9e440be73865d99cd408e0f87ec639882267b
author tony lee <tony.lee@quantatw.com> Tue Apr 23 09:09:15 2019 +0800
committer Tony Lee <tony.lee@quantatw.com> Wed May 15 17:27:05 2019 +0800
tree 83aaf6b86443b88a132a310c9925908ff7df3fb2
parent ab3b247f0406eeb4884131dfac44be870786abb6

Update the value of Period to D-bus obtained from configuration YAML.

Update the value of Period to D-bus obtained from configuration YAML
when the target group was asserted. The logical behavior
of Period is as same as Dutyon.

Change-Id: I4b064f2f7321479cabc7deeaa57779b0a8cc3d21
Signed-off-by: tony lee <tony.lee@quantatw.com>

manager.cpp

diff --git a/manager.cpp b/manager.cpp
index 0235b7b..04398e4 100644
--- a/manager.cpp
+++ b/manager.cpp
@@ -118,7 +118,8 @@
             std::string objPath = std::string(PHY_LED_PATH) + it.name;
             log<level::DEBUG>("De-Asserting LED",
                               entry("NAME=%s", it.name.c_str()));
-            drivePhysicalLED(objPath, Layout::Action::Off, it.dutyOn);
+            drivePhysicalLED(objPath, Layout::Action::Off, it.dutyOn,
+                             it.period);
         }
     }
 
@@ -129,7 +130,7 @@
             std::string objPath = std::string(PHY_LED_PATH) + it.name;
             log<level::DEBUG>("Asserting LED",
                               entry("NAME=%s", it.name.c_str()));
-            drivePhysicalLED(objPath, it.action, it.dutyOn);
+            drivePhysicalLED(objPath, it.action, it.dutyOn, it.period);
         }
     }
     return;
@@ -137,7 +138,8 @@
 
 // Calls into driving physical LED post choosing the action
 void Manager::drivePhysicalLED(const std::string& objPath,
-                               Layout::Action action, uint8_t dutyOn)
+                               Layout::Action action, uint8_t dutyOn,
+                               const uint16_t period)
 {
     using namespace phosphor::logging;
 
@@ -153,6 +155,7 @@
     if (action == Layout::Action::Blink)
     {
         drivePhysicalLED(service->second, objPath, "DutyOn", dutyOn);
+        drivePhysicalLED(service->second, objPath, "Period", period);
     }
     drivePhysicalLED(service->second, objPath, "State",
                      getPhysicalAction(action));