On Mon, Feb 15, 2021 at 09:58:53AM -0500, vincent.cheng...@renesas.com wrote:
> @@ -1730,7 +1743,10 @@ enum servo_state clock_synchronize(struct clock *c,
> tmv_t ingress, tmv_t origin)
>
> c->cur.offsetFromMaster = tmv_to_TimeInterval(c->master_offset);
>
> - if (c->free_running) {
> + if (c->free_running || c->step_window_counter) {
> + if (c->step_window_counter) {
> + c->step_window_counter--;
> + }
> return clock_no_adjust(c, ingress, origin);
This isn't quite right. clock_no_adjust() will return the wrong servo
state. Also, we want to prevent the Sync messages from entering the
tsproc.
I think the correct way is to drop the Sync messages altogether.
I hacked your patch with my suggested changes, and I'll post it
shortly. Please review it and let me know what you think.
Thanks,
Richard
_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel
