[riot-notifications] [RIOT-OS/RIOT] cpu/kinetis/rtc: add rtc_set_compensation() (#12280)

MichelRottleuthner notifications at github.com
Wed Sep 25 19:43:54 CEST 2019


When I saw this PR and remembered that I did something similar a while ago so I just like to share my opinion here, feel free to ignore ;)
Actually I think this feature is important and generic enough that it should be provided by the RTC API for all platforms (not saying that you should do all the implementation of course^^). But coming from that point, IMO it would be better (easier to use and understand) if the interface used some generically applicable values for the compensation function e.g. ppm or ppb as a signed number and have the function calculate the respective register values internally.

-- 
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/12280#issuecomment-535133276
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190925/0b54f879/attachment.htm>


More information about the notifications mailing list