Skip to content

Conversation

@ot0tot
Copy link
Contributor

@ot0tot ot0tot commented Dec 15, 2025

Need to verify gyro alignment.

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 complete hardware support for the AIKON H743 board, including configuration for the microcontroller target, enabled peripherals (sensors, storage, video output), and pin assignments for motors, LEDs, communication protocols (UART, I2C, SPI), analog input channels, and timing circuits.

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

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 15, 2025

Caution

Review failed

Failed to post review comments

Walkthrough

A new configuration header file is added for the AIKON H743 board, defining the MCU target (STM32H743), peripheral usage flags, comprehensive pin mappings for motors/LEDs/UARTs/I2C/SPI/ADCs/timers, DMA settings, and default behavioral constants.

Changes

Cohort / File(s) Summary
AIKON H743 Board Configuration
configs/AIKONH743/config.h
New board configuration header establishing hardware pinout and peripheral mappings for STM32H743 MCU, including motor/LED/UART/I2C/SPI/ADC/timer pins, DMA options, gyroscope selection, current/voltage meter sources, and branding strings.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

  • Pin mapping verification: Ensure all motor, LED, UART, I2C, SPI, ADC, and timer pin assignments match the AIKON H743 hardware schematic
  • Peripheral flag consistency: Verify that enabled peripherals (accelerometer, gyro, barometer, flash, MAX7456, etc.) align with actual board components
  • Timer and DMA configuration: Confirm TIMER_PIN_MAPPING macro and DMA options are correctly defined for the target MCU

Possibly related PRs

Suggested labels

Schematics approved, Awaiting prototype samples

Suggested reviewers

  • haslinghuis
  • nerdCopter
  • blckmn

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The description is largely incomplete, missing critical sections like hardware compliance details, manufacturer guidelines confirmation, and comprehensive checklist completion status. Add detailed explanation of hardware compliance, confirm adherence to manufacturer guidelines, complete the checklist with all items addressed, and provide context for 'gyro alignment' verification needs.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add AIKONH743' is concise and directly related to the main change, which introduces a new configuration for the AIKONH743 flight controller board.
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.

@haslinghuis
Copy link
Member

What about connector standard ?

@sugaarK sugaarK added the partner program this pr is for a target from the partner program and will automatically get tier 1 status label Dec 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Awaiting prototype samples Awaiting prototype samples partner program this pr is for a target from the partner program and will automatically get tier 1 status Schematics approved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants