-
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
sensor: tdk 42688 driver #52911
sensor: tdk 42688 driver #52911
Conversation
130ccb4
to
4fd3c6f
Compare
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.
I'm good with this, I have some patches I can PR once in.
4fd3c6f
to
55c9b96
Compare
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.
Please add the sensor to tests/drivers/build_all/sensor
to ensure it gets built in CI
bb2c8c3
to
1822dbe
Compare
@MaureenHelm I tried this but ran into a few issues (see the second commit in this PR)
|
c409918
to
7005493
Compare
7005493
to
7084c4e
Compare
Discussed with @teburd offline and there's no hard requirement for the SPI_SAM, CI should pass now |
7084c4e
to
b51754a
Compare
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.
Minor nit, but otherwise looks good. Also needs a rebase
b51754a
to
32b6195
Compare
c501ab7
to
ab4f363
Compare
Adds a driver for TDK InvenSense 42688 six axis IMU. Verified using the sensor shell sample app via: - sensor info - sensor get icm42688p@0 Signed-off-by: Tom Burdick <[email protected]> Signed-off-by: Yuval Peress <[email protected]>
The test at tests/drivers/build_all/sensor wasn't passing since a lot of nodes in both the i2c and spi had conflicting addresses. Fix that by making the addresses sequential. Signed-off-by: Yuval Peress <[email protected]>
Add a node for the icm42688 so that it builds as a part of the CI. Signed-off-by: Yuval Peress <[email protected]>
ab4f363
to
8ca6650
Compare
Adds a driver for TDK InvenSense 42688 six axis IMU. Verified using the sensor shell sample app via:
sensor info
sensor get icm42688p@0
Collaborated on by Tom Burdick and Yuval Peress