Skip to content

DAKEFPVF405 - Update sensor configurations#1026

Merged
blckmn merged 1 commit intomasterfrom
haslinghuis-patch-3
Feb 14, 2026
Merged

DAKEFPVF405 - Update sensor configurations#1026
blckmn merged 1 commit intomasterfrom
haslinghuis-patch-3

Conversation

@haslinghuis
Copy link
Member

@haslinghuis haslinghuis commented Feb 6, 2026

Removed MPU6500 definitions and added LSM6DSK320X.

Summary by CodeRabbit

  • Chores
    • Updated accelerometer and gyroscope sensor configuration.

Removed MPU6500 definitions and added LSM6DSK320X.
@haslinghuis haslinghuis self-assigned this Feb 6, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 6, 2026

Walkthrough

This pull request modifies the sensor configuration for the DAKEFPVF405 board by replacing separate MPU6500 accelerometer and gyro sensor macros with a single integrated LSM6DSK320X sensor macro in the config.h file.

Changes

Cohort / File(s) Summary
Sensor Configuration
configs/DAKEFPVF405/config.h
Replaced USE_ACC_SPI_MPU6500 and USE_GYRO_SPI_MPU6500 macros with USE_ACCGYRO_LSM6DSK320X to switch from separate sensor drivers to an integrated accelerometer/gyro sensor.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • betaflight/config#1019: Similar sensor configuration update switching to USE_ACCGYRO_LSM6DSK320X in a different board config file, replacing prior accelerometer/gyro macros.

Suggested reviewers

  • ot0tot
  • nerdCopter
  • blckmn
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description is incomplete. It only provides a brief summary of changes but does not follow the required template structure with mandatory sections like hardware compliance, checklist items, and design review confirmations. Expand the description to include the required template sections: hardware compliance details, manufacturer design guidelines adherence, schematics review confirmation, hardware testing status, and completion of the checklist items.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: updating sensor configurations for DAKEFPVF405, specifically switching from MPU6500 to LSM6DSK320X sensors.
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 docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch haslinghuis-patch-3

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.

@blckmn blckmn merged commit b0bb6da into master Feb 14, 2026
5 checks passed
@haslinghuis haslinghuis deleted the haslinghuis-patch-3 branch February 14, 2026 08:28
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