Request for Feedback :: Time Mode setting in timemanager

[Vishwanatha Subbanna]

Hi,

Further to the proposal I made to the community about removing TimeOwner feature, George Liu ( Thank you ) has put up a commit to :

- Remove TimeOwner feature :                          https://gerrit.openbmc-project.xyz/#/c/openbmc/phosphor-time-manager/+/31278/
- Consume changes to NTP/MANUAL settings immediately: https://gerrit.openbmc-project.xyz/#/c/openbmc/phosphor-time-manager/+/31284/

We have had many discussions with needed people here in IBM and we agree to the proposal. 

With that said: Both IPMI and PLDM will now use "/xyz/openbmc_project/time/bmc” as part of SetTime/GetTime instead of “/xyz/openbmc_project/time/host”

Thank you all for contributing into this discussion.

!! Vishwa !!

> On 25-Feb-2020, at 7:53 AM, Patrick Williams <patrick at stwcx.xyz> wrote:
> 
> On Tue, Feb 25, 2020 at 10:01:21AM +0800, Lei YU wrote:
>> On Tue, Feb 25, 2020 at 4:37 AM Patrick Williams <patrick at stwcx.xyz> wrote:
>>> 
>>> On Mon, Feb 24, 2020 at 11:38:56AM +0530, Vishwanatha Subbanna wrote:
>>>> Proposal for now is to:  *Remove the support for TimeOwner*. It will be as good as BOTH
>>> 
>>> "TimeOwner = BOTH" today creates two virtual clocks from the physical
>>> RTC by implementing the Host clock as an offset from the BMC clock,
>>> doesn't it?  Is that going to continue to be the functionality with your
>>> proposal or are you reverting back to a single physical clock where both
>>> Host and BMC can update?
>> 
>> "TimeOnwer = BOTH" does not creates two virtual clocks, "TimeOwner =
>> Split" does.
>> "BOTH" effectively enables both BMC and the Host to set the "single" clock.
> 
> Got it, my mistake.
> 
> -- 
> Patrick Williams