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

Add E22-400M30S support for Hydra #2930

Merged
merged 11 commits into from
Dec 2, 2023

Conversation

S5NC
Copy link
Contributor

@S5NC S5NC commented Nov 6, 2023

Add support for E22-400M30S, based on @kkwestt's request. Please test on your board, we have not tested SX1268 configured output power vs E22-400M30S output power. If you have an SA I strongly recommend that you set the output power to a low value, such as +5 dBm and measure the output power, increasing up to 22 if necessary for you to reach 30 dBm output power from your module.

@S5NC
Copy link
Contributor Author

S5NC commented Nov 6, 2023

image
The E21-400G30S outputs 30 dBm with 20 dBm input power. I suspect EBYTE would reuse their amplifiying circuit from their amplifying-only board?

Copy link
Contributor

github-actions bot commented Nov 6, 2023

🤖 Pull request artifacts

file commit
pr2930-firmware-2.2.14.0713be2.zip 0713be2

thebentern added a commit to meshtastic/artifacts that referenced this pull request Nov 6, 2023
@S5NC
Copy link
Contributor Author

S5NC commented Nov 6, 2023

We should wait to merge until after testing is done to find the correct amount of power to input into the E22-400M30S, we can't risk releasing firmware which damages radio modules. If the value is not 22 dBm, we would need to create a separate variant file for the E22-400M30S hydra, as we can't distinguish between the SX1268 or SX1262's max power on the go, as there is just one setting for the SX126x's max power.

@S5NC S5NC changed the title Update variant.h Add E22-400M30S support for Hydra Nov 6, 2023
@sniporbob
Copy link

If someone in the US has one of these and wants to ship it to me for testing, I'll put it on the spectrum analyzer to verify the proper setting before shipping it back.

thebentern added a commit to meshtastic/artifacts that referenced this pull request Nov 16, 2023
@S5NC S5NC mentioned this pull request Nov 22, 2023
thebentern added a commit to meshtastic/artifacts that referenced this pull request Nov 30, 2023
@thebentern thebentern merged commit 9e90b4a into meshtastic:master Dec 2, 2023
55 checks passed
@thebentern
Copy link
Contributor

Going to go ahead and integrate this to get it off of our plates. Nobody has / is running this hardware current, and it's a DIY variant which carries implicit risks. We can always circle back and figure out if any fixes are needed.

We should wait to merge until after testing is done to find the correct amount of power to input into the E22-400M30S, we can't risk releasing firmware which damages radio modules. If the value is not 22 dBm, we would need to create a separate variant file for the E22-400M30S hydra, as we can't distinguish between the SX1268 or SX1262's max power on the go, as there is just one setting for the SX126x's max power.

@S5NC S5NC deleted the hydra-SX1268-support branch December 2, 2023 15:54
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.

3 participants