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 adds support for the Microstrain CV7-AHRS IMU, as the communication protocol is slightly different. The change is selectable by running
make menuconfig
(a new entry underfirmware/README.md
has been added).For the CV7:
Message Format
command (0x0C, 0x0F) is different.I've decided to keep the original CX5 code unchanged, as I can't test with that IMU. It might be worth testing with it in any case, (and potentially refactoring the code). During startup the ESP32 should print the following, and the sensor readings should be reported correctly:
"Configuring for CX5-AHRS IMU..."
How I Tested
I fulfilled the following requirements
All new code is formatted according to our style guide (for C++ run clang-format, for Python, run flake8 and fix all warnings).
This has been skipped as it looks like it hasn't been ran before -
clang-format -i uart_imu.c
makes too many changes and would pollute the PR. Let me know if it would be better to add regardless!All new functions/classes are documented and existing documentation is updated according to changes.
No commented code from testing/debugging is kept (unless there is a good reason to keep it).