Skip to content

Conversation

MichelRottleuthner
Copy link
Contributor

Contribution description

This adapts the bh1750fvi driver to the new i2c api. I don't have hardware to test.

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 4, 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.

Untested ACK.
Changes look good, I also don't have the required hardware to test.

@dylad
Copy link
Member

dylad commented Jul 5, 2018

@MichelRottleuthner can you rebased this so we get rid of travis' error ?

@MichelRottleuthner
Copy link
Contributor Author

rebased

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

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

drivers/bh1750fvi: adapt to new i2c API
dylad pushed 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.

2 participants