Skip to content

Add HSE to AIRBOTG4AIO#1016

Merged
haslinghuis merged 1 commit intomasterfrom
ot0tot-patch-5
Jan 22, 2026
Merged

Add HSE to AIRBOTG4AIO#1016
haslinghuis merged 1 commit intomasterfrom
ot0tot-patch-5

Conversation

@ot0tot
Copy link
Contributor

@ot0tot ot0tot commented Jan 21, 2026

Add missing HSE define

Summary by CodeRabbit

  • Chores
    • Updated SPI DMA configuration settings for improved hardware compatibility
    • Configured system clock frequency to 8 MHz for hardware timing alignment

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 21, 2026

Walkthrough

Updates the AIRBOTG4AIO board configuration by removing four SPI DMA option macros and adding a SYSTEM_HSE_MHZ macro set to 8. A minor formatting adjustment is made to the MAG_I2C_INSTANCE line.

Changes

Cohort / File(s) Summary
AIRBOTG4AIO board configuration
configs/AIRBOTG4AIO/config.h
Removed SPI1/SPI2 RX/TX DMA option macros; added SYSTEM_HSE_MHZ macro (8 MHz); minor formatting adjustment to MAG_I2C_INSTANCE

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

Bugfix

Suggested reviewers

  • haslinghuis
  • nerdCopter
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is minimal and does not follow the required template structure for hardware/configuration submissions. It lacks mandatory sections like hardware compliance requirements, checklists, and proper context. Provide a comprehensive description following the repository template, including hardware compliance details, relevant checklists, and explanation of why the HSE define was missing and what impact this change has.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add HSE to AIRBOTG4AIO' clearly summarizes the main change: adding the SYSTEM_HSE_MHZ macro to the AIRBOTG4AIO configuration file.
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

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 merged commit 67fcfd4 into master Jan 22, 2026
5 checks passed
@haslinghuis haslinghuis deleted the ot0tot-patch-5 branch January 22, 2026 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants