On Thu, Jul 21, 2022 at 01:20:47PM +0000, Geva, Erez wrote:
> Hi,
> 
> I think the "if (type != TS_SOFTWARE" is wrong.
> The "type" only refer to TX.

No, "type" is an enumerated value, timestamp_type, and it describes
the flavor of time stamping in use.

> But hwts_init() also set the RX.
> So no reason to skip it.
> Setting to TS_SOFTWARE means no TX hardware time stamp.

There is no such thing as mixing HW/SW time stamping for Rx/Tx.
They have to match, otherwise the whole thing breaks.

Thanks,
Richard


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

Reply via email to