netdev_driver: modifications after TX debug

Author: Matthieu Saignemorte

stack/framework/hal/chips/netdev_driver/sx127x_getset.c

@@ -58,7 +58,6 @@ uint8_t sx127x_get_state(const sx127x_t *dev)
 
 void sx127x_set_state(sx127x_t *dev, uint8_t state)
 {
-#if ENABLE_DEBUG
     switch (state) {
     case SX127X_RF_IDLE:
         DEBUG("[sx127x] Change state: IDLE\n");
@@ -73,7 +72,6 @@ void sx127x_set_state(sx127x_t *dev, uint8_t state)
         DEBUG("[sx127x] Change state: UNKNOWN\n");
         break;
     }
-#endif
 
     dev->settings.state = state;
 }
@@ -629,7 +627,6 @@ uint8_t sx127x_get_op_mode(const sx127x_t *dev)
 
 void sx127x_set_op_mode(const sx127x_t *dev, uint8_t op_mode)
 {
-#if ENABLE_DEBUG
     switch(op_mode) {
     case SX127X_RF_OPMODE_SLEEP:
         DEBUG("[sx127x] Set op mode: SLEEP\n");
@@ -650,16 +647,13 @@ void sx127x_set_op_mode(const sx127x_t *dev, uint8_t op_mode)
         DEBUG("[sx127x] Set op mode: UNKNOWN (%d)\n", op_mode);
         break;
     }
-#endif
 
 #if defined(PLATFORM_SX127X_USE_MANUAL_RXTXSW_PIN) || defined(PLATFORM_USE_ABZ)
   set_antenna_switch(dev, op_mode);
 #endif
 
     /* Replace previous mode value and setup new mode value */
-    sx127x_reg_write(dev, SX127X_REG_OPMODE,
-                     (sx127x_reg_read(dev, SX127X_REG_OPMODE) &
-                      SX127X_RF_OPMODE_MASK) | op_mode);
+    sx127x_reg_write(dev, SX127X_REG_OPMODE, op_mode);
 }
 
 uint32_t computeRxBw( uint8_t mantisse, uint8_t exponent )

stack/framework/hal/chips/netdev_driver/sx127x_netdev.c

@@ -119,6 +119,7 @@ static int _send(netdev_t *netdev, const iolist_t *iolist)
         }
     }
 
+
     /* FIFO operations can not take place in Sleep mode
      * So wake up the chip */
     if (sx127x_get_op_mode(dev) == SX127X_RF_OPMODE_SLEEP) {
@@ -153,8 +154,17 @@ static int _send(netdev_t *netdev, const iolist_t *iolist)
                 dev->packet.pos = size;
                 if (dev->options & SX127X_OPT_TELL_TX_REFILL) // we expect to refill the FIFO with subsequent data
                 {
-                    sx127x_reg_write(dev, SX127X_REG_FIFOTHRESH, 0x80 | (size/2 - 1));
-                    dev->packet.fifothresh = size/2;
+                	// Setting the fifo threshold to half the packet length
+                	if ((dev->packet.length % 2) == 1)
+                	{
+                		dev->packet.fifothresh = ((dev->packet.length+1)/2);
+                		sx127x_reg_write(dev, SX127X_REG_FIFOTHRESH, (0x80 | (dev->packet.fifothresh-1)));
+                	}
+                	else
+                	{
+                		dev->packet.fifothresh = ((dev->packet.length/2)+1);
+                		sx127x_reg_write(dev, SX127X_REG_FIFOTHRESH, (0x80 | (dev->packet.fifothresh-1)));
+                	}
 
                     sx127x_write_fifo(dev, iolist->iol_base, size);
                     hw_gpio_set_edge_interrupt(dev->params.dio1_pin, GPIO_FALLING_EDGE);
@@ -221,9 +231,16 @@ static int _send(netdev_t *netdev, const iolist_t *iolist)
             timer_add_event(&dev->_internal.tx_timeout_timer);
         }
 
-        /* Put chip into transfer mode */
-        sx127x_set_state(dev, SX127X_RF_TX_RUNNING);
-        sx127x_set_op_mode(dev, SX127X_RF_OPMODE_TRANSMITTER);
+        if (sx127x_get_state(dev) & SX127X_RF_TX_RUNNING) {}
+        else {
+        	/* Put chip into Standby mode */
+            sx127x_set_state(dev, SX127X_RF_IDLE);
+            sx127x_set_op_mode(dev, SX127X_RF_OPMODE_STANDBY);
+
+        	/* Put chip into transfer mode */
+        	sx127x_set_state(dev, SX127X_RF_TX_RUNNING);
+        	sx127x_set_op_mode(dev, SX127X_RF_OPMODE_TRANSMITTER);
+        }
     }
 
     return 0;
@@ -372,6 +389,9 @@ static int _init(netdev_t *netdev)
     /* Put chip into sleep */
     sx127x_set_sleep(sx127x);
 
+    /* Put chip into standby mode*/
+    sx127x_set_standby(sx127x);
+
     DEBUG("[sx127x] init_radio: sx127x initialization done\n");
     init_done = true;