Re: [Linuxptp-devel] [PATCH v1] pi: Fix drift history when entering SERVO_LOCKED state

2023-08-22 Thread Maciek Machnikowski
On 8/22/2023 1:50 PM, Miroslav Lichvar wrote: > On Tue, Aug 22, 2023 at 12:34:35PM +0200, Maciek Machnikowski wrote: >> root@equinox:[~]#: phc_ctl ens6f0np0 set freq 5000 >> root@equinox:[~]#: sudo ptp4l -m -i ens6f0np0 -2 -s > >> ptp4l[248.614]: master offset -301008874 s0 freq -5000 p

Re: [Linuxptp-devel] [PATCH v1] pi: Fix drift history when entering SERVO_LOCKED state

2023-08-22 Thread Miroslav Lichvar
On Tue, Aug 22, 2023 at 12:34:35PM +0200, Maciek Machnikowski wrote: > root@equinox:[~]#: phc_ctl ens6f0np0 set freq 5000 > root@equinox:[~]#: sudo ptp4l -m -i ens6f0np0 -2 -s > ptp4l[248.614]: master offset -301008874 s0 freq -5000 path delay > 1136508 > ptp4l[249.615]: master offset -316

Re: [Linuxptp-devel] [PATCH v1] pi: Fix drift history when entering SERVO_LOCKED state

2023-08-22 Thread Maciek Machnikowski
On 8/22/2023 11:24 AM, Miroslav Lichvar wrote: > On Mon, Aug 21, 2023 at 07:02:55PM +0200, Maciek Machnikowski wrote: >> On 8/21/2023 2:04 PM, Miroslav Lichvar wrote: >>> s->drift is the drift of the uncompensated clock. It's not supposed to be >>> scaled by anything. >> If that's the intended us

Re: [Linuxptp-devel] [PATCH v1] pi: Fix drift history when entering SERVO_LOCKED state

2023-08-22 Thread Miroslav Lichvar
On Mon, Aug 21, 2023 at 07:02:55PM +0200, Maciek Machnikowski wrote: > On 8/21/2023 2:04 PM, Miroslav Lichvar wrote: > > s->drift is the drift of the uncompensated clock. It's not supposed to be > > scaled by anything. > If that's the intended use - then the line 147: > s->drift += ki_term; > > wh