Skip to content

Conversation

@Stefanbrnnr
Copy link

@Stefanbrnnr Stefanbrnnr commented Nov 13, 2025

Pull-Request requirements

Mandatory Review for All New Flight Controllers

  • All new flight controllers must undergo the Betaflight review process, regardless of whether they use an existing target.
  • Manufacturers may reuse the same target for multiple designs, but each new hardware release must be reviewed before approval.

Hardware Compliance Requirements

These measures help maintain high standards and ensure compatibility within the Betaflight ecosystem.

If you have any questions or need guidance, feel free to reach out to the Betaflight development team.

Housekeeping

  • Pull-Request only from a custom branch, not master.
  • Replace this text with details of your own.

Checklist (✓/✕, or y/n)

  • passed Betaflight team's schematics review
  • passed hardware samples testing
  • follows guidelines
  • follows connector standards
  • flight tested
  • comments/issues resolved

Summary by CodeRabbit

  • New Features
    • Added support for the BRENNER_H7 flight controller board with STM32H743 MCU
    • Integrated accelerometer, gyroscope, and barometer sensors
    • Configured motor, servo, communication, and power management peripherals with complete pin and timer mappings

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 13, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

A new configuration header file for the BRENNER_H7 board is introduced, defining hardware abstraction macros for the STM32H743 MCU. The file specifies sensor configurations (accelerometer, gyroscope, barometer), pin mappings for motors, servos, UARTs, I2C, SPI, ADC, and other peripherals, along with timer associations and DMA/flash/meter settings.

Changes

Cohort / File(s) Summary
BRENNER_H7 Board Configuration
configs/BRENNER_H7/config.h
New board configuration header introducing MCU target, board/manufacturer identification, sensor feature enablement, comprehensive peripheral pin mappings (motors, servos, UARTs, I2C, SPI, ADC, LEDs), timer-to-pin channel assignments with polarity, DMA options for ADC, SPI instance definitions, flash/blackbox settings, I2C instance mappings, default meter sources/scales, and PINIO/beeper configuration macros.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Pin mappings and macro values should be cross-referenced with hardware schematics and datasheets
  • Verify timer channel and polarity settings match the intended PWM/peripheral configurations
  • Confirm DMA options and SPI instance assignments are compatible with on-board peripherals
  • Check consistency with other similar STM32H7 board configurations to identify any deviations or errors

Possibly related PRs

  • Add SIMPLIFLYH7 #927: Adds an STM32H743 board config header with identical macro patterns for board identification, sensor features, peripheral pin mappings, and timer/DMA/SPI settings
  • Add SPEDIXH743 #791: Introduces STM32H743 board configuration with overlapping macro definitions including FC_TARGET_MCU, TIMER_PIN_MAPPING, DMA/SPI instances, and meter defaults
  • add AEROH7DIGITAL #900: Adds STM32H743 board config with highly similar macro structure covering sensor enablement, pin mappings, and peripheral instance configurations

Suggested labels

Awaiting prototype samples

Suggested reviewers

  • ot0tot
  • haslinghuis
  • nerdCopter
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8a7e533 and ea94cbe.

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

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.

@github-actions
Copy link

Auto-closing pull request because the source branch is named 'master'. Please create a feature branch instead. https://betaflight.com/docs/development#using-git-and-github

@github-actions github-actions bot closed this Nov 13, 2025
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