[riot-notifications] [RIOT-OS/RIOT] drivers/periph/ptp_clock (#15074)
notifications at github.com
Wed Nov 18 08:32:08 CET 2020
> Looks good, but I'm a bit confused about `PTPTTHR` being a 32 bit register, yet you store `seconds` in a `uint64_t` - why go through all the trouble with 64 bit arithmetic if the hardware is going to throw all those extra bits away anyway?
The PTP protocol defines time as a structure containing an 48 bit seconds and an 32 bit nanoseconds part. But 32 bit would certainly be sufficient until year 2106 - so maybe I should indeed replicate the design choice of ST to just crop the seconds to 32 bit.
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications