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

External nano modules missing on radios with standard JR bay #5467

Open
1 task done
JimB40 opened this issue Aug 20, 2024 · 8 comments
Open
1 task done

External nano modules missing on radios with standard JR bay #5467

JimB40 opened this issue Aug 20, 2024 · 8 comments
Labels
bug 🪲 Something isn't working triage Bug report awaiting review / sorting

Comments

@JimB40
Copy link
Collaborator

JimB40 commented Aug 20, 2024

Is there an existing issue for this problem?

  • I have searched the existing issues

What part of EdgeTX is the focus of this bug?

Transmitter firmware

Current Behavior

When selecting modules on radios with normal JR bay all modules using nano JR bay are missing. X9D, X9D2019, Jumper T16, RM TX16S etc
I've sold almost all JR bay modules leaving nano ones and using JR bay -> JR nano bay adapter.

While using full size JR modules on nano bay radios is clunky and inonvinient the other way, using nano module on radio with full JR bay using adapter very useful.

Expected Behavior

Radios with JR bay should have JR nano bay modules enlisted.

Steps To Reproduce

Version

2.10.3

Transmitter

RadioMaster TX16S / TX16SMK2

Operating System (OS)

No response

OS Version

No response

Anything else?

No response

@JimB40 JimB40 added bug 🪲 Something isn't working triage Bug report awaiting review / sorting labels Aug 20, 2024
@pfeerick
Copy link
Member

AFAIK, there are only three modules that exist only in Nano form, and thus not selectable:

  • XJT Lite
  • R9M Lite
  • R9M Pro

Are there any others? This is not a bug, but a very deliberate change => #3624

Something to keep in mind is that unless you 1) do the external access mod AND 2) have a Nano to JR module adapter that supports that (which I don't think can even exist, as the extra connections needed on the Nano module format have nowhere to go on the JR Micro bay)... they will never work with a JR bay radio.

Just FYI, there is no such thing as a JR Nano bay... JR Micro is the larger module format and Nano is the smaller one, and appears to have been created by FrSky, not Japan Remote Control.

All yours @3djc 🤭 🤣

@3djc
Copy link
Collaborator

3djc commented Aug 21, 2024

Depending on radio module combination, some may not be displayed of radio doesn't have a full usart module (

#if defined(HARDWARE_EXTERNAL_MODULE_SIZE_SML) and !defined(EXTMODULE_USART)
)

@pfeerick
Copy link
Member

pfeerick commented Aug 21, 2024

Just to give an example of what that means, the Jumper TPro doesn't have a full USART on the external module bay, so it is simply not possible to use ACCESS Nano modules with it (well, if you do some mods, and rejigger the firmware, maybe, just maybe you could... but why would you torture yourself when you can just get a radio that does work? 🤣)... even though it has a nano module bay... (i.e. nano module bays are not all made equal 😢). It is only the ACCESS modules that are a problem, as they need the additional USART connection.

@JimB40
Copy link
Collaborator Author

JimB40 commented Aug 21, 2024

Okay followin naming.

  1. JR Micro Bay (JRMB) - bigger module
  2. Nano Bay (NB) - smaller module

I wasn't aware about differences between hardware implementation of those two

Here are NB modules I have
IMG_2486

From Left to right.
0. BetaFPV JRMB -> NB adapter

  1. TBS nano
  2. Happymodel ELRS 2.4
  3. BetaFPV ELRS 900
  4. Ghost nano
  5. FrSky R9M Lite PRO
  6. FrSky XJT Lite
  7. Multimodule nano (missing on photo)

AFAIR almost all worked via adapter (hasn't checked 3 yet)
As I have 2.10 on all radios and NB modules are disabled for JRMB radios can't make tests.

Anyway AFAIU 90% cases they will work with some exceptions.

@3djc
Copy link
Collaborator

3djc commented Aug 21, 2024

Crossfire (ELRS or TBS), Ghost, Multi don't have specific size entry, so they are CRSF, GHOST, MULTI, been JR or nano doesn't matter

R9M Lite Pro / XJT Lite:

  • do require full UART that some radio don't have
  • require small module to be declared. On radio with JR bay, that means adding " -DMODULE_SIZE_SML=YES" to your compile

@pfeerick
Copy link
Member

Well, here are only three nano modules that won't be shown on a JR bay radio, listed in my earlier comment 🤣

Another point to consider is that ACCESS JR Micro bay modules will be excluded from being shown if the radio doesn't have the hardware capabilities, or hadn't have the externalaccessmod. The same is true for Nano bay radios also - although I don't think for them we have any externalaccessmods documented.

@JimB40
Copy link
Collaborator Author

JimB40 commented Aug 24, 2024

R9M Lite Pro / XJT Lite:

  • do require full UART that some radio don't have
  • require small module to be declared. On radio with JR bay, that means adding " -DMODULE_SIZE_SML=YES" to your compile

Did what you wrote ie build 2.10.3 fot TX16S with -DMODULE_SIZE_SML=YES
but the only one new module on list is R9MLite
no XJT Lite

@3djc
Copy link
Collaborator

3djc commented Aug 24, 2024

Dont think there is a full uart for the pxx2 on xjt lite, unless external access mod is done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🪲 Something isn't working triage Bug report awaiting review / sorting
Projects
None yet
Development

No branches or pull requests

3 participants