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.

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

iFlight BLITZ H743 Pro #26763

Merged
merged 2 commits into from
May 1, 2024
Merged

iFlight BLITZ H743 Pro #26763

merged 2 commits into from
May 1, 2024

Conversation

andyp1per
Copy link
Collaborator

@andyp1per andyp1per commented Apr 11, 2024

Not currently shipping. They would like to get the hwdef in without the associated wiring pictures.

@andyp1per andyp1per added the WIP label Apr 11, 2024
@andyp1per andyp1per changed the title Pr blitzh7pro iFlight Blitz H743 Pro Apr 12, 2024
@andyp1per andyp1per added Driver ChibiOS WikiNeeded needs wiki update and removed WIP labels Apr 12, 2024
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.

once I know if this is the WING (and should be titled as such ie BLITZWingH743), will review further...but just off the top of my head the hwdef is missing the camera switch def if it is the Wing

libraries/AP_HAL_ChibiOS/hwdef/BlitzH743Pro/README.md Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/BlitzH743Pro/README.md Outdated Show resolved Hide resolved
@andyp1per
Copy link
Collaborator Author

once I know if this is the WING (and should be titled as such ie BLITZWingH743), will review further...but just off the top of my head the hwdef is missing the camera switch def if it is the Wing

This is not the wing, this is a new copter flight controller that is not on their website (hence the lack of wiring diagram)

@Hwurzburg
Copy link
Collaborator

okay will wait to review until complete info is available...ping me

@andyp1per andyp1per changed the title iFlight Blitz H743 Pro iFlight BLITZ H743 Pro Apr 16, 2024
@andyp1per andyp1per requested a review from Hwurzburg April 27, 2024 13:13
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.

we ought to add that backside photo you took with expansion board...when the wiki page is done we can update from their website photos...

@andyp1per
Copy link
Collaborator Author

we ought to add that backside photo you took with expansion board...when the wiki page is done we can update from their website photos...

I think we should leave it out for now. They are quite protective of their brand and won't want a dodgy photo that I took.

@tridge
Copy link
Contributor

tridge commented Apr 29, 2024

@Hwurzburg I think @andyp1per has addressed your issues except the back-side photo. I think we could merge? can you check?

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.

I still cant find the analog airspeed sensor pin which is defined on any photo you have sent

@andyp1per
Copy link
Collaborator Author

I still cant find the analog airspeed sensor pin which is defined on any photo you have sent

I can't find one either, I think its only pinned out on the wing

@tridge tridge merged commit f38faaa into ArduPilot:master May 1, 2024
44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 4.5.3-beta1
Development

Successfully merging this pull request may close these issues.

6 participants