[Silabs] WiFi factory reset support for window-app 917SoC (#26066)

* adds factory reset support for 917SoC window-app * applied restyle
project-chip · Jan 15, 2024 · 1160256 · 1160256
1 parent e0bef62
commit 1160256
Show file tree

Hide file tree

Showing 6 changed files with 28 additions and 8 deletions.
diff --git a/examples/light-switch-app/silabs/SiWx917/src/AppTask.cpp b/examples/light-switch-app/silabs/SiWx917/src/AppTask.cpp
@@ -265,7 +265,7 @@ void AppTask::ButtonEventHandler(uint8_t button, uint8_t btnAction)
         button_event.Handler = SwitchActionEventHandler;
         sAppTask.PostEvent(&button_event);
     }
-    else if (button == SIWx917_BTN0)
+    else if (button == SIWx917_BTN0 && btnAction == SL_SIMPLE_BUTTON_PRESSED)
     {
         button_event.Handler = BaseApplication::ButtonHandler;
         sAppTask.PostEvent(&button_event);

diff --git a/examples/lighting-app/silabs/SiWx917/src/AppTask.cpp b/examples/lighting-app/silabs/SiWx917/src/AppTask.cpp
@@ -246,7 +246,7 @@ void AppTask::ButtonEventHandler(uint8_t button, uint8_t btnAction)
         button_event.Handler = LightActionEventHandler;
         sAppTask.PostEvent(&button_event);
     }
-    else if (button == SIWx917_BTN0)
+    else if (button == SIWx917_BTN0 && btnAction == SL_SIMPLE_BUTTON_PRESSED)
     {
         button_event.Handler = BaseApplication::ButtonHandler;
         sAppTask.PostEvent(&button_event);

diff --git a/examples/lock-app/silabs/SiWx917/src/AppTask.cpp b/examples/lock-app/silabs/SiWx917/src/AppTask.cpp
@@ -358,7 +358,7 @@ void AppTask::ButtonEventHandler(uint8_t button, uint8_t btnAction)
         button_event.Handler = LockActionEventHandler;
         sAppTask.PostEvent(&button_event);
     }
-    else if (button == SIWx917_BTN0)
+    else if (button == SIWx917_BTN0 && btnAction == SL_SIMPLE_BUTTON_PRESSED)
     {
         button_event.Handler = BaseApplication::ButtonHandler;
         sAppTask.PostEvent(&button_event);

diff --git a/examples/platform/silabs/SiWx917/SiWx917/hal/rsi_hal_mcu_m4.c b/examples/platform/silabs/SiWx917/SiWx917/hal/rsi_hal_mcu_m4.c
@@ -92,9 +92,10 @@ void IRQ021_Handler(void)
         btn1 = 0;
         sl_button_on_change(1, 1);
     }
-    if (RSI_NPSSGPIO_GetPin(NPSS_GPIO_0))
+    if (RSI_NPSSGPIO_GetPin(NPSS_GPIO_0) && (!btn0))
     {
         btn0 = 1;
+        sl_button_on_change(0, 0);
     }
     if (!RSI_NPSSGPIO_GetPin(NPSS_GPIO_0) && btn0)
     {

diff --git a/examples/window-app/silabs/SiWx917/include/WindowAppImpl.h b/examples/window-app/silabs/SiWx917/include/WindowAppImpl.h
@@ -44,6 +44,7 @@ class WindowAppImpl : public WindowApp
 {
 public:
     static WindowAppImpl sInstance;
+    bool mWindowAppInit = false;
 
     WindowAppImpl();
     CHIP_ERROR Init() override;

diff --git a/examples/window-app/silabs/SiWx917/src/WindowAppImpl.cpp b/examples/window-app/silabs/SiWx917/src/WindowAppImpl.cpp
@@ -227,6 +227,8 @@ CHIP_ERROR WindowAppImpl::Init()
     }
 #endif // QR_CODE_ENABLED
 
+    mWindowAppInit = true;
+
     return CHIP_NO_ERROR;
 }
 
@@ -531,14 +533,30 @@ WindowAppImpl::Button::Button(WindowApp::Button::Id id, const char * name) : Win
 void WindowAppImpl::OnButtonChange(uint8_t Btn, uint8_t btnAction)
 {
     WindowApp::Button * btn = static_cast<Button *>((Btn == SIWx917_BTN0) ? sInstance.mButtonUp : sInstance.mButtonDown);
-    btn->Press();
-    // since sl_button_on_change is being called only with button press, calling Release() without condition
-    btn->Release();
+    if (Btn == SIWx917_BTN1)
+    {
+        btn->Press();
+        btn->Release();
+    }
+    else
+    {
+        if (btnAction)
+        {
+            btn->Press();
+        }
+        else
+        {
+            btn->Release();
+        }
+    }
 }
 
 // Silabs button callback from button event ISR
 void sl_button_on_change(uint8_t btn, uint8_t btnAction)
 {
     WindowAppImpl * app = static_cast<WindowAppImpl *>(&WindowAppImpl::sInstance);
-    app->OnButtonChange(btn, btnAction);
+    if (app->mWindowAppInit)
+    {
+        app->OnButtonChange(btn, btnAction);
+    }
 }