commit 3c2f44962c163668272c473ddfb9c35bf8774570
author George Liu <liuxiwei@inspur.com> Sun Apr 12 11:35:57 2020 +0800
committer Vishwanatha Subbanna <vishwa@linux.vnet.ibm.com> Tue May 19 07:59:22 2020 +0000
tree 12edb61db0e979d9ef771001e22ac3a6eb7dea35
parent c09ac3fa78fff6fc8caad85b5ccfbfba311a2722

Remove TimeOwner Feature

The TimeOwner feature is confusing from feedback from the community and
hence removing the feature.
Remove the TimeOwner feature in the phosphor-time-manager repo and
needed settings objects.

Tested: Manually set the date time on the web and successfully update
        the date time of BMC (eg: 2020/01/01 08:07:50).
        busctrl get-property xyz.openbmc_project.Time.Manager
                /xyz/openbmc_project/time/bmc
                xyz.openbmc_project.Time.EpochTime Elapsed
        t 1577837156385836

Refer: https://lists.ozlabs.org/pipermail/openbmc/2020-April/021409.html

Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: Id47eb0a03e0e94eeff29d2b77dccefb89cded7b8

settings.cpp

diff --git a/settings.cpp b/settings.cpp
index 9add59d..61a575b 100644
--- a/settings.cpp
+++ b/settings.cpp
@@ -17,8 +17,7 @@
 
 Objects::Objects(sdbusplus::bus::bus& bus) : bus(bus)
 {
-    std::vector<std::string> settingsIntfs = {timeOwnerIntf, timeSyncIntf,
-                                              hostStateIntf};
+    std::vector<std::string> settingsIntfs = {timeSyncIntf, hostStateIntf};
     auto depth = 0;
 
     auto mapperCall = bus.new_method_call(mapperService, mapperPath, mapperIntf,
@@ -51,11 +50,7 @@
         {
             for (const Interface& interface : service_iter.second)
             {
-                if (timeOwnerIntf == interface)
-                {
-                    timeOwner = path;
-                }
-                else if (timeSyncIntf == interface)
+                if (timeSyncIntf == interface)
                 {
                     timeSyncMethod = path;
                 }