Skip to content

Conversation

MichelRottleuthner
Copy link
Contributor

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

@MichelRottleuthner MichelRottleuthner added Area: drivers Area: Device drivers TF: I2C Marks issues and PRs related to the work of the I²C rework task force labels Jul 3, 2018
Copy link
Member

@dylad dylad left a 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 !

@tcschmidt tcschmidt requested a review from MrKevinWeiss July 6, 2018 23:36
@MrKevinWeiss
Copy link
Contributor

@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.

Copy link
Contributor

@MrKevinWeiss MrKevinWeiss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@dylad
Copy link
Member

dylad commented Jul 9, 2018

Can you rebase please ? so Travis will be happy.

@MichelRottleuthner
Copy link
Contributor Author

rebased

Copy link
Member

@dylad dylad left a 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 !

@dylad dylad merged commit a9c421f into RIOT-OS:new_i2c_if Jul 9, 2018
basilfx pushed a commit to basilfx/RIOT that referenced this pull request Jul 10, 2018
…new_i2c

drivers/lc709203f: adapt to new i2c API
dylad added a commit to dylad/RIOT that referenced this pull request Jul 10, 2018
…new_i2c

drivers/lc709203f: adapt to new i2c API
@shr70
Copy link
Contributor

shr70 commented Jul 11, 2018

Eventhough its already merged: Just tested it on the jiminy-mega256rfr2 and can confirm the changes as working

@MichelRottleuthner
Copy link
Contributor Author

thanks for testing @shr70 :)

@dylad
Copy link
Member

dylad commented Jul 11, 2018

@shr70 thanks a lot for testing. This is highly appreciate !

dylad added a commit that referenced this pull request Jul 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: drivers Area: Device drivers TF: I2C Marks issues and PRs related to the work of the I²C rework task force
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants