On Mon, Dec 16, 2019 at 04:04:21PM -0800, Cliff Spradlin via Linuxptp-devel 
wrote:
> Could you please check my work?

Hm, I think you are right.  We define the offset in terms of the slave
and master time values taken at the same instant:

        offset = slave - master

The sysoff_measure() routine provides 'offset' and 'slave', but for
this case we want to reverse the roles.  The needed local time stamp
is the 'master' time value:

        master = slave - offset

And so the phc2sys.c starting at line 767 should read:

        /* use reversed sysoff */
        if (sysoff_measure(CLOCKID_TO_FD(clock->clkid),
                           clock->sysoff_method,
                           node->phc_readings,
                           &offset, &ts, &delay) < 0)
                return -1;
        offset = -offset;
        ts += offset;

Miroslav, care to double check that?

Thanks,
Richard




_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel

Reply via email to