Skip to content

Conversation

@haslinghuis
Copy link
Member

@haslinghuis haslinghuis commented Jan 7, 2026

Summary by CodeRabbit

  • New Features
    • Added support for the SIMPLIFLYF405 board (STM32F405): onboard gyro/accelerometer, barometer, OSD, flash logging, beeper, LED strip, camera control, and mapped I/O for motors and receiver.
    • Enabled multiple UART/I2C/SPI ports, ADC-based current/voltage/RSSI sensing with defaults, gyro clock-in support, additional timer mappings, and a user-configurable "9V BEC" auxiliary switch.

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 7, 2026

Caution

Review failed

Failed to post review comments

Walkthrough

Adds a new SIMPLIFLYF405 board configuration header that defines MCU, sensors, peripherals, pin mappings (motors, RX, LEDs, UARTs, I2C/SPI, camera, ADC), TIMER_PIN_MAPPING (including beeper and gyro clock-in), USE_GYRO_CLKIN/GYRO_1_CLKIN_PIN, ADC3/DMA options, and BOX_USER1_NAME.

Changes

Cohort / File(s) Summary
SIMPLIFLYF405 Board Configuration
configs/SIMPLIFLYF405/config.h
New board config header adding MCU/board IDs; many USE_* feature macros (gyro, acc, baro, MAX7456, flash, etc.); full pin mappings (MOTOR1–4, RX_PPM, LEDs, UART1–5 TX/RX, I2C1, SPI1/2/3, MAX7456, FLASH, CAMERA_CONTROL); GYRO_1_* defs and USE_GYRO_CLKIN with GYRO_1_CLKIN_PIN (PA8); TIMER_PIN_MAPPING entries (motors, LED, beeper, gyro clkin); ADC config (ADC3, ADC3_DMA_OPT), default meter sources, PINIO settings, and BOX_USER1_NAME."

Sequence Diagram(s)

(omitted)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

  • Add HGLRCH743 #872 — Adds board config headers defining the same hardware-mapping macros including USE_GYRO_CLKIN, GYRO_1_CLKIN_PIN, TIMER_PIN_MAPPING, SPI/FLASH/MAX7456 pins.
  • Add SKYSTARSF435PRO #750 — New board config header with overlapping macros: sensors (ICM42688P), MAX7456/FLASH, SPI/I2C instances, SYSTEM_HSE_MHZ and TIMER_PIN_MAPPING.
  • Add SIMPLIFLYH7 #927 — Adds a SIMPLIFLY board config with similar macro definitions and the same GYRO_1_CLKIN_PIN (PA8).

Suggested labels

Schematics approved, Uses BF connector standard

Suggested reviewers

  • ot0tot
  • nerdCopter
  • sugaarK
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is entirely missing. The template requires mandatory information including hardware compliance verification, design guideline adherence, and a checklist of critical reviews and testing. Add a complete description addressing all mandatory template sections: schematics review status, hardware testing results, guideline compliance, and all checklist items.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add SIMPLIFLYF405' directly relates to the changeset, which introduces a new flight controller configuration for the SIMPLIFLYF405 board.
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.

@ot0tot
Copy link
Contributor

ot0tot commented Jan 7, 2026

What FC is this?

@haslinghuis
Copy link
Member Author

@ot0tot this board is from Macfos manufacturer

@haslinghuis haslinghuis requested a review from ot0tot January 7, 2026 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants