Skip to content

drivers/mag3110: apply unified params definition scheme#8758

Merged
aabadie merged 3 commits intoRIOT-OS:new_i2c_iffrom
aabadie:pr/drivers/params/mag3110
Jul 4, 2018
Merged

drivers/mag3110: apply unified params definition scheme#8758
aabadie merged 3 commits intoRIOT-OS:new_i2c_iffrom
aabadie:pr/drivers/params/mag3110

Conversation

@aabadie
Copy link
Copy Markdown
Contributor

@aabadie aabadie commented Mar 8, 2018

Contribution description

This PR update the params definitions scheme for this device driver.
Also forgot this one last week.

Issues/PRs references

Initially done in #7937 and related to #7519

#6577

@aabadie aabadie added Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation Area: drivers Area: Device drivers CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Mar 8, 2018
Copy link
Copy Markdown
Contributor Author

@aabadie aabadie left a comment

Choose a reason for hiding this comment

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

anyone for reviewing this, it should be fast @miri64 ?

@smlng
Copy link
Copy Markdown
Member

smlng commented Jun 29, 2018

@aabadie please rebase and do you want to add I2C API adaption, too? This one is missing for the I2C rework but this PR is referenced anyway. However, then please rebase onto new_i2c_if branch, first.

@aabadie aabadie force-pushed the pr/drivers/params/mag3110 branch from 6264419 to 45a8a83 Compare June 29, 2018 13:52
@aabadie aabadie changed the base branch from master to new_i2c_if June 29, 2018 13:52
@aabadie
Copy link
Copy Markdown
Contributor Author

aabadie commented Jun 29, 2018

The work was already done, but I forgot to push it and to change the base branch...

@aabadie aabadie removed the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jul 2, 2018
@aabadie aabadie force-pushed the pr/drivers/params/mag3110 branch from 45a8a83 to a73efed Compare July 2, 2018 06:44
@aabadie aabadie added the TF: I2C Marks issues and PRs related to the work of the I²C rework task force label Jul 2, 2018
Copy link
Copy Markdown
Member

@smlng smlng left a comment

Choose a reason for hiding this comment

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

tested ACK, used pba-d-01-kw2x with onboard sensor

@aabadie aabadie merged commit 47b4c22 into RIOT-OS:new_i2c_if Jul 4, 2018
basilfx pushed a commit to basilfx/RIOT that referenced this pull request Jul 10, 2018
drivers/mag3110: apply unified params definition scheme
dylad pushed a commit to dylad/RIOT that referenced this pull request Jul 10, 2018
drivers/mag3110: apply unified params definition scheme
dylad pushed a commit that referenced this pull request Jul 11, 2018
drivers/mag3110: apply unified params definition scheme
@aabadie aabadie deleted the pr/drivers/params/mag3110 branch September 23, 2018 08:47
@aabadie aabadie added this to the Release 2018.10 milestone Nov 5, 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 Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants