Add LIS2MDL driver & implement in Imu component #16
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
hrfarmer
added a commit
that referenced
this pull request
Oct 1, 2025
* Use custom board definition * Initial IMU component generation lastFile:FprimeZephyrReference/Components/Imu/Imu.fpp * Initial imu data pull * Reading magnetic field data * Add new telemetry for accel and angvel * Acceleration and AngularVelocity telemetry working * Add temp * Improve documentation * Update FprimeZephyrReference/Components/Imu/docs/sdd.md Co-authored-by: Copilot <[email protected]> * Add LIS2MDL driver & implement in Imu component (#16) * add lis2mdl driver and implement in imu component * fix formatting * Create lsm6dso driver component & implementation (#17) * Update README.md * Create lmsd6dsoDriver component * create ports for sensor reading output * fix port naming issue * Use custom board definition * Remove commented config * Moving prj.conf sensors to board definition * Add d board * Try referencing c definitions in d board * Making a common v5 board definition * Fix readme * Minor Makefile, Readme and cmake presets fixes * change lsm6dso internal structs to carry F64 and added output ports to imu * removed lmsdso init from imu constructor * remove old sensor reading from imu * add lms6dso driver to topology * change base id to allow for lis2mdl driver * remove line endings * add channel retrieval during sensor data * Ensure submodules are downloaded before venv is created * Fix infinit submodule make target recursion... --------- Co-authored-by: ineskhou <[email protected]> Co-authored-by: Nate Gay <[email protected]> Co-authored-by: Nate Gay <[email protected]> Co-authored-by: Saidi Adams <[email protected]> Co-authored-by: Michael Pham <[email protected]> * Spit out common helpers and types from drivers, standardize port name schema, update SDDs * Remove unnecessary board files * remove fpp helper file * finished renaming to build * add parentheses for clearer intentions * remove FW_ASSERTs * clear throttle after successful device ready check * adding telemetry to lis2mdlManager * fix fprime files to match new names & telemetry structure * edit cpp implementation files to output telemetry * formatting * clear throttle, configure lsm6 sensor, read->get * Add IMU tests * Add type hinting * Comment fix --------- Co-authored-by: aychar <[email protected]> Co-authored-by: Copilot <[email protected]> Co-authored-by: Aaron Siemsen <[email protected]> Co-authored-by: ineskhou <[email protected]> Co-authored-by: Saidi Adams <[email protected]> Co-authored-by: Michael Pham <[email protected]> Co-authored-by: asiemsen <[email protected]>
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 moves the sensor functionality from lis2mdl to a new driver, and outputs the sensor data to the Imu component through a port.
It working in the GDS:
