From 75504793e84f760c3c10a313d93d6ab8fe89d302 Mon Sep 17 00:00:00 2001 From: Ben Meadors Date: Thu, 11 May 2023 06:52:27 -0500 Subject: [PATCH] Skip setting dio2 as rf switch altogether if txen is defined --- src/mesh/SX126xInterface.cpp | 15 ++++++++------- variants/diy/v1/variant.h | 6 +++--- 2 files changed, 11 insertions(+), 10 deletions(-) diff --git a/src/mesh/SX126xInterface.cpp b/src/mesh/SX126xInterface.cpp index c26b773f7..a6a4e9dc2 100644 --- a/src/mesh/SX126xInterface.cpp +++ b/src/mesh/SX126xInterface.cpp @@ -60,19 +60,20 @@ template bool SX126xInterface::init() LOG_DEBUG("Current limit set to %f\n", currentLimit); LOG_DEBUG("Current limit set result %d\n", res); -#ifdef SX126X_E22 - // E22 Emulation explicitly requires DIO2 as RF switch, so set it to TRUE again for good measure. In case somebody defines - // SX126X_TX for an E22 Module - if (res == RADIOLIB_ERR_NONE) - res = lora.setDio2AsRfSwitch(true); -#endif - #if defined(SX126X_TXEN) && (SX126X_TXEN != RADIOLIB_NC) // lora.begin sets Dio2 as RF switch control, which is not true if we are manually controlling RX and TX if (res == RADIOLIB_ERR_NONE) { + LOG_DEBUG("SX126X_TX/RX EN pins defined. Setting RF Switch: RXEN=%i, TXEN=%i\n", SX126X_RXEN, SX126X_TXEN); res = lora.setDio2AsRfSwitch(false); lora.setRfSwitchPins(SX126X_RXEN, SX126X_TXEN); } +#elif defined(SX126X_E22) + // E22 Emulation explicitly requires DIO2 as RF switch, so set it to TRUE again for good measure. In case somebody defines + // SX126X_TX for an E22 Module + if (res == RADIOLIB_ERR_NONE) { + LOG_DEBUG("SX126X_E22 mode enabled. Setting DIO2 as RF Switch\n"); + res = lora.setDio2AsRfSwitch(true); + } #endif if (config.lora.sx126x_rx_boosted_gain) { diff --git a/variants/diy/v1/variant.h b/variants/diy/v1/variant.h index da9e41553..51bc0f749 100644 --- a/variants/diy/v1/variant.h +++ b/variants/diy/v1/variant.h @@ -38,9 +38,9 @@ #define SX126X_DIO1 LORA_DIO1 #define SX126X_BUSY LORA_DIO2 #define SX126X_RESET LORA_RESET -//#define SX126X_RXEN 14 -//#define SX126X_TXEN 13 -#define SX126X_POWER_EN (13) +#define SX126X_RXEN 14 +#define SX126X_TXEN 13 +//#define SX126X_POWER_EN (13) // RX/TX for RFM95/SX127x #define RF95_RXEN 14