Implement ICM45686 I2CM interface #105
Draft
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
Do not re-trigger one shot Cleanup debug logs
Fix single/continuous mode switching Fix triggering one shot mode when continuous Add a warning for long sensor update
some mags were not in the mag list, and qmc5883l had incorrect address and swapped with hmc5883l
Conflicts resolved: src/sensor/sensor.c src/sensor/sensors.h src/sensor/sensors_enum.h
Revert sensor mode changes Fix ODR for single shot mode Always check DRDY flag Cleanup debug traces
Support QMC6309 magnetometer
odr already guaranteed to be >=0 if this statement is reached
fixes issue where the loaded data is empty
Add board configuration and sensor orientation
config recommended by @kounocom
* Mochi * Update Mochi * Format * fix: board: mochi: Formatting --------- Co-authored-by: ShineBrightMeow <[email protected]>
* ci: nRF-Connect SDK 3.1 Stable Branch * ci: Update actions/checkout
reduce power usage for SPI during runtime by 25-50uA
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.
This PR implements the external I2CM interface for ICM-45686. It also fixes a small issue with the common interface write-read implementation.
Currently the mag seems to return a NACK for the writes after the soft reset routine, not sure why, hence draft.