-
Notifications
You must be signed in to change notification settings - Fork 2k
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/lc709203f: adapt to new i2c API #9481
drivers/lc709203f: adapt to new i2c API #9481
Conversation
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 changes are ok.
Would be great if someone can test it !
@dylad So it builds for me and the code looks good. I don't think we can find the hardware so I will add it to the list and you can merge if you like. |
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.
Looks good!
Can you rebase please ? so Travis will be happy. |
93e2c5f
to
79da68d
Compare
rebased |
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.
Untested ACK.
Let's go !
…new_i2c drivers/lc709203f: adapt to new i2c API
…new_i2c drivers/lc709203f: adapt to new i2c API
Eventhough its already merged: Just tested it on the jiminy-mega256rfr2 and can confirm the changes as working |
thanks for testing @shr70 :) |
@shr70 thanks a lot for testing. This is highly appreciate ! |
drivers/lc709203f: adapt to new i2c API
Contribution description
This adapts the lc709203f driver to the new i2c api. I don't have hardware to test. (maybe @shr70 want's to help out here?)
(note for future PR: the driver has a lot of code that can be deduplicated)
Issues/PRs references
#6577