Skip to content

Heltec v4.3: enable LNA by default#9906

Merged
thebentern merged 1 commit intomeshtastic:developfrom
weebl2000:heltec-v4.3-enable-lna-by-default
Mar 19, 2026
Merged

Heltec v4.3: enable LNA by default#9906
thebentern merged 1 commit intomeshtastic:developfrom
weebl2000:heltec-v4.3-enable-lna-by-default

Conversation

@weebl2000
Copy link
Contributor

@weebl2000 weebl2000 commented Mar 14, 2026

It should only be disabled by users that have problems with it. Anyone using Heltec v4.3 and not setting it explicitly will have bad RX.

Get firmware here:

@CLAassistant
Copy link

CLAassistant commented Mar 14, 2026

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Wessel Nieboer seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions github-actions bot added needs-review Needs human review hardware-support Hardware related: new devices or modules, problems specific to hardware labels Mar 14, 2026
@weebl2000 weebl2000 force-pushed the heltec-v4.3-enable-lna-by-default branch from 8bbb621 to a62feea Compare March 14, 2026 13:55
@weebl2000 weebl2000 changed the title Enable LNA by default Heltec v4.3: enable LNA by default Mar 14, 2026
@weebl2000 weebl2000 force-pushed the heltec-v4.3-enable-lna-by-default branch from a62feea to 08e6792 Compare March 14, 2026 13:56
@fifieldt
Copy link
Member

Thanks! Can we get a few v4 users to share their results? Code seems fine, just feels like we're flipping back and forth a little on this device.

@weebl2000
Copy link
Contributor Author

weebl2000 commented Mar 15, 2026

Thanks! Can we get a few v4 users to share their results? Code seems fine, just feels like we're flipping back and forth a little on this device.

Fairly certain ON is the way to go by default. FYI, Heltec v4.2 and earlier didn't have a way to turn off the low noise amplifier at all. It was always physically in the RX path.

Heltec v4.3 came out with the main purpose being for users to able to bypass the LNA without doing a hardware bypass mod. Realistically you'd only want to disable it if you have problematic RF interference.

@weebl2000 weebl2000 force-pushed the heltec-v4.3-enable-lna-by-default branch from 08e6792 to 85fa380 Compare March 15, 2026 07:41
@iuvi7
Copy link

iuvi7 commented Mar 15, 2026

I'm Alien here, but maybe it can be an option to choose in web flasher for v4 like "Choose version you have 4.1\4.2 or 4.3" ?

@weebl2000
Copy link
Contributor Author

I'm Alien here, but maybe it can be an option to choose in web flasher for v4 like "Choose version you have 4.1\4.2 or 4.3" ?

Not needed. All versions are supported by the firmware. 4.2 and earlier didn't have an LNA you could toggle on/off with software so the configuration does not apply there. It's only relevant to 4.3+

@fifieldt fifieldt added the enhancement New feature or request label Mar 16, 2026
@weebl2000 weebl2000 force-pushed the heltec-v4.3-enable-lna-by-default branch from 85fa380 to ce37a6e Compare March 16, 2026 14:37
It should only be disabled by users that have problems with it.
@weebl2000 weebl2000 force-pushed the heltec-v4.3-enable-lna-by-default branch from a9a9c1a to e0c6abb Compare March 19, 2026 13:01
@thebentern thebentern merged commit 1be2529 into meshtastic:develop Mar 19, 2026
64 checks passed
thebentern pushed a commit that referenced this pull request Mar 19, 2026
It should only be disabled by users that have problems with it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request hardware-support Hardware related: new devices or modules, problems specific to hardware needs-review Needs human review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants