-
Notifications
You must be signed in to change notification settings - Fork 85
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Suggestion: NTP Client as IoT2020 has no battery backup. #24
Comments
Adding ntpd will be one of my first projects for a self-baked image. Hopefully I can get it working as stratum 1 server with a locally connected GPS receiver and even with PPS, but this depends on the hardware and kernel drivers. |
Busybox has a built-in service for this, we just lack its activation. But we are taking patches 😉. |
BusyBox's ntp client is small, but also has limitations. A short look at its REAME says for example that is doesn't support multiple servers. Nevertheless BusyBox will be appropriate for many users. |
Hmm, the code says the busybox client supports multiple servers. It also has an optional server mode as well, but I have no own experience to compare it to a full-blown one. |
I have now ntpd running using the standard recipe from OpenEmbedded and an adapted config file. Do you want a patch or merge request or a description how to get it into the image? |
Pull request, please. Regarding the style of the commit, follow other commits in this repo. The log should describe why we want this change and carry a signed-off according to https://developercertificate.org. I would suggest to preconfigure things generically for ntpclient operation, using e.g. the parameters of http://www.pool.ntp.org/en/use.html. Did you also try the busybox applet? In any case, please document the reason for choosing the full-featured one. |
sorry, I won't submit a pull request, because I quit using yocto and switch to debian. |
...at your own risk. |
Feature merged - thanks for the suggestion and inputs. |
It would be fantastic if an NTP client was added to facilitate setting the clock on power up.
The text was updated successfully, but these errors were encountered: