PX4-Autopilot
PX4-Autopilot copied to clipboard
PX4
CUAV V6X board support
Continuing https://github.com/PX4/PX4-Autopilot/pull/19750 here.
Rebased on current master.
@davids5 I believe your proposed change for the RM3100 self test bug does not address the fundamental problem with the current self test implementation. The software timeout needs to be greater than what is programmed into the RM3100 BIST register (0x33). The current implementation sets the BW and BP bits to all 1's (the maximum value). This equates to a timeout value of 480us (4 sleep oscillation cycles per LR timeout period and 4 LR periods for BIST) according to the latest published datasheet. Currently, the driver is waiting 26ms (2*RM3100_INTERVAL) which is significantly longer than required. See pull request #19583 for a proposed fix to address the RM3100 self test failures. I’d be interested to know if my proposed changes address the RM3100 self test failures you are experiencing on your flight controller.
@bmeagher - Thank you! This was not my work, just a continuation of a PR I could not write to.
@mxiaogit Let's drop 8ea548d and when https://github.com/PX4/PX4-Autopilot/pull/19583 comes it will be fixed
@davids5 I believe your proposed change for the RM3100 self test bug does not address the fundamental problem with the current self test implementation. The software timeout needs to be greater than what is programmed into the RM3100 BIST register (0x33). The current implementation sets the BW and BP bits to all 1's (the maximum value). This equates to a timeout value of 480us (4 sleep oscillation cycles per LR timeout period and 4 LR periods for BIST) according to the latest published datasheet. Currently, the driver is waiting 26ms (2*RM3100_INTERVAL) which is significantly longer than required. See pull request #19583 for a proposed fix to address the RM3100 self test failures. I’d be interested to know if my proposed changes address the RM3100 self test failures you are experiencing on your flight controller.
@bmeagher - Thank you! This was not my work, just a continuation of a PR I could not write to.
@mxiaogit Let's drop 8ea548d and when #19583 comes it will be fixed
@davids5 Yes, we can delete it.
@davids5 Pixhawk V6X kit will include CAN PMU Lite, can you enable Dronecan battery monitor by default?
@davids5 Pixhawk V6X kit will include CAN PMU Lite, can you enable Dronecan battery monitor by default?
@cuhome Please do a PR against this PR and test it.
@dagar - was in the middle of testing and had 2 meetings. I Give me a bit....
@dagar @davids5 This pr was not merged into the newly released 1.13.1stable, but was merged in 1.13.0master, can you please help me?
The v1.13.2 has been released but I have no objections to a v1.13.3 once more backports like this one are in.
@julianoes we need to make sure things like this don't slip from releases, there doesn't seem to be any process for that currently
I don't have the overview of all these things and what needs to be backported, sorry.
@dagar Thanks, when there is no gps positioning, the wrong altitude (>1000) (local_position_ned) appears.
