Merge pull request #14734 from jeromecoutant/PR_RFSWITCH

STM32WL LORA: HW specific out of STM32WL_LoRaRadio class

Author: 0xc0170

connectivity/drivers/lora/TARGET_STM32WL/STM32WL_LoRaRadio.cpp

@@ -123,16 +123,10 @@ static DigitalOut _rf_dbg_tx(MBED_CONF_STM32WL_LORA_DRIVER_DEBUG_TX, 0);
 #endif
 
 
-STM32WL_LoRaRadio::STM32WL_LoRaRadio(PinName rf_switch_ctrl1,
-                                     PinName rf_switch_ctrl2,
-                                     PinName rf_switch_ctrl3
-                                    )
-    :
-    _rf_switch_ctrl1(rf_switch_ctrl1, 0),
-    _rf_switch_ctrl2(rf_switch_ctrl2, 0),
-    _rf_switch_ctrl3(rf_switch_ctrl3, 0)
-
+STM32WL_LoRaRadio::STM32WL_LoRaRadio()
 {
+    set_antenna_switch(RBI_SWITCH_OFF);
+
     _radio_events = NULL;
     _image_calibrated = false;
     _force_image_calibration = false;
@@ -481,49 +475,6 @@ void STM32WL_LoRaRadio::Radio_SMPS_Set(uint8_t level)
     }
 }
 
