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

MRO Control Zero F7 enable UART7 #16605

Closed
wants to merge 2 commits into from

Conversation

ryanjAA
Copy link
Contributor

@ryanjAA ryanjAA commented Jan 21, 2021

UART7 is currently set to Console but as USART6 is not available/working, this makes sense to enable as a serial port (Telemetry 3). I tried setting it as GPS 2 but for some reason that would not work. It would not enable when selecting gps 2 port for a mavlink instance in qgc.

Even if USART6 was working, setting UART7 to an available port makes sense on this board.

Console was moved to UART8 which is Frsky Telemetry, that is likely not used very much.

The objective was to disable the Console completely but by doing so the autopilot would not boot up.

Tested and works correctly.

USART6 is not working correctly so enabling UART7 as TEL3 makes sense while disabling Console on the port.
The objective was to disable the Console (as UART7 is needed due to USART6 not available/working) but by omitting the Console line completely it stops the Control Zero from booting. Moving this to UART8 (which is Frsky Telemetry and likely used sparsely makes the most sense).

Tested and works.
@mrpollo
Copy link
Contributor

mrpollo commented Feb 15, 2021

@ryanjAA did you test fly this? if so can you please post a flight log? I will make sure we get this in ASAP before the v1.12 release which we are getting ready to cut.

@ryanjAA
Copy link
Contributor Author

ryanjAA commented Feb 15, 2021

I haven’t flown this one yet since there is an H7 (processor) and H7 oem variant we’re working on as well. We’ll get them all test flown and logs updated the same day. Just waiting on getting the h7 units operational.

@dagar wanted to change a few things that were added to master and with that and once the H7 stuff is finalized I’ll get it uploaded. Hopefully before 1.12 is cut.

@mrpollo
Copy link
Contributor

mrpollo commented Feb 15, 2021

Thanks for the update I'll keep a close eye on this one @ryanjAA

@ryanjAA
Copy link
Contributor Author

ryanjAA commented Feb 15, 2021

Sorry @mrpollo this specific Pr I’ve flown and have logs. Let me track them down.

I thought this was the pr on bringing up the the f7 oem board (responding from my phone in our strange Texas snow storm).

@mrpollo
Copy link
Contributor

mrpollo commented Mar 2, 2021

@ryanjAA were you able to find the test logs?

@ryanjAA
Copy link
Contributor Author

ryanjAA commented Mar 2, 2021

@mrpollo
Copy link
Contributor

mrpollo commented Mar 2, 2021

@dagar what do you think? we good to go?

@ryanjAA
Copy link
Contributor Author

ryanjAA commented Mar 17, 2021

From a continuity standpoint the other MRO Ctrl Zero boards implemented I did the UART mapping the same as this PR so this will be the one board different from the other 3 until this is merged in.

@mrpollo

@mrpollo
Copy link
Contributor

mrpollo commented Mar 17, 2021

Why is this one different?

@ryanjAA
Copy link
Contributor Author

ryanjAA commented Mar 18, 2021

It's not that it is different per se, it's that the use case of serial ports makes more sense to have them this way so when doing the other boards, that's the way I routed them so this is the only one that is different since I didn't do the original board config.

@stale
Copy link

stale bot commented Jun 26, 2021

This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions.

@stale stale bot added the stale label Jun 26, 2021
@dagar
Copy link
Member

dagar commented Sep 15, 2022

I believe this is no longer relevant?

@dagar dagar closed this Sep 15, 2022
mRo Hardware automation moved this from In progress to Done Sep 15, 2022
@ryanjAA
Copy link
Contributor Author

ryanjAA commented Sep 15, 2022

F7 EOL so yes, not relevant.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants