[riot-notifications] [RIOT-OS/RIOT] drivers/periph/ptp_clock (#15074)

Marian Buschsieweke 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:
https://github.com/RIOT-OS/RIOT/pull/15074#issuecomment-729493279
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20201117/c3444c29/attachment.htm>


More information about the notifications mailing list