KAKUTEH7: add LSM6DSK320X support#1122
Conversation
WalkthroughAdded a single ChangesSensor driver enablement
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested labels
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
we won't approve adding the LSM6DSV16X its not flyable as far as gyro platforms go. we have spent over a year working with and in parallel with sam on making this work and the consensus from our team and the STM mems team is its not suitable for use on flight controllers. |
sugaarK
left a comment
There was a problem hiding this comment.
remove the DSV16 define... its not suitable for use on flight controllers
|
Updated.
|
|
@sugaarK Updated as requested.
Could you please take another look when you have time? |
Summary
This PR updates the existing
KAKUTEH7target config to enable support forLSM6DSV16XandLSM6DSK320X.This is a config-only change for an existing target. No new target is introduced, and no driver implementation changes are included.
Hardware Context
The existing
KAKUTEH7target supports theICM42688P. This PR addsLSM6DSV16XandLSM6DSK320Xsupport using the existingSPI4, chip select,INT1data-ready interrupt, and sensor alignment configuration.The schematic and PCB review confirms that the existing IMU interface and target pin configuration remain unchanged.
Validation
KAKUTEH7firmware successfully.HBRO/KAKUTEH7.LSM6DSV16X:GYRO: (1) LSM6DSV16X enabled locked dmaACC: LSM6DSV16XLSM6DSK320X:GYRO: (1) LSM6DSK320X enabled locked dmaACC: LSM6DSK320XAttached Configurator and CLI
statusscreenshots:HBRO/KAKUTEH7target identificationLSM6DSV16XCLIstatusLSM6DSK320XCLIstatusChecklist
LSM6DSK320X)Summary by CodeRabbit