summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVishwanatha Subbanna <vishwa@linux.vnet.ibm.com>2017-07-05 13:15:19 +0530
committerVishwanatha Subbanna <vishwa@linux.vnet.ibm.com>2017-07-05 14:07:25 +0530
commit25b4ce95f9d0bea33fbf0364e72f304542a19bc5 (patch)
tree80c8703f9c121ca6cd43e9ff2f744f8727310dd9
parent2a96cfc0ed4e5466793a82292ee5640c992b963c (diff)
downloadphosphor-time-manager-25b4ce95f9d0bea33fbf0364e72f304542a19bc5.tar.gz
phosphor-time-manager-25b4ce95f9d0bea33fbf0364e72f304542a19bc5.zip
Default Mode and Owner to MANUAL and BOTH respectively
Time manager tries to read time settings from settings daemon and in the case it fails, it would set NTP as default mode and BMC as default owner. Doing this was prohibiting Host from setting the time and it was seen as a concern having to reboot to set the time post changing the setting. Making default owner to BOTH will make sure that HOST / BMC can set the time. Having mode as MANUAL comes as a need since setting the time having mode as NTP results in failure. Fixes openbmc/phosphor-time-manager#2 Change-Id: I491213a8fc2f6d4e1a373d436cb926ad79f44d22 Signed-off-by: Vishwanatha Subbanna <vishwa@linux.vnet.ibm.com>
-rw-r--r--time-config.cpp8
1 files changed, 4 insertions, 4 deletions
diff --git a/time-config.cpp b/time-config.cpp
index b46ff54..7dbd5c4 100644
--- a/time-config.cpp
+++ b/time-config.cpp
@@ -22,10 +22,10 @@ std::map<std::string, TimeConfig::FUNCTOR> TimeConfig::iv_TimeParams = {
TimeConfig::TimeConfig() :
iv_dbus(nullptr),
- iv_CurrTimeMode(timeModes::NTP),
- iv_RequestedTimeMode(timeModes::NTP),
- iv_CurrTimeOwner(timeOwners::BMC),
- iv_RequestedTimeOwner(timeOwners::BMC),
+ iv_CurrTimeMode(timeModes::MANUAL),
+ iv_RequestedTimeMode(timeModes::MANUAL),
+ iv_CurrTimeOwner(timeOwners::BOTH),
+ iv_RequestedTimeOwner(timeOwners::BOTH),
iv_CurrDhcpNtp("yes"),
iv_SettingChangeAllowed(false),
iv_SplitModeChanged(false)
OpenPOWER on IntegriCloud