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

Added support for the LR1121 radio to the NRF52 Pro-Micro #5515

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

Nestpebble
Copy link
Contributor

Added support for the LR1121 radio, tested as functional with an E80-900M2213S from CDEbyte.

@Nestpebble
Copy link
Contributor Author

@markbirss I tried it and it works.
image

@markbirss
Copy link
Contributor

markbirss commented Dec 6, 2024

Added support for the LR1121 radio, tested as functional with an E80-900M2213S from CDEbyte.

I have also myself tried other values for "#define LR11X0_DIO3_TCXO_VOLTAGE 1.6"

1.8 and 3.0

@Nestpebble

It failing with trunk formatting issues

variants/diy/nrf52_promicro_diy_tcxo/Schematic_Pro-Micro_Pinouts.png
-:- fmt Unoptimized image, fix by running 'trunk fmt' oxipng

variants/diy/nrf52_promicro_diy_tcxo/variant.h
-:- fmt Incorrect formatting, autoformat by running 'trunk fmt' clang-format

I think there is a space line 158 after #endif in variant.h?

the png you might convert to pdf ?

@Nestpebble
Copy link
Contributor Author

I have also myself tried other values for "#define LR11X0_DIO3_TCXO_VOLTAGE 1.6"

1.8 and 3.0

Yeah, I just went with what has worked on other modules of the same type. 3.0V seems high unless the guidance is similar to Seeed, who iirc advise 0.2V under vcc

It failing with trunk formatting issues
[...]
I think there is a space line 158 after #endif in variant.h?

the png you might convert to pdf ?

Trunk doesn't work on windows, and my WSL is a mess. I'll sort it on a VM later. I'll convert the PNG to WEBP, too. Pdf may be too much.

@lyusupov
Copy link

lyusupov commented Dec 6, 2024

Ebyte reference source code uses 1.8 V of TCXO power supply voltage.

image

@lyusupov
Copy link

lyusupov commented Dec 6, 2024

E80-xxxM2213S_UserManual_EN_v1.0

DIO7 is a participant of RF switch matrix.

image

@markbirss
Copy link
Contributor

E80-xxxM2213S_UserManual_EN_v1.0

DIO7 is a participant of RF switch matrix.

image

But the E80 don't have a additional connection for it ?

@Nestpebble
Copy link
Contributor Author

E80-xxxM2213S_UserManual_EN_v1.0

DIO7 is a participant of RF switch matrix.

@caveman99 Can you comment on this? I'm cribbing your content here...

Added support for the LR1121 radio, tested as functional with an E80-900M2213S from CDEbyte.
It worked at 1.6v, but ¯\_(ツ)_/¯
@caveman99
Copy link
Member

caveman99 commented Dec 12, 2024

Not familiar with the actual E80 design here, but if you don't wanna use GNSS scanning the DIO7 can be left undefined. For good measure i would define it as it is switched to the right position by the lib then for sure. In the rfswitch.h simply add a 3d switch column for DIO7 that is always false except in the GNSS case.

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

Successfully merging this pull request may close these issues.

4 participants