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

Fix PINIO in KAKUTEH7WING #10509

Merged
merged 2 commits into from
Dec 7, 2024
Merged

Fix PINIO in KAKUTEH7WING #10509

merged 2 commits into from
Dec 7, 2024

Conversation

ot0tot
Copy link
Contributor

@ot0tot ot0tot commented Dec 7, 2024

PINIO2 should be inverted so the VTX power is on by default, per the Holybro docs
https://docs.holybro.com/autopilot/kakute-h743-wing/camera-and-on-off-pit-switch

@sensei-hacker
Copy link
Collaborator

Confirmed USER2 is power, user1 is camera switcher.
Also confirmed USER2 needs to be on in order to power the camera, which is backwards from what the manufacturer documentation says.

So this PR should make it match the documentation.

@mmosca mmosca added this to the 8.0 milestone Dec 7, 2024
@mmosca mmosca merged commit a25df70 into iNavFlight:master Dec 7, 2024
20 checks passed
@MrD-RC
Copy link
Collaborator

MrD-RC commented Dec 7, 2024

Personally, I would have left the inversion it was. As now the UI is counter intuitive. At very least this change needs to be mentioned in the release notes, to minimise screwing up pilots who already use this flight controller.

@MrD-RC MrD-RC added the Release Notes Add this when a PR needs to be mentioned in the release notes label Dec 7, 2024
@mmosca
Copy link
Collaborator

mmosca commented Dec 7, 2024

it can be mentioned. This turns the vtx on by default, correct?

@sensei-hacker
Copy link
Collaborator

it can be mentioned. This turns the vtx on by default, correct?

Yep. In my testing with a user.

@ot0tot ot0tot deleted the patch-1 branch December 7, 2024 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release Notes Add this when a PR needs to be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants