From 4c3e14c3444f2a3dc1b45128c56857cda547d749 Mon Sep 17 00:00:00 2001
From: Davide Mor <davide.mor@skywarder.eu>
Date: Mon, 29 May 2023 22:59:31 +0200
Subject: [PATCH] [sx1278] Updated submodule

---
 src/tests/radio/sx1278/fsk/test-sx1278-bidir.cpp    | 2 +-
 src/tests/radio/sx1278/fsk/test-sx1278-mavlink.cpp  | 2 +-
 src/tests/radio/sx1278/lora/test-sx1278-bidir.cpp   | 2 +-
 src/tests/radio/sx1278/lora/test-sx1278-mavlink.cpp | 2 +-
 src/tests/radio/sx1278/lora/test-sx1278-simple.cpp  | 2 +-
 src/tests/radio/sx1278/sx1278-init.h                | 8 ++++----
 src/tests/radio/sx1278/test-sx1278-bench.cpp        | 8 ++++----
 7 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/src/tests/radio/sx1278/fsk/test-sx1278-bidir.cpp b/src/tests/radio/sx1278/fsk/test-sx1278-bidir.cpp
index 0472f464d..0772d57e4 100644
--- a/src/tests/radio/sx1278/fsk/test-sx1278-bidir.cpp
+++ b/src/tests/radio/sx1278/fsk/test-sx1278-bidir.cpp
@@ -51,7 +51,7 @@ using rxen = Gpio<GPIOD_BASE, 4>;
 #define SX1278_SPI SPI4
 
 #define SX1278_IRQ_DIO0 EXTI6_IRQHandlerImpl
-#define SX1278_IRQ_DIO1 EXTI2_IRQHandlerImpl
+#define SX1278_IRQ_DIO1 EXTI4_IRQHandlerImpl
 #define SX1278_IRQ_DIO3 EXTI11_IRQHandlerImpl
 
 #else
diff --git a/src/tests/radio/sx1278/fsk/test-sx1278-mavlink.cpp b/src/tests/radio/sx1278/fsk/test-sx1278-mavlink.cpp
index 109315e1b..095dc3076 100644
--- a/src/tests/radio/sx1278/fsk/test-sx1278-mavlink.cpp
+++ b/src/tests/radio/sx1278/fsk/test-sx1278-mavlink.cpp
@@ -67,7 +67,7 @@ using mosi = interfaces::spi4::mosi;
 #define SX1278_SPI SPI4
 
 #define SX1278_IRQ_DIO0 EXTI6_IRQHandlerImpl
-#define SX1278_IRQ_DIO1 EXTI2_IRQHandlerImpl
+#define SX1278_IRQ_DIO1 EXTI4_IRQHandlerImpl
 #define SX1278_IRQ_DIO3 EXTI11_IRQHandlerImpl
 
 #else
diff --git a/src/tests/radio/sx1278/lora/test-sx1278-bidir.cpp b/src/tests/radio/sx1278/lora/test-sx1278-bidir.cpp
index 79ee1725d..ec7946566 100644
--- a/src/tests/radio/sx1278/lora/test-sx1278-bidir.cpp
+++ b/src/tests/radio/sx1278/lora/test-sx1278-bidir.cpp
@@ -54,7 +54,7 @@ using rxen = Gpio<GPIOD_BASE, 4>;
 #define SX1278_SPI SPI4
 
 #define SX1278_IRQ_DIO0 EXTI6_IRQHandlerImpl
-#define SX1278_IRQ_DIO1 EXTI2_IRQHandlerImpl
+#define SX1278_IRQ_DIO1 EXTI4_IRQHandlerImpl
 #define SX1278_IRQ_DIO3 EXTI11_IRQHandlerImpl
 
 #else
diff --git a/src/tests/radio/sx1278/lora/test-sx1278-mavlink.cpp b/src/tests/radio/sx1278/lora/test-sx1278-mavlink.cpp
index 63ed5eeb7..a35320666 100644
--- a/src/tests/radio/sx1278/lora/test-sx1278-mavlink.cpp
+++ b/src/tests/radio/sx1278/lora/test-sx1278-mavlink.cpp
@@ -74,7 +74,7 @@ using rxen = Gpio<GPIOD_BASE, 4>;
 #define SX1278_SPI SPI4
 
 #define SX1278_IRQ_DIO0 EXTI6_IRQHandlerImpl
