Skip to content

Disable PA in TXRXdisable() whether in rx_enable or tx_enable state for esp8285#3485

Merged
pkendall64 merged 1 commit intoExpressLRS:3.x.x-maintenancefrom
Ncerzzk:3.x.x-maintenance
Jan 11, 2026
Merged

Disable PA in TXRXdisable() whether in rx_enable or tx_enable state for esp8285#3485
pkendall64 merged 1 commit intoExpressLRS:3.x.x-maintenancefrom
Ncerzzk:3.x.x-maintenance

Conversation

@Ncerzzk
Copy link
Contributor

@Ncerzzk Ncerzzk commented Jan 8, 2026

Hi,
For now, we only disable PA in tx_enable state in 8285.
This patch disable PA in all situation when call TXRXdisable.

And Why to do this change:
Actually I'm making a RX hardware which could let esp8285 use SX1280's antenna by a RF switch.
(I'm fed up with the terrible Wi-Fi signal in current ESP8285 hardware because I often use
the msp2wifi function to configure INav FC)

and I try to find a PIN which could control the RF Switch when wifi enable, PA_ENABLE is fitable.
and I find the PA_ENABLE never be lower when in rx_enable state, so do this change.

On the other hand, to keep the behavior consistency with esp32 and esp32C3, we should also do it.

Thank you! feel free to comment.

@Ncerzzk Ncerzzk changed the title Disable PA in TXRXdisable() whether in rx_enable or tx_enable state Disable PA in TXRXdisable() whether in rx_enable or tx_enable state for esp8285 Jan 8, 2026
@pkendall64 pkendall64 merged commit b036e05 into ExpressLRS:3.x.x-maintenance Jan 11, 2026
51 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants