sensor: Add ICM42686 Driver Support #92579
Open
+1,675
−1,433
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ubieda
force-pushed
the
feature/icm42686-driver-support
branch
3 times, most recently
from
b71b4bd to
729ca0b
Compare
As a first step to enable the similar variants (e.g: ICM42686), refactor common functionality into icm4268x files. As a result, applications using the icm42688 will need to have both compatible properties: "invensense,icm42688" and "invensense,icm4268x" defined. In-tree boards have been modified to comply with this pattern. This patch does not contain functional changes. The driver should work the same as before. Signed-off-by: Luis Ubieda <luisf@croxel.com>
ubieda
force-pushed
the
feature/icm42686-driver-support
branch
from
729ca0b to
0f64e78
Compare
Addressing low-hanging fruits. Put in a separate commit in order to make it easier to keep track of changes. Signed-off-by: Luis Ubieda <luisf@croxel.com>
Now this driver supports both ICM42688 and ICM42686. Tested with read-decode as well as streaming mode. Signed-off-by: Luis Ubieda <luisf@croxel.com>
When IMU is set to stream with High-res mode, the FSR settings are fixed: - ICM42688 at 16g and 2000dps. - ICM42686 at 32g and 4000dps. Unless this is enforced, the driver will provide incorrect scaling for readings coming through streaming when the current FS setting is any different than these. Signed-off-by: Luis Ubieda <luisf@croxel.com>
Additionally, remove overlay from sensor-shell sample, which is redundant. Signed-off-by: Luis Ubieda <luisf@croxel.com>
For build-time validation. Signed-off-by: Luis Ubieda <luisf@croxel.com>
ubieda
force-pushed
the
feature/icm42686-driver-support
branch
from
0f64e78 to
7477da4
Compare
|
|
Looks like the issues flagged by SonarQube are due to moving existing in-tree files. I wonder if we're good with addressing them on follow-up PRs instead of blocking this one. |
github-actions
bot
added
area: Samples
github-actions
bot
requested review from
afontaine-invn,
asemjonovs,
avisconti,
bperseghetti,
danieldegrasse,
gjabouley-invn,
jeppenodgaard,
manuargue,
MaureenHelm and
PetervdPerk-NXP
github-actions
bot
requested review from
rbuisson-invn,
sriccardi-invn,
teburd,
tristan-google and
yperess
bperseghetti
approved these changes
Jul 4, 2025
teburd
approved these changes
Jul 4, 2025
PetervdPerk-NXP
approved these changes
Jul 7, 2025
afontaine-invn
approved these changes
Jul 8, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR adds support for ICM42686 driver, by extending and refactoring existing ICM42688 driver.
The new variant contains feature parity with the original driver:
Testing
This was tested on VMU RT1170, which contains both ICM42688 and ICM42686, running the sensor-shell sample to set up and obtain sensor readings: