[riot-notifications] [RIOT-OS/RIOT] drivers/rtc: Improved documentation (#11360)

Kaspar Schleiser notifications at github.com
Tue Apr 9 12:56:03 CEST 2019


> I took a look into the implementations of `rtc_set_alarm()`: Those don't match the current documentation nor do they match the proposal here. Most just ignore error handling at all. (E.g. the result of `mktime()` is just used as is, without checking the result.)

Then I actually think we shouldn't invest too much time here. Having *one* rtc alarm is not very useful anyways. IMO, rtc should be completely abstracted and used as low-power timer that gets multiplexed as usual. calendar / clock alarms should then be implemented on top of that.

For a low-level peripheral interface, returning "-1" on any error should be completely acceptable.

-- 
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/11360#issuecomment-481203600
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190409/3db99e40/attachment.html>


More information about the notifications mailing list