On Wed, Jan 12, 2022 at 12:08:00PM +0100, Miroslav Lichvar wrote:

> If the only reason for these ifdefs is an optimization for older
> kernels, I think it's ok to remove them as this runs only once on
> start. I'd prefer code that is easier to read, even if it does
> something unnecessary.

+1

> I'd prefer these ifdefs to be removed too, but if others agree it
> should stay, would it make sense to move it to port_change_phc(),
> which would accept the fd and port_link_status, so it could call
> rtnl_link_status() if the support is missing?

Agree that the ifdefs should go.

Thanks,
Richard


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

Reply via email to