-
Notifications
You must be signed in to change notification settings - Fork 16.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add support for checked register in Invensensev2 Driver #14823
add support for checked register in Invensensev2 Driver #14823
Conversation
72aa908
to
7bb4a60
Compare
Here is the Flight log using the firmware built over this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think we need to extend DEVICE_OP with a mavlink2 extension for bank, and extend mavproxy devop, so we can test what happens when we overwrite a single register on an Invensense2
c220502
to
d51bad2
Compare
Hi @tridge I have added the support for devops using banks for SPI sensors only: Also tested by modifying the register using the same, and I get correct behaviour. |
d91e04d
to
4254e1c
Compare
4254e1c
to
b2cd9aa
Compare
pending CI, then merge |
Passed CI; merged. |
This PR adds support for checked register setup for bank based addressing setups, like the one employed in Invensensev2 Driver.
The Logic trace is attached herewith for anyone to look. The contains data on the SPI bus on IMU board in CubeOrange, the sensor comms were disabled by pulling up the CS pin of the Invensensev2 sensor. Will also upload the flight test results as soon as done
Download Logic Data Trace