Skip to content
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

Add basic RTC API #26

Merged
merged 6 commits into from Aug 13, 2019
Merged

Add basic RTC API #26

merged 6 commits into from Aug 13, 2019

Conversation

hannobraun
Copy link
Contributor

@hannobraun hannobraun commented Aug 9, 2019

This pull request adds a very basic RTC API. It can be used to read and set the time, but no alarms or other interrupts are supported yet. There are also a few commits adding general debug stuff to the time module, as that was useful to me while I developed this.

I'm marking this PR as a draft, as it's based on #25. Otherwise, this is ready to review.

cc @lthiery

@hannobraun
Copy link
Contributor Author

hannobraun commented Aug 12, 2019

Build fails for STM32L0x1 due to warnings. Will fix shortly.

@hannobraun hannobraun marked this pull request as ready for review Aug 12, 2019
@hannobraun
Copy link
Contributor Author

hannobraun commented Aug 12, 2019

There was also another problem that was masked by the first: The RTC example only works on STM32L0x2.

Everything's fixed now. Pull request has been rebased. I've rolled the fixes into the existing commits.

This is ready for review now.

@arkorobotics arkorobotics merged commit 977af77 into stm32-rs:master Aug 13, 2019
@arkorobotics
Copy link
Member

arkorobotics commented Aug 13, 2019

You rock! Thanks again for all your recent PRs!

@hannobraun hannobraun deleted the rtc branch Aug 13, 2019
@hannobraun
Copy link
Contributor Author

hannobraun commented Aug 13, 2019

You rock! Thanks again for all your recent PRs!

Thank you :-)
And thank you again for all the reviewing/merging!

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

Successfully merging this pull request may close these issues.

None yet

2 participants