Skip to content

HGLRC_H743_LITE: Add support for LSM6DSK320X and remove GYRO_CLK feat#1044

Closed
HGLRC-T wants to merge 1 commit intobetaflight:masterfrom
HGLRC-T:HGLRC-H743-LITE-FIX
Closed

HGLRC_H743_LITE: Add support for LSM6DSK320X and remove GYRO_CLK feat#1044
HGLRC-T wants to merge 1 commit intobetaflight:masterfrom
HGLRC-T:HGLRC-H743-LITE-FIX

Conversation

@HGLRC-T
Copy link
Contributor

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

Summary by CodeRabbit

  • New Features

    • Added support for LSM6DSV16X and LSM6DSK320X inertial measurement units on HGLRC_H743_LITE flight controller boards to provide improved sensor compatibility and expanded hardware configuration options.
  • Chores

    • Removed external gyro clock input configuration and support from HGLRC_H743_LITE boards.
    • Updated hardware pin assignments and timer mappings on HGLRC_H743_LITE to accommodate the new sensor module support.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 3, 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 e2f254e and 9f05fd2.

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

Walkthrough

Configuration modifications to the HGLRC_H743_LITE board config to update sensor support by adding accelerometer/gyro sensor options (LSM6DSV16X and LSM6DSK320X) and removing gyro clock-input support.

Changes

Cohort / File(s) Summary
Sensor and Gyro Clock Configuration
configs/HGLRC_H743_LITE/config.h
Added support for two new accelerometer/gyro sensors (USE_ACCGYRO_LSM6DSV16X and USE_ACCGYRO_LSM6DSK320X); removed USE_GYRO_CLKIN and GYRO_1_CLKIN_PIN definitions; updated TIMER_PIN_MAPPING to remove the GYRO_1_CLKIN pin mapping entry.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Suggested labels

Ready to merge

Suggested reviewers

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

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning No pull request description was provided by the author. The template requires mandatory information including hardware compliance details, checklists, and configuration changes documentation. Add a detailed pull request description following the repository template, including hardware compliance information, relevant checklists, and clear explanation of the LSM6DSK320X support addition and GYRO_CLK removal.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the main changes: adding support for LSM6DSK320X and removing GYRO_CLK feature in the HGLRC_H743_LITE 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

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.

@HGLRC-T HGLRC-T closed this Mar 3, 2026
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