Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PTP disabled #17

Open
thecaptury opened this issue Nov 29, 2019 · 2 comments
Open

PTP disabled #17

thecaptury opened this issue Nov 29, 2019 · 2 comments

Comments

@thecaptury
Copy link

Hi,

is there a reason that PTP support is disabled by default?

In the header tn40.h uncommenting TN40_PTP allows me to use PTP and it doesn't appear to have any negative side effects...

@acooks
Copy link
Owner

acooks commented Jan 8, 2020

Thanks for the report!
I don't know why the PTP support is disabled by default, but I also don't have a way to test it. Could you provide any more information about the environment you tested in? Kernel version, hardware device, and any other test equipment would provide a valuable reference point.

@thecaptury
Copy link
Author

I tested on stock kernels on Debian 9, Debian 10, Ubuntu 16.04, Ubuntu 18.04, and Ubuntu 19.10.

I also closely examined the source code. I don't see anything that could go wrong. It essentially just publishes the timestamp when a packet is pushed to the hardware to the meta data of the packet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants