Andy Duan <fugang.d...@nxp.com> writes: > From: Sergey Organov <sorga...@gmail.com> Sent: Monday, July 6, 2020 10:26 PM >> PPS feature could be useful even when hardware time stamping of network >> packets is not in use, so remove offending check for this condition from >> fec_ptp_enable_pps(). > > If hardware time stamping of network packets is not in use, PPS is > based on local > clock, what is the use case ?
First, having special code to disable something that does no harm seems to be wrong idea in general. In this particular case, if PPS is not needed, it is still disabled by default, and one is still free not to use it. Then, as I'm not aware of a rule that renders PPS based on local clock useless, I'm to assume it might be useful. Finally, as an attempt to give direct answer to your question, suppose I have external device that is capable to time stamp PPS against known time scale (such as GPS system time) with high precision. Now I can get nice estimations of local time drifts and feed, say, "chrony", with the data to adjust my local clock accordingly. Thanks, -- Sergey
