Skip to content

Add support for MS5525DSO digital airspeed sensor#11467

Open
nventurino wants to merge 1 commit intoiNavFlight:masterfrom
nventurino:feature/ms5525-pitot
Open

Add support for MS5525DSO digital airspeed sensor#11467
nventurino wants to merge 1 commit intoiNavFlight:masterfrom
nventurino:feature/ms5525-pitot

Conversation

@nventurino
Copy link
Copy Markdown

This commit introduces a hardware driver for the TE Connectivity MS5525DSO digital airspeed sensor over I2C, specifically calibrated for the 1 PSI variant.

Changes included:

  • Registered MS5525 in pitot_hardware for CLI assignment

  • Implemented pitotmeter_ms5525.c/h conforming to the 20-byte scratchpad limit

  • Alternating D1/D2 polling for non-blocking execution length

  • Added standard I2C bus device discovery for 0x76/0x77 addresses

  • Added missing USE_PITOT_MS5525 flag to standard build targets

  • Included I2C transaction success checks to fallback on failure

Tested compiling and executing successfully on KAKUTEH7WING target.

Resolves #4881

This commit introduces a hardware driver for the TE Connectivity MS5525DSO digital airspeed sensor over I2C, specifically calibrated for the 1 PSI variant.

Changes included:

- Registered MS5525 in pitot_hardware for CLI assignment

- Implemented pitotmeter_ms5525.c/h conforming to the 20-byte scratchpad limit

- Alternating D1/D2 polling for non-blocking execution length

- Added standard I2C bus device discovery for 0x76/0x77 addresses

- Added missing USE_PITOT_MS5525 flag to standard build targets

- Included I2C transaction success checks to fallback on failure

Tested compiling and executing successfully on KAKUTEH7WING target.

Resolves iNavFlight#4881
@github-actions
Copy link
Copy Markdown

Branch Targeting Suggestion

You've targeted the master branch with this PR. Please consider if a version branch might be more appropriate:

  • maintenance-9.x - If your change is backward-compatible and won't create compatibility issues between INAV firmware and Configurator 9.x versions. This will allow your PR to be included in the next 9.x release.

  • maintenance-10.x - If your change introduces compatibility requirements between firmware and configurator that would break 9.x compatibility. This is for PRs which will be included in INAV 10.x

If master is the correct target for this change, no action is needed.


This is an automated suggestion to help route contributions to the appropriate branch.

@nventurino
Copy link
Copy Markdown
Author

Follow-up note for ddef163: early review suggests the current MS5525 path is likely too sensitive to post-boot zero-offset changes and thermal/environmental shifts around the sensor or reference side. The implementation currently performs pitot zeroing only at boot, and later signed offset error is converted into positive IAS via fabsf(pressure - pressureZero), so a stale zero can appear as false airspeed on the ground. I think follow-up work should focus on pitot robustness rather than sensor support itself: manual or delayed re-zero after warm-up, possible re-zero while disarmed, exposing raw pitot diagnostics for debugging, and documenting installation/static-side sensitivity. Related INAV history: #5742 (manual airspeed calibration request) and #9216 (pitot calibration robustness work).

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.

Support for MS5525 based Digital Airspeed Sensors

1 participant