fix(pitotmeter): guard MS5525 i2c address writes with USE_I2C for SITL builds

[sensei-hacker]

Summary

The MS5525 pitotmeter driver failed to compile for SITL targets because it directly accessed pitot->busDev->busdev.i2c.address, but the i2c union member in busDevice_t.busdev is conditionally compiled under #ifdef USE_I2C. The SITL target explicitly #undef USE_I2C (target.h), so the member does not exist in that build.

Changes

• Wrapped the two busdev.i2c.address assignment lines in pitotmeter_ms5525.c with #ifdef USE_I2C / #endif, matching the pattern already used throughout bus.h and bus.c

Testing

• SITL target compiled successfully with the fix applied (ninja -j4 in build_sitl/)
• No behavioral change on hardware targets where USE_I2C is defined — the guards are compile-time only
• Note: full hardware I2C testing of the MS5525 sensor was not performed (no hardware available); the fix is a compile guard only and does not alter runtime logic

Code Review

Reviewed for correctness and INAV coding standards — no issues found. The fix mirrors the existing #ifdef USE_I2C pattern used in bus.c and bus.h for all direct busdev union member accesses.

[qodo-code-review]

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

[github-actions]

Test firmware build ready — commit 73c59e8

Download firmware for PR #11570

234 targets built. Find your board's .hex file by name on that page (e.g. MATEKF405SE.hex). Files are individually downloadable — no GitHub login required.

Development build for testing only. Use Full Chip Erase when flashing.