On 09/03/18 01:40, Richard Cochran wrote:
For the kernel, I (and others) would like to get away from the clunky
timespec altogether. Dividing a time value into two fields might have
made sense 30 years ago when operations on eight byte words were
expensive, but today it makes no sense. It only causes pointless
extra computation, first in user space and again the kernel,
converting to and from the timespec.
How about this?
SOF_TIMESTAMPING_HIGH_RES requests time stamps in a new format:
struct {
uint64_t nanoseconds;
uint32_t fractional_nanoseconds;
uint32_t reserved;
};
Would that be enough resolution for you?
Yes, that's plenty of resolution for us. The clock in question is a
nominal 64GHz.
I'll put together a patch based on your suggested format.
Also, will you be able to mainline your driver? It helps new kernel
API acceptance when there is an actual, realistic use case and users.
I'm aiming to get the code up on GitHub as an out-of-tree driver today
or early next week, and we'll be mainlining it as soon as possible.
We can also send you a pair of sample devices, if that would be of interest.
Michael
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Linuxptp-devel mailing list
Linuxptp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linuxptp-devel