-/**
-  * Sets up radio switch position according to the
-  * radio mode
-  */
-void STM32WL_LoRaRadio::set_antenna_switch(RBI_Switch_TypeDef state)
-{
-    // here we got to do ifdef for changing controls
-    // as some pins might be NC
-    switch (state) {
-        case RBI_SWITCH_OFF: {
-            /* Turn off switch */
-            _rf_switch_ctrl3 = 0;
-            _rf_switch_ctrl1 = 0;
-            _rf_switch_ctrl2 = 0;
-            break;
-        }
-        case RBI_SWITCH_RX: {
-            /*Turns On in Rx Mode the RF Switch */
-            _rf_switch_ctrl3 = 1;
-            _rf_switch_ctrl1 = 1;
-            _rf_switch_ctrl2 = 0;
-            break;
-        }
-        case RBI_SWITCH_RFO_LP: {
-            /*Turns On in Tx Low Power the RF Switch */
-            _rf_switch_ctrl3 = 1;
-            _rf_switch_ctrl1 = 1;
-            _rf_switch_ctrl2 = 1;
-            break;
-        }
-        case RBI_SWITCH_RFO_HP: {
-            /*Turns On in Tx High Power the RF Switch */
-            _rf_switch_ctrl3 = 1;
-            _rf_switch_ctrl1 = 0;
-            _rf_switch_ctrl2 = 1;
-            break;
-        }
-        default:
-            break;
-    }
-}
-/* End STM32WL specific HW defs */
-
 void STM32WL_LoRaRadio::calibrate_image(uint32_t freq)
 {
     uint8_t cal_freq[2];

connectivity/drivers/lora/TARGET_STM32WL/STM32WL_LoRaRadio.h

@@ -51,16 +51,14 @@ SPDX-License-Identifier: BSD-3-Clause
 #define MAX_DATA_BUFFER_SIZE_STM32WL                        255
 #endif
 
-
+extern void set_antenna_switch(RBI_Switch_TypeDef state);
 
 class STM32WL_LoRaRadio : public LoRaRadio {
 
 public:
-    STM32WL_LoRaRadio(PinName rf_switch_ctrl1 = MBED_CONF_STM32WL_LORA_DRIVER_RF_SWITCH_CTL1,
-                      PinName rf_switch_ctrl2 = MBED_CONF_STM32WL_LORA_DRIVER_RF_SWITCH_CTL2,
-                      PinName rf_switch_ctrl3 = MBED_CONF_STM32WL_LORA_DRIVER_RF_SWITCH_CTL3);
+    STM32WL_LoRaRadio();
 
-    virtual ~STM32WL_LoRaRadio();
+    ~STM32WL_LoRaRadio();
 
     /**
      * Registers radio events with the Mbed LoRaWAN stack and
@@ -315,11 +313,6 @@ class STM32WL_LoRaRadio : public LoRaRadio {
 
 private:
 
-    // Radio specific controls (TX/RX duplexer switch control)
-    mbed::DigitalOut _rf_switch_ctrl1;
-    mbed::DigitalOut _rf_switch_ctrl2;
-    mbed::DigitalOut _rf_switch_ctrl3;
-
     // Access protection
     PlatformMutex mutex;
 
@@ -369,7 +362,6 @@ class STM32WL_LoRaRadio : public LoRaRadio {
     uint8_t SUBGRF_SetRfTxPower(int8_t power);
     void SUBGRF_SetTxParams(uint8_t paSelect, int8_t power, radio_ramp_time_t rampTime);
     void Radio_SMPS_Set(uint8_t level);
-    void set_antenna_switch(RBI_Switch_TypeDef state);
     uint32_t RadioGetWakeupTime(void);
 
 

connectivity/drivers/lora/TARGET_STM32WL/STM32WL_radio_driver.cpp

@@ -0,0 +1,64 @@
+/* mbed Microcontroller Library
+ * SPDX-License-Identifier: BSD-3-Clause
+ ******************************************************************************
+ *
+ * Copyright (c) 2021 STMicroelectronics.
+ * All rights reserved.
+ *
+ * This software component is licensed by ST under BSD 3-Clause license,
+ * the "License"; You may not use this file except in compliance with the
+ * License. You may obtain a copy of the License at:
+ *                        opensource.org/licenses/BSD-3-Clause
+ *
+ ******************************************************************************
+ */
+
+#include "STM32WL_radio_driver.h"
+#include "drivers/DigitalOut.h"
+
+
+/* Sets up radio switch position according to the radio mode */
+/* This configuration is for NUCLEO_WL55JC */
+/* But provided as __weak so it has to be overwritten to match each specicific HW board */
+MBED_WEAK void set_antenna_switch(RBI_Switch_TypeDef state)
+{
+
+    // Radio specific controls (TX/RX duplexer switch control)
+    mbed::DigitalOut _rf_switch_ctrl1(MBED_CONF_STM32WL_LORA_DRIVER_RF_SWITCH_CTL1);
+    mbed::DigitalOut _rf_switch_ctrl2(MBED_CONF_STM32WL_LORA_DRIVER_RF_SWITCH_CTL2);
+    mbed::DigitalOut _rf_switch_ctrl3(MBED_CONF_STM32WL_LORA_DRIVER_RF_SWITCH_CTL3);
+
+    switch (state) {
+        case RBI_SWITCH_OFF: {
+            /* Turn off switch */
+            _rf_switch_ctrl3 = 0;
+            _rf_switch_ctrl1 = 0;
+            _rf_switch_ctrl2 = 0;
+            break;
+        }
+        case RBI_SWITCH_RX: {
+            /*Turns On in Rx Mode the RF Switch */
+            _rf_switch_ctrl3 = 1;
+            _rf_switch_ctrl1 = 1;
+            _rf_switch_ctrl2 = 0;
+            break;
+        }
+        case RBI_SWITCH_RFO_LP: {
+            /*Turns On in Tx Low Power the RF Switch */
+            _rf_switch_ctrl3 = 1;
+            _rf_switch_ctrl1 = 1;
+            _rf_switch_ctrl2 = 1;
+            break;
+        }
+        case RBI_SWITCH_RFO_HP: {
+            /*Turns On in Tx High Power the RF Switch */
+            _rf_switch_ctrl3 = 1;
+            _rf_switch_ctrl1 = 0;
+            _rf_switch_ctrl2 = 1;
+            break;
+        }
+        default:
+            break;
+    }
+}
+