-#define SX1278_IRQ_DIO1 EXTI2_IRQHandlerImpl
+#define SX1278_IRQ_DIO1 EXTI4_IRQHandlerImpl
 #define SX1278_IRQ_DIO3 EXTI11_IRQHandlerImpl
 
 #else
diff --git a/src/tests/radio/sx1278/lora/test-sx1278-simple.cpp b/src/tests/radio/sx1278/lora/test-sx1278-simple.cpp
index 02bbe5489..8bd407e8e 100644
--- a/src/tests/radio/sx1278/lora/test-sx1278-simple.cpp
+++ b/src/tests/radio/sx1278/lora/test-sx1278-simple.cpp
@@ -50,7 +50,7 @@ using rxen = Gpio<GPIOD_BASE, 4>;
 #define SX1278_SPI SPI4
 
 #define SX1278_IRQ_DIO0 EXTI6_IRQHandlerImpl
-#define SX1278_IRQ_DIO1 EXTI2_IRQHandlerImpl
+#define SX1278_IRQ_DIO1 EXTI4_IRQHandlerImpl
 #define SX1278_IRQ_DIO3 EXTI11_IRQHandlerImpl
 
 #else
diff --git a/src/tests/radio/sx1278/sx1278-init.h b/src/tests/radio/sx1278/sx1278-init.h
index 2ed08f3af..30cc3ae33 100644
--- a/src/tests/radio/sx1278/sx1278-init.h
+++ b/src/tests/radio/sx1278/sx1278-init.h
@@ -58,7 +58,7 @@ using rxen = miosix::Gpio<GPIOD_BASE, 4>;
 #define SX1278_SPI SPI4
 
 #define SX1278_IRQ_DIO0 EXTI6_IRQHandlerImpl
-#define SX1278_IRQ_DIO1 EXTI2_IRQHandlerImpl
+#define SX1278_IRQ_DIO1 EXTI4_IRQHandlerImpl
 #define SX1278_IRQ_DIO3 EXTI11_IRQHandlerImpl
 
 #elif defined _BOARD_STM32F429ZI_SKYWARD_RIG
@@ -132,19 +132,19 @@ void initBoard()
 #ifdef SX1278_IRQ_DIO0
     miosix::GpioPin dio0_pin = dio0::getPin();
     enableExternalInterrupt(dio0_pin.getPort(), dio0_pin.getNumber(),
-                            InterruptTrigger::RISING_EDGE);
+                            InterruptTrigger::RISING_FALLING_EDGE);
 #endif
 
 #ifdef SX1278_IRQ_DIO1
     miosix::GpioPin dio1_pin = dio1::getPin();
     enableExternalInterrupt(dio1_pin.getPort(), dio1_pin.getNumber(),
-                            InterruptTrigger::RISING_EDGE);
+                            InterruptTrigger::RISING_FALLING_EDGE);
 #endif
 
 #ifdef SX1278_IRQ_DIO3
     miosix::GpioPin dio3_pin = dio3::getPin();
     enableExternalInterrupt(dio3_pin.getPort(), dio3_pin.getNumber(),
-                            InterruptTrigger::RISING_EDGE);
+                            InterruptTrigger::RISING_FALLING_EDGE);
 #endif
 }
 
diff --git a/src/tests/radio/sx1278/test-sx1278-bench.cpp b/src/tests/radio/sx1278/test-sx1278-bench.cpp
index a68dcbff5..fd0607ad3 100644
--- a/src/tests/radio/sx1278/test-sx1278-bench.cpp
+++ b/src/tests/radio/sx1278/test-sx1278-bench.cpp
@@ -114,7 +114,7 @@ void recvLoop()
         TestMsg msg = {};
 
         sx1278->receive((uint8_t *)&msg, sizeof(msg));
-        if (true)
+        if (msg.validate())
         {
             stats.recv_count++;
 
@@ -167,14 +167,14 @@ void spawnThreads()
     send.detach();
 #endif
 
-    /* For now, I'll keep it here, just in case ...
-    std::thread watchdog([]() {
+    // For now, I'll keep it here, just in case ...
+    /* std::thread watchdog([]() {
         while(1) {
             {
                 FastInterruptDisableLock dlock;
                 sx1278->handleDioIRQ();
             }
-            Thread::sleep(200);
+            Thread::sleep(2);
         }
     });
     watchdog.detach();
-- 
GitLab