Skip to content

HGLRC_F405_S: Add support for LSM6DSV16X/K320X gyro and LPS22DF baro#1048

Open
HGLRC-T wants to merge 1 commit intobetaflight:masterfrom
HGLRC-T:HGLRC_F405_S-FIX
Open

HGLRC_F405_S: Add support for LSM6DSV16X/K320X gyro and LPS22DF baro#1048
HGLRC-T wants to merge 1 commit intobetaflight:masterfrom
HGLRC-T:HGLRC_F405_S-FIX

Conversation

@HGLRC-T
Copy link
Contributor

@HGLRC-T HGLRC-T commented Mar 6, 2026

  • Enable LSM6DSV16X and LSM6DSK320X gyro/accel drivers
  • Enable LPS22DF baro driver alongside existing DPS310

Summary by CodeRabbit

  • New Features
    • Added support for additional accelerometer and gyroscope sensor variants, improving motion detection accuracy and responsiveness
    • Expanded barometer sensor compatibility for enhanced environmental and altitude monitoring with more flexible hardware configuration options

- Enable LSM6DSV16X and LSM6DSK320X gyro/accel drivers
- Enable LPS22DF baro driver alongside existing DPS310
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 6, 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: ad07cf83-3a28-4719-83e5-053517cddadc

📥 Commits

Reviewing files that changed from the base of the PR and between e2f254e and 66e777e.

📒 Files selected for processing (1)
  • configs/HGLRC_F405_S/config.h

Walkthrough

Added four new sensor support macros to the HGLRC_F405_S configuration file: two accelerometer/gyroscope variants (LSM6DSV16X, LSM6DSK320X) and two barometer variants (LPS, LPS22DF). No existing definitions were removed or modified.

Changes

Cohort / File(s) Summary
Sensor Configuration Macros
configs/HGLRC_F405_S/config.h
Added four preprocessor macro definitions to enable additional sensor support variants: accelerometer/gyroscope options (USE_ACCGYRO_LSM6DSV16X, USE_ACCGYRO_LSM6DSK320X) and barometer options (USE_BARO_LPS, USE_BARO_LPS22DF).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

Ready to merge

Suggested reviewers

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

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is incomplete; it lacks the required checklist and mandatory review information specified in the repository template for new flight controllers. Add the complete checklist items (schematics review, hardware testing, guidelines compliance, connector standards, flight testing, and issue resolution) and confirm whether this is a new hardware release requiring Betaflight team review.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: adding support for specific gyro and barometer sensors to the HGLRC_F405_S configuration.
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

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.

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.

1 participant