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

BETAFPV F405 #24625

Merged
merged 2 commits into from
Aug 16, 2023
Merged

BETAFPV F405 #24625

merged 2 commits into from
Aug 16, 2023

Conversation

andyp1per
Copy link
Collaborator

@andyp1per andyp1per commented Aug 12, 2023

@andyp1per
Copy link
Collaborator Author

andyp1per commented Aug 12, 2023

For anyone who wants to try:

arducopter.zip

@andyp1per andyp1per force-pushed the pr-betafpv-f405-hwdef branch 3 times, most recently from eaaee2d to 8d407b8 Compare August 13, 2023 21:15
@andyp1per andyp1per changed the title betafpv f405 BETAFPV F405 Aug 15, 2023
@tridge tridge merged commit 7896b04 into ArduPilot:master Aug 16, 2023
39 checks passed
@Hwurzburg Hwurzburg added the WikiNeeded needs wiki update label Aug 16, 2023
@Hwurzburg
Copy link
Collaborator

@andyp1per I do not see UART5RX pinned out anywhere....where is it?

@andyp1per
Copy link
Collaborator Author

@andyp1per I do not see UART5RX pinned out anywhere....where is it?

No idea - maybe we should remove it from the hwdef

@andyp1per andyp1per deleted the pr-betafpv-f405-hwdef branch August 16, 2023 15:26
@Hwurzburg
Copy link
Collaborator

maybe it was ESC telem before they had BlueJay....but it should be removed...

Also, their recommended usages are:
UART1 (DMA enabled): GPS or CRSF...its protocol is currently MAV2....not okay for CRSF, nor for GPS since UART4 is GPS by default
UART4 (no DMA): only shown as DJI but its protocol default is GPS not DJI

I think we should (to make BF/iNav migration easy and avoid user confusion):

  1. remove UART5 and mark it EMPTY since its unusable
  2. set DJI protocol for UART4
  3. set GPS protocol for UART1
  4. change PC7 to TIM8 RCINT to allow SBUS to work even if a UART(3 in this case) is set for RC protocol

@andyp1per
Copy link
Collaborator Author

Yes agreed on 1-3. Don't agree with 4 without testing - they already have an SBUS pin out which I suspect is hardware inverted so no need for RCININT.

@Hwurzburg
Copy link
Collaborator

Hwurzburg commented Aug 17, 2023

Actually, I do not think that the SBUS pin has every been tested with this hwdef....I do not think it can work....it has an inverter which makes it useless as a normal UART RX since this is an F405....in order for it to work, I think you MUST use the definition I propose...

@Hwurzburg Hwurzburg added this to 4.4.0-beta2 in Plane 4.4 Aug 18, 2023
@rmackay9 rmackay9 added this to Pending in Copter-4.4 Sep 5, 2023
@rmackay9 rmackay9 moved this from Pending to 4.4.2-beta1 in Copter-4.4 Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Copter-4.4
4.4.2 / 4.4.2-beta1
Plane 4.4
4.4.0-beta2
Development

Successfully merging this pull request may close these issues.

None yet

4 participants