adding sync method in phosphor-time-manager

[Patrick Williams]

On Tue, Oct 06, 2020 at 05:38:13PM +0000, Velumani T-ERS,HCLTech wrote:
> Classification:
> Hi Team,
> 
> We wanted to add another time sync method in phosphor-time-manager to get the time from the host and set it to BMC. To have this option configurable I propose a dbus property in the time interface(given below). Please provide your feedback/comments.
> 
> https://github.com/openbmc/phosphor-dbus-interfaces/blob/master/xyz/openbmc_project/Time/Synchronization.interface.yaml
> enumerations:
>     - name: Method
>       description: >
>         Possible methods of time synchronization.
>       values:
>         - name: NTP
>           description: >
>             Sync by using the Network Time Protocol.
>         - name: Manual
>           description: >
>             Sync time manually.
>          - name: HostSync
>           description: >
>             Sync the time from host.
> 

Hopefully Vishwa can weigh in here.

It is my understanding that the default implementation allows the host
to synchronize the time down already.  There use to be an interface that
determined the "TimeOwner" to specify if the BMC or the Host were in
charge of the time.  That was removed not too long ago.

https://github.com/openbmc/phosphor-dbus-interfaces/commit/70c76a29b898f05e28c803808bd57a2b6c3f7a6f

I think you should look through the history on this commit (it
references a discussion related to the mailing list) and check with the
people in that chain as to why it was removed before we add back in
something very similar.

-- 
Patrick Williams
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20201006/822c00d5/attachment.sig>