From bb5800564138f8176b98827542679acd5e59ab59 Mon Sep 17 00:00:00 2001
From: BayCom GmbH <software@baycom.de>
Date: Thu, 21 Sep 2023 21:10:53 +0200
Subject: [PATCH] move RADIOLIB_SX127X_PREAMBLE_POLARITY_55 from ::config to
 ::begin & ::beginFSK

---
 src/modules/SX127x/SX127x.cpp | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/src/modules/SX127x/SX127x.cpp b/src/modules/SX127x/SX127x.cpp
index c0dba2a7..f79a11ed 100644
--- a/src/modules/SX127x/SX127x.cpp
+++ b/src/modules/SX127x/SX127x.cpp
@@ -51,6 +51,10 @@ int16_t SX127x::begin(uint8_t chipVersion, uint8_t syncWord, uint16_t preambleLe
   state = SX127x::setPreambleLength(preambleLength);
   RADIOLIB_ASSERT(state);
 
+  // set preamble polarity
+  state = setPreamblePolarity(RADIOLIB_SX127X_PREAMBLE_POLARITY_55);
+  RADIOLIB_ASSERT(state);
+
   // disable IQ inversion
   state = SX127x::invertIQ(false);
   RADIOLIB_ASSERT(state);
@@ -118,6 +122,10 @@ int16_t SX127x::beginFSK(uint8_t chipVersion, float freqDev, float rxBw, uint16_
   state = SX127x::setPreambleLength(preambleLength);
   RADIOLIB_ASSERT(state);
 
+  // set preamble polarity
+  state = setPreamblePolarity(RADIOLIB_SX127X_PREAMBLE_POLARITY_55);
+  RADIOLIB_ASSERT(state);
+
   // set default sync word
   uint8_t syncWord[] = {0x12, 0xAD};
   state = setSyncWord(syncWord, 2);
@@ -1453,10 +1461,6 @@ int16_t SX127x::configFSK() {
   state |= this->mod->SPIsetRegValue(RADIOLIB_SX127X_REG_PACKET_CONFIG_2, RADIOLIB_SX127X_DATA_MODE_PACKET | RADIOLIB_SX127X_IO_HOME_OFF, 6, 5);
   RADIOLIB_ASSERT(state);
 
-  // set preamble polarity
-  state =this->mod->SPIsetRegValue(RADIOLIB_SX127X_REG_SYNC_CONFIG, RADIOLIB_SX127X_PREAMBLE_POLARITY_55, 5, 5);
-  RADIOLIB_ASSERT(state);
-
   // set FIFO threshold
   state = this->mod->SPIsetRegValue(RADIOLIB_SX127X_REG_FIFO_THRESH, RADIOLIB_SX127X_TX_START_FIFO_NOT_EMPTY, 7, 7);
   state |= this->mod->SPIsetRegValue(RADIOLIB_SX127X_REG_FIFO_THRESH, RADIOLIB_SX127X_FIFO_THRESH, 5, 0);