Skip to content

Add DPS310 define per manufacturer request#1053

Merged
sugaarK merged 1 commit intobetaflight:masterfrom
ot0tot:Update-Flycolor-baro
Mar 14, 2026
Merged

Add DPS310 define per manufacturer request#1053
sugaarK merged 1 commit intobetaflight:masterfrom
ot0tot:Update-Flycolor-baro

Conversation

@ot0tot
Copy link
Contributor

@ot0tot ot0tot commented Mar 12, 2026

Summary by CodeRabbit

Release Notes

  • New Features
    • Added DPS310 barometer sensor support for multiple flight controller variants, enabling enhanced altitude and pressure sensing capabilities across different hardware configurations.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 12, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 3377c585-77cc-4122-8118-882dbcac15d8

📥 Commits

Reviewing files that changed from the base of the PR and between 82d3447 and db87f0a.

📒 Files selected for processing (5)
  • configs/FLYCOLORF7/config.h
  • configs/FLYCOLORF7MINI/config.h
  • configs/FLYCOLORF7V2/config.h
  • configs/FLYCOLORF7_AIO_HD/config.h
  • configs/TMOTORF7_AIO/config.h

Walkthrough

Enables DPS310 barometer support across five board configuration files by adding the USE_BARO_DPS310 or USE_BARO_SPI_DPS310 preprocessor macro to their respective config headers, allowing conditional compilation of barometer driver code.

Changes

Cohort / File(s) Summary
Flycolor F7 Variants (Standard I2C)
configs/FLYCOLORF7/config.h, configs/FLYCOLORF7MINI/config.h, configs/FLYCOLORF7V2/config.h
Added USE_BARO_DPS310 macro to enable I2C-based DPS310 barometer support.
Flycolor F7 AIO Variants (SPI)
configs/FLYCOLORF7_AIO_HD/config.h, configs/TMOTORF7_AIO/config.h
Added USE_BARO_SPI_DPS310 macro to enable SPI-based DPS310 barometer support.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • sugaarK
  • nerdCopter
  • haslinghuis
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is empty, but the repository template requires mandatory information including hardware compliance details, checklist items, and housekeeping notes. Provide a complete pull request description following the template, including hardware compliance confirmation, relevant checklist items, and details about the DPS310 barometer changes.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: adding DPS310 barometer defines to multiple flight controller configs per manufacturer request.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

CodeRabbit can use Clang for C/C++ static analysis and code quality checks.

Clang provides comprehensive static analysis for C and C++ code, including syntax checking, type checking, and various warning diagnostics.

@sugaarK sugaarK merged commit edb8a26 into betaflight:master Mar 14, 2026
9 checks passed
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.

3 participants