Skip to content

Add AXISFLYINGH743MINI#1088

Open
osirisinferi wants to merge 4 commits into
betaflight:masterfrom
osirisinferi:add-axfl-h743-mini
Open

Add AXISFLYINGH743MINI#1088
osirisinferi wants to merge 4 commits into
betaflight:masterfrom
osirisinferi:add-axfl-h743-mini

Conversation

@osirisinferi
Copy link
Copy Markdown
Contributor

@osirisinferi osirisinferi commented Apr 28, 2026

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 AXISFLYINGH743MINI flight controller with onboard accelerometer (ICM42688P), SPI gyro, DPS310 barometer, OSD (MAX7456), and onboard flash.
    • Provides hardware defaults for motors, LED strip, inverted beeper, UART/I2C/SPI routing, power monitoring, and a labeled "VTX PWR" pin.
    • Sets sensible defaults for telemetry, blackbox source, and DSHOT bit‑banging mode.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 28, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 79cfe1a8-6618-4dfb-ae1d-fee0dd28fa4b

📥 Commits

Reviewing files that changed from the base of the PR and between edb608e and 2b1c9a9.

📒 Files selected for processing (1)
  • configs/AXISFLYINGH743MINI/config.h
🚧 Files skipped from review as they are similar to previous changes (1)
  • configs/AXISFLYINGH743MINI/config.h

Walkthrough

Adds a new Betaflight board configuration header for AXISFLYINGH743MINI (STM32H743) defining MCU/board identity, enabled peripherals, full GPIO/timer/DMA mappings, serial/SPI/I2C/ADC routing, default runtime options, and a PINIO1 user label. (50 words)

Changes

Cohort / File(s) Summary
New Board Configuration
configs/AXISFLYINGH743MINI/config.h
Added complete board config for AXISFLYINGH743MINI: MCU/board macros, enabled peripherals (ICM42688P accel/gyro via SPI, DPS310 baro, Py25Q128HA flash, MAX7456 OSD), GPIO mappings for motors/LED/beeper/UART/I2C/SPI/ADC, gyro CS/EXTI/CLKIN pins, TIMER_PIN_MAPPING, DMA option constants, conditional UART routing (GPS/MSP/OSD), serial assignments (RX/ESC telemetry), default DSHOT bitbanging and meter/blackbox sources, gyro alignment, SPI instance selection, and PINIO1 label "VTX PWR".

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • Add HGLRC_F722_S board #995 — Adds a per-board config.h with similar MCU, peripheral enables, pin mappings, timer/DMA settings, and default runtime options.

Suggested labels

Schematics Approved, Uses Betaflight Connector Standard

Suggested reviewers

  • haslinghuis
  • sugaarK
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description includes only a partial checklist without explaining what the PR does, missing mandatory sections like PR summary, hardware compliance details, and design documentation references. Complete the description with a brief PR summary explaining what this board is, confirm all mandatory review requirements are met, and reference any design documentation or schematics reviewed.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically identifies the board being added (AXISFLYINGH743MINI), directly matching the main content of the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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
Review rate limit: 0/1 reviews remaining, refill in 60 minutes.

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Warning

CodeRabbit couldn't request changes on this pull request because it doesn't have sufficient GitHub permissions.

Please grant CodeRabbit Pull requests: Read and write permission and re-run the review.

👉 Steps to fix this

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@configs/AXISFLYINGH743MINI/config.h`:
- Around line 129-130: Confirm the physical orientation of the gyro for this
board and update GYRO_1_ALIGN to the correct enum (e.g., CW0_DEG, CW90_DEG,
etc.) in the config, replacing the placeholder/comment; only remove the "NEEDS
CONFIRMATION" remark after verifying axis mapping with the board schematic or
bench test (use the sensor output/attitude checks) and then commit the corrected
GYRO_1_ALIGN value so the craft's attitude frame is not inverted.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 39f6ee06-cc55-4ac7-88e5-5dcee609e6f0

📥 Commits

Reviewing files that changed from the base of the PR and between 8e39400 and a7a40c1.

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

Comment thread configs/AXISFLYINGH743MINI/config.h Outdated
Confirmed by AxisFlying on Discord that the gyro has an orientation of 0°
Comment thread configs/AXISFLYINGH743MINI/config.h Outdated
@haslinghuis
Copy link
Copy Markdown
Member

Does this replace #822 ?

@osirisinferi
Copy link
Copy Markdown
Contributor Author

Does this replace #822 ?

No, that one is supposed to be the AIO I was told earlier. See this Discord post at 2026-03-18. This PR is for a different board, #822 should be renamed to something else.

osirisinferi and others added 2 commits April 28, 2026 15:59
Forgot to remove it when I borrowed the `config.h` from the pro board.

Co-authored-by: Mark Haslinghuis <mark@numloq.nl>
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.

2 participants