Skip to content
This repository has been archived by the owner on Jul 22, 2022. It is now read-only.

Explore replacing default agenttime.c adapter #17

Open
sandeepmistry opened this issue Apr 14, 2016 · 2 comments
Open

Explore replacing default agenttime.c adapter #17

sandeepmistry opened this issue Apr 14, 2016 · 2 comments
Assignees

Comments

@sandeepmistry
Copy link
Contributor

From discussion in #16 (comment)

The default agenttime.c is currently used, and then the POSIX based time layer is implemented in src/samd/time.cpp using RTCZero on SAMD.

We should explore using a custom agenttime.h implementation that is backed by the NTPClient. This would remove the dependency on RTCZero as the NTPClient can auto-sync the time with the NTP server at a preconfigured interval.

Then a UDP instance or NTPClient instance could be passed in the constructor. Another added benefit is the library is portable to any Arduino board that provides a SSLClient type and UDP type.

@tameraw
Copy link
Contributor

tameraw commented Apr 6, 2017

@sandeepmistry - We're looking at possibly setting up the NTP services during Platform_init().

@JetstreamRoySprowl
Copy link

JetstreamRoySprowl commented Apr 12, 2017

Good stuff, @sandeepmistry. We've captured this in our backlog as an improvement to make. Thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants