-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
drivers: rtc: new maxim ds1307 rtc driver #64219
Conversation
0b8eaae
to
324ec8c
Compare
@bjarki-trackunit Kindly have a look |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall it looks good, I have a few requests I have outlined that need to be addressed :)
b428951
to
e5a717e
Compare
@bjarki-trackunit Please review |
Looks good, last request, could you run the RTC API test suite? (ignore the y2k test failing once you run it) |
@bjarki-trackunit I ran RTC API test_suite. All checks are passed except yday, isdst, nsec. Because this chip does not support that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This last change should allow it to fail only because of the missing yday :)
e5a717e
to
0773194
Compare
@bjarki-trackunit Please revisit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!
@bjarki-trackunit Thanks for the review |
0773194
to
2665551
Compare
Added rtc driver for ds1307. It allows to read and set the date and time Signed-off-by: Arunmani Alagarsamy <[email protected]>
2665551
to
5dcb7b2
Compare
@bjarki-trackunit @fabiobaltieri Gentle ping! |
No need to ping, GitHub notifications already do it for you. :-) |
Added rtc driver for ds1307. It allows to read and set the date and time