Skip to content

IMU Driver#5

Open
mkezzedd wants to merge 3 commits intomainfrom
dev-karim
Open

IMU Driver#5
mkezzedd wants to merge 3 commits intomainfrom
dev-karim

Conversation

@mkezzedd
Copy link

  • IMU driver for the TDK InvenSense ICM-456xx family (supports ICM-45686 and ICM-45605).

  • Implements interrupt-driven FIFO reads over SPI using DMA to minimize ISR load.

  • Provides a mock implementation + corresponding tests

Copy link
Contributor

@TheoHollender TheoHollender left a comment

Choose a reason for hiding this comment

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

Could not find the files mentionnes in InvIMU.hpp, I would like to know where they come from and where I can find them, apart from that it looks good to me for now.


#ifdef USE_ICM45605
#include "icm45605/inv_imu_driver.h"
#include "icm45605/inv_imu_driver_advanced.h"
Copy link
Contributor

Choose a reason for hiding this comment

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

I cannot find these files inside the repo, are they automatically generated by STM32 in some way ?

Copy link
Author

@mkezzedd mkezzedd Feb 6, 2026

Choose a reason for hiding this comment

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

Actually, those aren't generated by STM32, they’re the official TDK vendor drivers. I intentionally excluded them to keep third-party code out of this commit.
You can find them here:
ICM-45605: https://github.com/tdk-invn-oss/motion.mcu.icm45605.driver
ICM-45686: https://github.com/tdk-invn-oss/motion.mcu.icm45686.driver
I can just commit the necessary files if you'd prefer

Copy link
Contributor

Choose a reason for hiding this comment

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

Okay thanks I see, in that case I think it is interesting to have them inside the repo (either as files directly or as a git submodule inside the driver folder), so we can quickly see where to find the relevant code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants