Skip to content

Conversation

@hydra
Copy link
Contributor

@hydra hydra commented Dec 18, 2025

SPRacingH7EXTREME-SE is just an SPRacingH7EXTREME with a different pin-compatible MCU and a different flash chip.

Hardware already reviewed, it's the same as the SPRacingH7EXTREME.

This target was created in order to develop the MT29F1G01ABAFDWB-IT:F flash chip driver.

Summary by CodeRabbit

  • New Features
    • Added configuration support for SPRACINGH7EXTREME_SE flight controller with integrated hardware support for sensors, flash memory, SD card, and motor control features.

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 18, 2025

Caution

Review failed

Failed to post review comments

Walkthrough

A new Betaflight configuration header for the SPRACINGH7EXTREME_SE board targeting STM32H743 MCU is introduced. The file defines hardware features, peripheral mappings for SPI, I2C, UARTs, ADC, and specialized components including flash, SD card, gyroscope, barometer, and LED support with comprehensive pin assignments.

Changes

Cohort / File(s) Change Summary
New board configuration
configs/SPRACINGH7EXTREME_SE/config.h
Added Betaflight config header defining MCU (STM32H743), flash chip (MT29F1G01ABAFDWB-IT:F), board identifiers, hardware features (Quad-SPI, SDIO, SPI devices, I2C, UART, ADC), peripheral pin mappings, and device-specific settings (gyro, baro, OSD, transponder, LED strip).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

  • Verify all pin assignments align with board schematics and official documentation
  • Check for pin conflicts or duplications across SPI, I2C, UART, and ADC peripherals
  • Validate QuadSPI and SDIO configuration (bank selections, mode flags)
  • Confirm timer pin mapping table completeness and correctness
  • Review DMA configuration for ADC and timer channels
  • Ensure gyroscope/accelerometer and barometer I2C/SPI selections are consistent with hardware

Suggested reviewers

  • haslinghuis
  • ot0tot
  • nerdCopter

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description is largely incomplete and fails to follow the required template structure with mandatory review requirements and comprehensive checklist items. Complete the description using the required template, including all checklist items (schematics review, hardware testing, guideline compliance, connector standards, flight testing, and issue resolution confirmation).
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the specific flight controller target being added (SPRacingH7EXTREME_SE) and describes the main action (initial commit), directly matching the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 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.

@hydra
Copy link
Contributor Author

hydra commented Dec 18, 2025

Note for reviewers:

  • the SPRacingH7EXTREME was one of the FC's that helped set many of the Betaflight hardware design guidelines, schematics are available on the SPRacing github repo (Diptrace required).
  • as noted, this isn't a new hardware platform, it's a pre-existing platform with just 2 components swapped.

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