Hoping for XR1 Module. Generic Tuya Window/Door Sensor with CB3S

commit b39ae29e8faa59b1ad604750a67c534f049a44f4
Author: NonPIayerCharacter <18557343+NonPIayerCharacter@users.noreply.github.com>
Date:   Fri May 31 14:47:19 2024 +0100

    --

diff --git a/src/driver/drv_battery.c b/src/driver/drv_battery.c
index 0cfb93d9cbbdbd377917a744350002ad0f37a358..7081b3bc23b7b20ecc8241a27542ee82a5a1333d 100644
--- a/src/driver/drv_battery.c
+++ b/src/driver/drv_battery.c
@@ -12,21 +12,31 @@
 #include "../hal/hal_adc.h"
 #include "drv_battery.h"
 
-static int g_pin_adc = 0, channel_adc = 0, channel_rel = 0, g_pin_rel = 0, g_battcycle = 1, g_battcycleref = 10;
+static int g_pin_adc = 0, channel_adc = 0, channel_rel = 0, g_pin_rel = 0, g_battcycle = 1, g_battcycleref = 10, g_oversamp = 1, g_sampdelay = 100;
 static float g_battvoltage = 0.0, g_battlevel = 0.0;
 static int g_lastbattvoltage = 0, g_lastbattlevel = 0;
 static float g_vref = 2400, g_vdivider = 2.29, g_maxbatt = 3000, g_minbatt = 2000, g_adcbits = 4096;
+static bool g_isoversamp = false;
+
+int cmp_adc(const void* a, const void* b)
+{
+   int arg1 = *(const int*)a;
+   int arg2 = *(const int*)b;
+
+   return (arg1 > arg2) - (arg1 < arg2);
+}
 
 static void Batt_Measure() {
    //this command has only been tested on CBU
    float batt_ref, batt_res, vref;
    ADDLOG_INFO(LOG_FEATURE_DRV, "DRV_BATTERY : Measure Battery volt en perc");
    g_pin_adc = PIN_FindPinIndexForRole(IOR_BAT_ADC, g_pin_adc);
+   bool hasRelay = true;
    if (PIN_FindPinIndexForRole(IOR_BAT_Relay, -1) == -1 && PIN_FindPinIndexForRole(IOR_BAT_Relay_n, -1) == -1) {
-      g_vdivider = 1;
+      hasRelay = false;
    }
    // if divider equal to 1 then no need for relay activation
-   if (g_vdivider > 1) {
+   if (g_vdivider > 1 && hasRelay) {
       g_pin_rel = PIN_FindPinIndexForRole(IOR_BAT_Relay, -1);
       if (g_pin_rel == -1) {
          g_pin_rel = PIN_FindPinIndexForRole(IOR_BAT_Relay_n, -1);
@@ -38,13 +48,32 @@ static void Batt_Measure() {
    if (g_battlevel < 1024) {
       ADDLOG_INFO(LOG_FEATURE_DRV, "DRV_BATTERY : ADC Value low device not on battery");
    }
-   if (g_vdivider > 1) {
+   if(g_vdivider > 1 && hasRelay)
+   {
       CHANNEL_Set(channel_rel, 1, 0);
       rtos_delay_milliseconds(10);
    }
-   g_battvoltage = HAL_ADC_Read(g_pin_adc);
+   if(g_isoversamp)
+   {
+      g_battvoltage = 0;
+      int oversamp = g_oversamp > 0 ? g_oversamp : 2;
+      int vals[oversamp];
+      for(int i = 0; i < oversamp; i++)
+      {
+         vals[i] = HAL_ADC_Read(g_pin_adc);
+         ADDLOG_EXTRADEBUG(LOG_FEATURE_DRV, "DRV_BATTERY: adc %i", vals[i]);
+         rtos_delay_milliseconds(g_sampdelay);
+      }
+      qsort(vals, oversamp, sizeof(int), cmp_adc);
+      g_battvoltage = vals[oversamp / 2];
+   }
+   else
+   {
+      g_battvoltage = HAL_ADC_Read(g_pin_adc);
+   }
    ADDLOG_DEBUG(LOG_FEATURE_DRV, "DRV_BATTERY : ADC binary Measurement : %f and channel %i", g_battvoltage, channel_adc);
-   if (g_vdivider > 1) {
+   if (g_vdivider > 1 && hasRelay)
+   {
       CHANNEL_Set(channel_rel, 0, 0);
    }
    ADDLOG_DEBUG(LOG_FEATURE_DRV, "DRV_BATTERY : Calculation with param : %f %f %f", g_vref, g_adcbits, g_vdivider);
@@ -123,13 +152,37 @@ commandResult_t Battery_cycle(const void* context, const char* cmd, const char*
    {
       return CMD_RES_NOT_ENOUGH_ARGUMENTS;
    }
-   g_battcycleref = Tokenizer_GetArgFloat(0);
+   g_battcycleref = Tokenizer_GetArgInteger(0);
 
    ADDLOG_INFO(LOG_FEATURE_CMD, "Battery Cycle : Measurement will run every %i seconds", g_battcycleref);
 
    return CMD_RES_OK;
 }
 
+commandResult_t Battery_Sampling(const void* context, const char* cmd, const char* args, int cmdFlags)
+{
+   Tokenizer_TokenizeString(args, TOKENIZER_ALLOW_QUOTES | TOKENIZER_DONT_EXPAND);
+   if(Tokenizer_CheckArgsCountAndPrintWarning(cmd, 2))
+   {
+      return CMD_RES_NOT_ENOUGH_ARGUMENTS;
+   }
+   g_oversamp = Tokenizer_GetArgInteger(0);
+   g_sampdelay = Tokenizer_GetArgInteger(1);
+   g_sampdelay = g_sampdelay < 10 ? 10 : g_sampdelay;
+
+   if((g_oversamp * g_sampdelay + 100) > g_battcycleref * 1000)
+   {
+      ADDLOG_WARN(LOG_FEATURE_CMD, "Battery_Sampling: New cycle will begin before old one is completed, disabling oversampling!");
+      g_isoversamp = false;
+   }
+   else
+   {
+      g_isoversamp = true;
+      ADDLOG_INFO(LOG_FEATURE_CMD, "Battery_Sampling: Oversampling %i times with delay %ims between sampling", g_oversamp, g_sampdelay);
+   }
+
+   return CMD_RES_OK;
+}
 
 // startDriver Battery
 void Batt_Init() {
@@ -146,6 +199,11 @@ void Batt_Init() {
    //cmddetail:"examples":"Battery_cycle 60"}
    CMD_RegisterCommand("Battery_cycle", Battery_cycle, NULL);
 
+   //cmddetail:{"name":"Battery_Sampling","args":"[samples][delay]",
+   //cmddetail:"descr":"Oversample readings default is off. args: samples, delay between cycles is ms",
+   //cmddetail:"fn":"Battery_Sampling","file":"drv/drv_battery.c","requires":"",
+   //cmddetail:"examples":"Battery_Sampling 10 100"}
+   CMD_RegisterCommand("Battery_Sampling", Battery_Sampling, NULL);
 }
 
 void Batt_OnEverySecond() {