fix: RTC boot check not working

radio/src/gui/colorlcd/LvglWrapper.cpp

@@ -267,14 +267,14 @@ static void rotaryDriverRead(lv_indev_drv_t *drv, lv_indev_data_t *data)
   static int8_t prevDir = 0;
   static uint32_t lastDt = 0;
 
-  rotenc_t newPos = rotaryEncoderGetRawValue();
-  rotenc_t diff = (newPos - prevPos) / ROTARY_ENCODER_GRANULARITY;
-  prevPos += diff * ROTARY_ENCODER_GRANULARITY;
+  rotenc_t newPos = rotaryEncoderGetValue();
+  rotenc_t diff = newPos - prevPos;
 
   data->enc_diff = (int16_t)diff;
   data->state = LV_INDEV_STATE_RELEASED;
 
   if (diff != 0) {
+    prevPos = newPos;
     reset_inactivity();
 
     int8_t dir = 0;

radio/src/hal/rotary_encoder.h

@@ -40,9 +40,5 @@ void rotaryEncoderInit();
 // return impulses / granularity
 rotenc_t rotaryEncoderGetValue();
 
-// returns raw # impulses
-rotenc_t rotaryEncoderGetRawValue();
-
-
 int8_t rotaryEncoderGetAccel();
 void rotaryEncoderResetAccel();

radio/src/opentx.cpp

@@ -629,7 +629,7 @@ static void checkFailsafe()
 #endif
 
 #if defined(GUI)
-void checkAll()
+void checkAll(bool isBootCheck)
 {
 #if defined(EEPROM_RLC) && !defined(SDCARD_RAW) && !defined(SDCARD_YAML)
   checkLowEEPROM();
@@ -645,9 +645,9 @@ void checkAll()
   checkSwitches();
   checkFailsafe();
 
-
-  if (isVBatBridgeEnabled() && !g_eeGeneral.disableRtcWarning) {
+  if (isBootCheck && !g_eeGeneral.disableRtcWarning) {
     // only done once at board start
+    enableVBatBridge();
     checkRTCBattery();
   }
   disableVBatBridge();
@@ -1532,7 +1532,7 @@ void edgeTxInit()
     }
     else if (!(startOptions & OPENTX_START_NO_CHECKS)) {
       checkAlarm();
-      checkAll();
+      checkAll(true);
       PLAY_MODEL_NAME();
     }
 #endif // defined(GUI)

radio/src/opentx.h

@@ -336,7 +336,7 @@ void checkLowEEPROM();
 void checkThrottleStick();
 void checkSwitches();
 void checkAlarm();
-void checkAll();
+void checkAll(bool isBootCheck = false);
 
 void getADC();
 

radio/src/targets/common/arm/stm32/diskio_sdio.cpp

@@ -263,15 +263,12 @@ static DRESULT sdio_read(BYTE lun, BYTE * buff, DWORD sector, UINT count)
   res = _read_dma(buff, sector, count);
   if (res != RES_OK) return res;
 
-  uint32_t timeout = HAL_GetTick();
-  while((HAL_GetTick() - timeout) < SD_TIMEOUT) {
-    if (sdio_check_card_state() == SD_TRANSFER_OK) {
-      return RES_OK;
-    }
+  if (sdio_check_card_state_with_timeout(SD_TIMEOUT) < 0) {
+    TRACE("SD getstatus timeout, s:%u c:%u", sector, (uint32_t)count);
+    return RES_ERROR;
   }
 
-  TRACE("SD getstatus timeout, s:%u c:%u", sector, (uint32_t)count);
-  return RES_ERROR;
+  return RES_OK;
 }
 
 static DRESULT _write_dma(const BYTE *buff, DWORD sector, UINT count)
@@ -322,16 +319,13 @@ static DRESULT sdio_write(BYTE lun, const BYTE *buff, DWORD sector, UINT count)
 
   res = _write_dma(buff, sector, count);
   if (res != RES_OK) return res;
-
-  uint32_t timeout = HAL_GetTick();
-  while((HAL_GetTick() - timeout) < SD_TIMEOUT) {
-    if (sdio_check_card_state() == SD_TRANSFER_OK) {
-      return RES_OK;
-    }
+
+  if (sdio_check_card_state_with_timeout(SD_TIMEOUT) < 0) {
+    TRACE("SD getstatus timeout, s:%u c: %u", sector, (uint32_t)count);
+    return RES_ERROR;
   }
 
-  TRACE("SD getstatus timeout, s:%u c: %u", sector, (uint32_t)count);
-  return RES_ERROR;
+  return RES_OK;
 }
 
 static DRESULT sdio_get_sector_count(DWORD* sectors)

radio/src/targets/common/arm/stm32/rotary_encoder_driver.cpp

@@ -34,106 +34,114 @@
   #include "opentx.h"
 #endif
 
+#if ROTARY_ENCODER_GRANULARITY == 2
+  #define ON_DETENT(p) ((p == 3) || (p == 0))
+#elif ROTARY_ENCODER_GRANULARITY == 4
+  #define ON_DETENT(p) (p == 3)
+#elif
+#error "Unknown ROTARY_ENCODER_GRANULARITY"
+#endif
+
 volatile rotenc_t rotencValue = 0;
 volatile uint32_t rotencDt = 0;
 
-#if defined(BOOT)
-#define INC_ROT        1
-#define INC_ROT_2      2
-#else
-#define INC_ROT \
-  (g_eeGeneral.rotEncMode == ROTARY_ENCODER_MODE_INVERT_BOTH ? -1 : 1);
-#define INC_ROT_2 \
-  (g_eeGeneral.rotEncMode == ROTARY_ENCODER_MODE_INVERT_BOTH ? -2 : 2);
-#endif
+// Last encoder pins state
+static uint8_t lastPins = 0;
+// Record encoder position change between detents
+int8_t reChgPos = 0;
+// Used on start to ignore movement until encoder position on detent
+bool skipUntilDetent = false;
 
 rotenc_t rotaryEncoderGetValue()
-{
-  return rotencValue / ROTARY_ENCODER_GRANULARITY;
-}
-
-rotenc_t rotaryEncoderGetRawValue()
 {
   return rotencValue;
 }
 
 void rotaryEncoderCheck()
 {
-  static uint8_t state = 0;
-  static uint8_t re_count = 0;
+  // Value increment for each state transition of the RE pins
+#if defined(ROTARY_ENCODER_INVERTED)
+  static int8_t reInc[4][4] = {
+    // Prev = 0
+    {  0, -1,  1, -2 },
+    // Prev = 1
+    {  1,  0,  0, -1 },
+    // Prev = 2
+    { -1,  0,  0,  1 },
+    // Prev = 3
+    {  2,  1, -1,  0 },
+  };
+#else
+  static int8_t reInc[4][4] = {
+    // Prev = 0
+    {  0,  1, -1,  2 },
+    // Prev = 1
+    { -1,  0,  0,  1 },
+    // Prev = 2
+    {  1,  0,  0, -1 },
+    // Prev = 3
+    { -2, -1,  1,  0 },
+  };
+#endif
+
   uint8_t pins = ROTARY_ENCODER_POSITION();
 
-#if defined(ROTARY_ENCODER_SUPPORT_BUGGY_WIRING)
-  if (pins != (state & 0x03) && !(readKeys() & (1 << KEY_ENTER))) {
-    if (re_count == 0) {
-      // Need at least 2 values to correctly determine initial direction
-      re_count = 1;
-    } else {
-      if ((pins ^ (state & 0x03)) == 0x03) {
-        if (pins == 3) {
-          rotencValue += INC_ROT_2;
-        } else {
-          rotencValue -= INC_ROT_2;
-        }
-      } else {
-        if ((state & 0x01) ^ ((pins & 0x02) >> 1)) {
-          rotencValue -= INC_ROT;
-        } else {
-          rotencValue += INC_ROT;
-        }
-      }
-
-      if (re_count == 1)
-      {
-        re_count = 2;
-        // Assume 1st value is same direction as 2nd value
-        rotencValue = rotencValue * 2;
-      }
+  // No change - do nothing
+  if (pins == lastPins) {
+    return;
+  }
+
+  // Handle case where radio started with encoder not on detent position
+  if (skipUntilDetent) {
+    if (ON_DETENT(pins)) {
+      lastPins = pins;
+      skipUntilDetent = false;
     }
-    state &= ~0x03;
-    state |= pins;
+    return;
   }
-#else
-  if (pins != state && !(readKeys() & (1 << KEY_ENTER))) {
-    if (re_count == 0) {
-      // Need at least 2 values to correctly determine initial direction
-      re_count = 1;
-    } else {
-#if defined(ROTARY_ENCODER_INVERTED)
-      if (!(state & 0x01) ^ ((pins & 0x02) >> 1)) {
-#else
-      if ((state & 0x01) ^ ((pins & 0x02) >> 1)) {
+
+  // Get increment value for pin state transition
+  int inc = reInc[lastPins][pins];
+
+#if !defined(BOOT)
+  if (g_eeGeneral.rotEncMode == ROTARY_ENCODER_MODE_INVERT_BOTH)
+    inc = -inc;
 #endif
-        rotencValue -= INC_ROT;
-      } else {
-        rotencValue += INC_ROT;
-      }
-
-      if (re_count == 1)
-      {
-        re_count = 2;
-        // Assume 1st value is same direction as 2nd value
-        rotencValue = rotencValue * 2;
-      }
+
+  // Update position change between detents
+  reChgPos += inc;
+
+  // Update reported value on full detent change
+  if (reChgPos >= ROTARY_ENCODER_GRANULARITY) {
+    // If ENTER pressed - ignore scrolling
+    if ((readKeys() & (1 << KEY_ENTER)) == 0) {
+      rotencValue += 1;
+    }
+    reChgPos -= ROTARY_ENCODER_GRANULARITY;
+  } else if (reChgPos <= -ROTARY_ENCODER_GRANULARITY) {
+    // If ENTER pressed - ignore scrolling
+    if ((readKeys() & (1 << KEY_ENTER)) == 0) {
+      rotencValue -= 1;
     }
-    state = pins;
+    reChgPos += ROTARY_ENCODER_GRANULARITY;
   }
-#endif
+
+  lastPins = pins;
 
 #if !defined(BOOT) && defined(COLORLCD)
   static uint32_t last_tick = 0;
   static rotenc_t last_value = 0;
 
   rotenc_t value = rotencValue;
-  rotenc_t diff = (value - last_value) / ROTARY_ENCODER_GRANULARITY;
+  rotenc_t diff = (value - last_value);
 
   if (diff != 0) {
     uint32_t now = RTOS_GET_MS();
     uint32_t dt = now - last_tick;
     // pre-compute accumulated dt (dx/dt is done later in LVGL driver)
     rotencDt += dt;
     last_tick = now;
-    last_value += diff * ROTARY_ENCODER_GRANULARITY;
+    last_value = value;
   }
 #endif
 }
@@ -171,6 +179,10 @@ void rotaryEncoderInit()
 
   NVIC_EnableIRQ(ROTARY_ENCODER_TIMER_IRQn);
   NVIC_SetPriority(ROTARY_ENCODER_TIMER_IRQn, 7);
+
+  // Get initial position
+  lastPins = ROTARY_ENCODER_POSITION();
+  skipUntilDetent = !ON_DETENT(lastPins);
 }
 
 extern "C" void ROTARY_ENCODER_TIMER_IRQHandler(void)

radio/src/targets/horus/hal.h

@@ -112,10 +112,6 @@
     #define ROTARY_ENCODER_INVERTED
 #endif
 
-#if defined(RADIO_FAMILY_T16) && !defined(RADIO_T18) && !defined(RADIO_T15)
-  #define ROTARY_ENCODER_SUPPORT_BUGGY_WIRING
-#endif
-
 // Switches
 #if defined(RADIO_T15)
   #define STORAGE_SWITCH_A

radio/src/targets/simu/simpgmspace.cpp

@@ -60,8 +60,6 @@ rotenc_t rotaryEncoderGetValue()
   return rotencValue / ROTARY_ENCODER_GRANULARITY;
 }
 
-rotenc_t rotaryEncoderGetRawValue() { return rotencValue; }
-
 // TODO: remove all STM32 defs
 
 extern const etx_hal_adc_driver_t simu_adc_driver;

radio/src/targets/taranis/hal.h

@@ -303,9 +303,6 @@
     #define USE_EXTI15_10_IRQ
     #define EXTI15_10_IRQ_Priority 5
   #endif
-  #if defined(RADIO_TX12)
-    #define ROTARY_ENCODER_SUPPORT_BUGGY_WIRING
-  #endif
   #if defined(RADIO_TX12MK2) || defined(RADIO_BOXER) || defined(RADIO_ZORRO) || defined(RADIO_MT12) || defined(RADIO_POCKET) || defined(RADIO_T14)
     #define ROTARY_ENCODER_INVERTED
   #endif

radio/src/thirdparty/FatFs/ffconf.h

@@ -234,7 +234,7 @@
 / System Configurations
 /---------------------------------------------------------------------------*/
 
-#define FF_FS_TINY		0
+#define FF_FS_TINY		1
 /* This option switches tiny buffer configuration. (0:Normal or 1:Tiny)
 /  At the tiny configuration, size of file object (FIL) is shrinked FF_MAX_SS bytes.
 /  Instead of private sector buffer eliminated from the file object, common sector