add activeModeThreshold check for LIT ICDs

mkardous-silabs · Jan 22, 2024 · 92e2135 · 92e2135
1 parent c961d1b
commit 92e2135
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 5 deletions.
diff --git a/examples/lit-icd-app/linux/include/CHIPProjectAppConfig.h b/examples/lit-icd-app/linux/include/CHIPProjectAppConfig.h
@@ -39,4 +39,4 @@
 // ICD configurations
 #define CHIP_CONFIG_ICD_IDLE_MODE_DURATION_SEC 3600
 #define CHIP_CONFIG_ICD_ACTIVE_MODE_DURATION_MS 10000
-#define CHIP_CONFIG_ICD_ACTIVE_MODE_THRESHOLD_MS 1000
+#define CHIP_CONFIG_ICD_ACTIVE_MODE_THRESHOLD_MS 5000
diff --git a/examples/lit-icd-app/silabs/openthread.gni b/examples/lit-icd-app/silabs/openthread.gni
@@ -40,4 +40,4 @@ sl_ot_active_interval_ms = 1000  # 1000ms Active Polling Interval
 # ICD Matter Configuration flags
 sl_idle_mode_interval_s = 3600  # 60min Idle Mode Interval
 sl_active_mode_interval_ms = 0  # 0 Active Mode Interval
-sl_active_mode_threshold_ms = 30000  # 30s Active Mode Threshold
+sl_active_mode_threshold_ms = 5000  # 5s Active Mode Threshold
diff --git a/src/app/icd/ICDConfigurationData.h b/src/app/icd/ICDConfigurationData.h
@@ -61,6 +61,8 @@ class ICDConfigurationData
 
     System::Clock::Milliseconds32 GetFastPollingInterval() { return mFastPollingInterval; }
 
+    uint32_t GetMinLitActiveModeThresholdMs() { return kMinLitActiveModeThreshold_ms; }
+
     /**
      * If ICD_ENFORCE_SIT_SLOW_POLL_LIMIT is set to 0, function will always return the configured Slow Polling interval
      * (CHIP_DEVICE_CONFIG_ICD_SLOW_POLL_INTERVAL).
@@ -93,6 +95,8 @@ class ICDConfigurationData
     void SetSlowPollingInterval(System::Clock::Milliseconds32 slowPollInterval) { mSlowPollingInterval = slowPollInterval; };
     void SetFastPollingInterval(System::Clock::Milliseconds32 fastPollInterval) { mFastPollingInterval = fastPollInterval; };
 
+    static constexpr uint32_t kMinLitActiveModeThreshold_ms = 5000;
+
     static_assert((CHIP_CONFIG_ICD_IDLE_MODE_DURATION_SEC) <= 64800,
                   "Spec requires the IdleModeDuration to be equal or inferior to 64800s.");
     static_assert((CHIP_CONFIG_ICD_IDLE_MODE_DURATION_SEC) >= 1,

diff --git a/src/app/icd/ICDManager.cpp b/src/app/icd/ICDManager.cpp
@@ -50,9 +50,14 @@ void ICDManager::Init(PersistentStorageDelegate * storage, FabricTable * fabricT
 
     bool supportLIT = SupportsFeature(Feature::kLongIdleTimeSupport);
     VerifyOrDieWithMsg((supportLIT == false) || SupportsFeature(Feature::kCheckInProtocolSupport), AppServer,
-                       "The CheckIn protocol feature is required for LIT support");
+                       "The CheckIn protocol feature is required for LIT support.");
     VerifyOrDieWithMsg((supportLIT == false) || SupportsFeature(Feature::kUserActiveModeTrigger), AppServer,
-                       "The user ActiveMode trigger feature is required for LIT support");
+                       "The user ActiveMode trigger feature is required for LIT support.");
+
+    VerifyOrDieWithMsg((supportLIT == false) ||
+                           ICDConfigurationData::GetInstance().GetMinLitActiveModeThresholdMs() <=
+                               ICDConfigurationData::GetInstance().GetActiveModeThresholdMs(),
+                       AppServer, "The minimum ActiveModeThreshold value for a LIT ICD is 5 seconds.");
 
     // Disabling check until LIT support is compelte
     // VerifyOrDieWithMsg((supportLIT == false) && (GetSlowPollingInterval() <= GetSITPollingThreshold()) , AppServer,

diff --git a/src/app/tests/suites/TestIcdManagementCluster.yaml b/src/app/tests/suites/TestIcdManagementCluster.yaml
@@ -58,7 +58,7 @@ tests:
       command: "readAttribute"
       attribute: "ActiveModeThreshold"
       response:
-          value: 1000
+          value: 5000
 
     - label: "Read ICDCounter"
       command: "readAttribute"