-
-
Notifications
You must be signed in to change notification settings - Fork 350
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: Internal PXX exclusive S.PORT #3822
Conversation
50fc013
to
c9b5033
Compare
Tested on X7 with internal XJT and external multi, each having a receiver connected to it:
what doesnt: |
Ok, so I'll to check it out with real hardware. Maybe I can add some unit tests for MPM and see how that goes. |
345bf43
to
b040846
Compare
b040846
to
964bd62
Compare
I'm having trouble with the MPM here. Before this PR, with X9D+2019 (ISRM) + external MPM, I could use both MPM and ISRM simultaneously... just the MPM telemetry would get cut. As soon as I load this PR, I can not get the MPM telemetry up, regardless of whether the ISRM is on or off. |
This is necessary as these module hog the S.PORT line, which is shared with the external module.
964bd62
to
d48e0d3
Compare
I'm having issues understanding this. This PR does not change PXX2 handling, nor should it change anything in the MPM behaviour in your specific case. There must be something big I missed here. |
I'll try this again today, and also double-check against main to make sure the issue isn't actually there. |
Both nightly and the latest rebase are fine on X9D+2019 (ISRM) now. 🤷 I've also tried on the X9D+ now with this PR, and gone through all three of JCs failure cases and they are working for me. It's nice to see the prompt about disabling internal RF rather than just "no telemetry". |
A further enhancement that would be worth considering (since you mentioned CRSF;GHOST) would be for it to be somehow blocked - whether unelectable or error msg shown - until internal RF is disabled (since it will effectively kill internal RF if you enable GHOST after the internal RF is enabled - which you can do still on the X9D+ - you just can't enable the internal RF after enabling GHST). The internal RF is also currently not re-enabled if it was originally active, and GHOST is then enabled. and disabled. I've not tried with CRSF, but assume it would be the same. |
This PR is meant to solve the issue caused by the shared S.PORT line between internal and external module on FrSky radios. The internal module on these radios take the S.PORT line in ownership when turned ON, and there is no way to turn it OFF.
Summary of Changes:
Please note: In case an external module uses S.PORT both for TX and RX (CRSF, GHOST), the initialisation of the internal module will fail. The UI should prevent this with the existing code.
PXX2 internal modules:
2.1.9 EU
on X10 Express) do not clog S.PORT anymore,