Skip to content

HOBBYWING_XROTORF7CONV - USE_ACCGYRO_LSM6DSK320X#1019

Merged
blckmn merged 1 commit intomasterfrom
haslinghuis-patch-3
Jan 29, 2026
Merged

HOBBYWING_XROTORF7CONV - USE_ACCGYRO_LSM6DSK320X#1019
blckmn merged 1 commit intomasterfrom
haslinghuis-patch-3

Conversation

@haslinghuis
Copy link
Member

@haslinghuis haslinghuis commented Jan 26, 2026

Requires

Summary by CodeRabbit

  • Bug Fixes
    • Updated inertial measurement unit configuration for the HOBBYWING_XROTORF7CONV device to improve compatibility and performance.

✏️ Tip: You can customize this high-level summary in your review settings.

@haslinghuis haslinghuis self-assigned this Jan 26, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 26, 2026

Walkthrough

A single configuration macro is replaced in the HOBBYWING_XROTORF7CONV flight controller config: the accelerometer/gyroscope sensor enablement is switched from LSM6DSV16X to LSM6DSK320X. No functional logic changes.

Changes

Cohort / File(s) Summary
Sensor Macro Update
configs/HOBBYWING_XROTORF7CONV/config.h
Replaced USE_ACCGYRO_LSM6DSV16X with USE_ACCGYRO_LSM6DSK320X to update the accelerometer/gyroscope sensor configuration for this flight controller variant.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

Bugfix

Suggested reviewers

  • blckmn
  • KarateBrot
  • nerdCopter
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is largely incomplete. It only lists required dependent PRs but lacks mandatory sections like hardware compliance details, checklist completion, and verification status as specified in the template. Complete the description by filling in the mandatory sections: hardware compliance requirements, checklist items (schematics review, hardware testing, guideline adherence, etc.), and any relevant implementation details for the sensor change.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title specifically identifies the hardware config change and the new sensor macro being applied, directly matching the changeset's main modification.

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

✨ Finishing touches
  • 📝 Generate docstrings

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.

@haslinghuis haslinghuis changed the title Change accelerometer/gyroscope configuration HOBBYWING_XROTORF7CONV - USE_ACCGYRO_LSM6DSK320X Jan 26, 2026
@haslinghuis haslinghuis requested review from blckmn and ot0tot January 28, 2026 16: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