Skip to content

DAKEFPV - Update sensor configurations#1042

Merged
haslinghuis merged 1 commit intobetaflight:masterfrom
engineer-dakefpv:DAKEFPVFUpdateSensorConfigurations
Feb 28, 2026
Merged

DAKEFPV - Update sensor configurations#1042
haslinghuis merged 1 commit intobetaflight:masterfrom
engineer-dakefpv:DAKEFPVFUpdateSensorConfigurations

Conversation

@engineer-dakefpv
Copy link
Contributor

@engineer-dakefpv engineer-dakefpv commented Feb 26, 2026

1、Removed BMI270 definitions and added LSM6DSV16X.
2、All flight controllers uniformly support the following four gyroscopes:
MPU6000
ICM42688P
LSM6DSV16X
LSM6DSK320X

Summary by CodeRabbit

  • Chores
    • Updated hardware sensor configurations across multiple device models.
    • Added support for new accelerometer and gyroscope sensor variants.
    • Removed support for legacy sensor models.
    • Re-enabled barometer and flash memory support on select devices.
    • Reorganized configuration settings to reduce redundancy.

1、Removed BMI270 definitions and added LSM6DSV16X.
2、All flight controllers uniformly support the following four gyroscopes:
MPU6000
ICM42688P
LSM6DSV16X
LSM6DSK320X
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2801d8e and 87e3b87.

📒 Files selected for processing (9)
  • configs/DAKEFPVF405/config.h
  • configs/DAKEFPVF411/config.h
  • configs/DAKEFPVF435/config.h
  • configs/DAKEFPVF435HD/config.h
  • configs/DAKEFPVF722/config.h
  • configs/DAKEFPVF722X8/config.h
  • configs/DAKEFPVH743/config.h
  • configs/DAKEFPVH743PRO/config.h
  • configs/DAKEFPVH743_SLIM/config.h

Walkthrough

Updates to nine board configuration files standardize gyro support enablement and add two new accelerometer/gyroscope sensor variants (LSM6DSV16X, LSM6DSK320X) across DAKE flight controller boards. Additionally, legacy sensor support macros are removed and sensor configurations are reorganized in some boards.

Changes

Cohort / File(s) Summary
Major Sensor Configuration Updates
configs/DAKEFPVF405/config.h
Re-enables barometer and flash support; replaces BMI270 accelerometer/gyro support with LSM6DSV16X.
Multi-Option Gyro/Accelerometer Expansion
configs/DAKEFPVF411/config.h
Removes legacy USE_GYRO and ICM42688P pairing; adds multiple independent gyro options (MPU6000, ICM42688P) and new ACCGYRO combinations (LSM6DSV16X, LSM6DSK320X); removes BMI270 and BMP280 barometer support.
Gyro and ACCGYRO Standardization
configs/DAKEFPVF435/config.h, configs/DAKEFPVF435HD/config.h, configs/DAKEFPVF722/config.h, configs/DAKEFPVF722X8/config.h, configs/DAKEFPVH743/config.h, configs/DAKEFPVH743PRO/config.h, configs/DAKEFPVH743_SLIM/config.h
Consolidates USE_GYRO macro to top-level section, removes duplicate definitions, and adds two new ACCGYRO variants (LSM6DSV16X, LSM6DSK320X) across all boards.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

Possibly related PRs

Suggested labels

Ready to merge

Suggested reviewers

  • ot0tot
  • haslinghuis
  • blckmn
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is largely incomplete and fails to follow the mandatory template requirements for flight controller submissions. Provide schematics review status, hardware testing results, and complete all checklist items as required by the submission template for flight controller changes.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change—updating sensor configurations across DAKEFPV flight controller configs.
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

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

@engineer-dakefpv
Copy link
Contributor Author

@ot0tot

@ot0tot
Copy link
Contributor

ot0tot commented Feb 27, 2026

Are you using the same footprint? The MPU6000 and ICM gyros have different orientations, so they will require separate targets.

@engineer-dakefpv
Copy link
Contributor Author

This time my modification did not change the MPU6000; it originally existed, and our flight controller has reserved pads for it. We will choose to install one of the four gyroscopes. We now need the flight controller to support the LSM6DSV16X and LSM6DSK320X, which are Pin-to-Pin compatible with the ICM42688P.

@haslinghuis haslinghuis merged commit 2f5574f into betaflight:master Feb 28, 2026
13 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.

4 participants