Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add the SULILGH7 flight controller board #29209

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Add the SULILGH7 flight controller board #29209

wants to merge 10 commits into from
+536 −0

Conversation

SULILG
Copy link

@SULILG SULILG commented Feb 1, 2025

No description provided.

@Hwurzburg
Copy link
Collaborator

needs readme and images/pinout....ala https://github.com/ArduPilot/ardupilot/tree/master/libraries/AP_HAL_ChibiOS/hwdef/AET-H743-Basic
I suggest a separate PR to reserve the bd id, instead on including it here...those get merged fast and holds the id until this gets finished and reviewed

@SULILG
Copy link
Author

SULILG commented Feb 2, 2025

I will add some images and descriptions.

Hwurzburg
Hwurzburg requested changes Feb 2, 2025
View reviewed changes
Copy link
Collaborator

@Hwurzburg Hwurzburg left a comment

Choose a reason for hiding this comment

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

still suggest you reserve the board id by splitting out the board_types.txt into a new PR by itself

libraries/AP_HAL_ChibiOS/hwdef/SULILGH7-P1-P2/hwdef.dat Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/SULILGH7-P1-P2/README.md Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/SULILGH7-P1-P2/README.md Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/SULILGH7-P1-P2/hwdef-bl.dat Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/SULILGH7-P1-P2/hwdef.dat Outdated Show resolved Hide resolved
@SULILG SULILG requested a review from Hwurzburg February 2, 2025 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Hwurzburg Hwurzburg Awaiting requested review from Hwurzburg

Assignees
No one assigned
Labels
NewBoardSupport WikiNeeded needs wiki update
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@SULILG @Hwurzburg