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.

Sign up for GitHub

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

Jump to bottom

R9M/FLEX colorlcd has missing options #1187

Closed
pfeerick opened this issue Nov 28, 2021 · 6 comments · Fixed by #4198
Closed

R9M/FLEX colorlcd has missing options #1187

pfeerick opened this issue Nov 28, 2021 · 6 comments · Fixed by #4198
Labels
color Related generally to color LCD radios hardware support

Comments

@pfeerick
Copy link
Member

In enabling FLEX support as part of #1043, it was noticed that there is no "S.Port Link Enabled" shown at any time or RF Power option when FLEX selected.

image
image
@pfeerick pfeerick added color Related generally to color LCD radios hardware support labels Nov 28, 2021
@pfeerick
Copy link
Member Author

Tested and working on NV14 and T16. I can now bind to models using R9 FLEX firmware. Still missing the option for the S.Port link and the power setting but good start.

#1043 (comment)

@DamienPlo
Copy link

I confirm after testing 2.6 RC3 on my TX16S, no power setting available in flex mode after selecting 915 or 868 Mhz. So I don't know what output power is by default.

@federikk
Copy link

federikk commented Feb 1, 2022 •
edited
Loading

I'm appendig this one here, don't know if it's actually a separate issue:
It must be selected FCC mode on the TX in order to have any output from the RX toward the FC.
Even if registration and binding has been obtained using EU or 868.
RcLink and telemetry can be estabilished instead using any band(FCC/EU/915/868). Outputs on BF/iNav rclink page are stuck but not in "no pulse" status/triggering failsafe, simply not moving. Setting the radio to FCC solves the problem (also on 868/EU binded rx) but i can't measure at which frequency (868 or 915) and maxpower (overlapping to this 1187).
Found with Tx16s, R9M2019, 2.6.0 custom externalaccessmod, F-port protocol, R9mm-ota, R9 Slim+ ota, betaflight 4.3, iNav 4.0. frsky stuff on latest firmware.

@Arvycka
Copy link
Contributor

Arvycka commented Apr 20, 2022

R9M Flex power option is still missing in 2.7, but is available in the edgetx companion as well as inside the model yaml file. But is the value set there even used by the radio/module?

Arvycka added a commit to Arvycka/edgetx that referenced this issue Apr 23, 2022
@Arvycka
R9M Flex power option menu for colorlcd
e2257ef 
Adds power selection menu for R9M Flex  (ACCST, pre 2019) on colorlcd radios. Tested working on a tx16s.
EdgeTX#1187
This was referenced Apr 23, 2022
Merged
Merged
pfeerick added a commit that referenced this issue May 8, 2022
@Arvycka @pfeerick
fix(colorlcd): Missing R9M Flex power option menu (#1880)
9070a55 
* Update model_setup.cpp

Adds R9M Flex (ACCST, non ACCESS) Power option GUI menu item, which is currently missing.

* Update model_setup.cpp

* R9M Flex power option menu for colorlcd

Adds power selection menu for R9M Flex  (ACCST, pre 2019) on colorlcd radios. Tested working on a tx16s.
#1187

* Just test for FCC variants

Co-authored-by: Peter Feerick <[email protected]>
pfeerick added a commit that referenced this issue May 8, 2022
@Arvycka @pfeerick
fix(colorlcd): Missing R9M Flex power option menu (#1880)
905d75e 
* Update model_setup.cpp

Adds R9M Flex (ACCST, non ACCESS) Power option GUI menu item, which is currently missing.

* Update model_setup.cpp

* R9M Flex power option menu for colorlcd

Adds power selection menu for R9M Flex  (ACCST, pre 2019) on colorlcd radios. Tested working on a tx16s.
#1187

* Just test for FCC variants

Co-authored-by: Peter Feerick <[email protected]>
@pfeerick pfeerick mentioned this issue Feb 13, 2023
Closed
3 tasks
@gagarinlg
Copy link
Member

Ths S.Port link option is still missing in main

@raphaelcoeffic
Copy link
Member

Ths S.Port link option is still missing in main

This is just a status display, not an option. Here is the code in OpenTX:
Screenshot 2023-10-14 at 13 40 39

We have now some better code as shown in #4198 (modulePortHasRx(module)).

raphaelcoeffic added a commit that referenced this issue Oct 14, 2023
@raphaelcoeffic
feat: add S.PORT status for external R9M module
ff27271 
Fixes #1187
@raphaelcoeffic raphaelcoeffic mentioned this issue Oct 14, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
color Related generally to color LCD radios hardware support
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: MPM module status EdgeTX/edgetx
6 participants
@raphaelcoeffic @pfeerick @federikk @Arvycka @gagarinlg @DamienPlo