[Silabs] Fix linkage and some apps build for the Sparkfun board (#26370)

* Add missing sources files to the build. ifdef button usage * Restyled by whitespace --------- Co-authored-by: Restyled.io <commits@restyled.io>
project-chip · Jul 17, 2023 · 1271655 · 1271655
1 parent 58e4f85
commit 1271655
Show file tree

Hide file tree

Showing 15 changed files with 20 additions and 15 deletions.
diff --git a/examples/chef/efr32/include/AppTask.h b/examples/chef/efr32/include/AppTask.h
@@ -29,7 +29,6 @@
 #include "AppEvent.h"
 #include "BaseApplication.h"
 #include "FreeRTOS.h"
-#include "sl_simple_button_instances.h"
 #include "timers.h" // provides FreeRTOS timer support
 #include <app/clusters/identify-server/identify-server.h>
 #include <ble/BLEEndPoint.h>

diff --git a/examples/chef/efr32/src/AppTask.cpp b/examples/chef/efr32/src/AppTask.cpp
@@ -169,6 +169,7 @@ void AppTask::AppTaskMain(void * pvParameter)
     }
 }
 
+#ifdef SL_CATALOG_SIMPLE_BUTTON_PRESENT
 void AppTask::ButtonEventHandler(uint8_t button, uint8_t btnAction)
 {
     AppEvent button_event           = {};
@@ -181,3 +182,4 @@ void AppTask::ButtonEventHandler(uint8_t button, uint8_t btnAction)
         sAppTask.PostEvent(&button_event);
     }
 }
+#endif // SL_CATALOG_SIMPLE_BUTTON_PRESENT
diff --git a/examples/light-switch-app/silabs/efr32/include/AppTask.h b/examples/light-switch-app/silabs/efr32/include/AppTask.h
@@ -29,7 +29,6 @@
 #include "AppEvent.h"
 #include "BaseApplication.h"
 #include "FreeRTOS.h"
-#include "sl_simple_button_instances.h"
 #include "timers.h" // provides FreeRTOS timer support
 #include <app/clusters/identify-server/identify-server.h>
 #include <ble/BLEEndPoint.h>

diff --git a/examples/light-switch-app/silabs/efr32/src/AppTask.cpp b/examples/light-switch-app/silabs/efr32/src/AppTask.cpp
@@ -229,6 +229,7 @@ void AppTask::OnIdentifyStop(Identify * identify)
 #endif
 }
 
+#ifdef SL_CATALOG_SIMPLE_BUTTON_PRESENT
 void AppTask::SwitchActionEventHandler(AppEvent * aEvent)
 {
     VerifyOrReturn(aEvent->Type == AppEvent::kEventType_Button);
@@ -271,3 +272,4 @@ void AppTask::ButtonEventHandler(uint8_t button, uint8_t btnAction)
         sAppTask.PostEvent(&button_event);
     }
 }
+#endif // SL_CATALOG_SIMPLE_BUTTON_PRESENT
diff --git a/examples/lighting-app/silabs/efr32/include/AppTask.h b/examples/lighting-app/silabs/efr32/include/AppTask.h
@@ -30,9 +30,6 @@
 #include "BaseApplication.h"
 #include "FreeRTOS.h"
 #include "LightingManager.h"
-#ifdef SL_CATALOG_SIMPLE_BUTTON_PRESENT
-#include "sl_simple_button_instances.h"
-#endif
 #include "timers.h" // provides FreeRTOS timer support
 #include <app/clusters/identify-server/identify-server.h>
 #include <ble/BLEEndPoint.h>

diff --git a/examples/lighting-app/silabs/efr32/src/AppTask.cpp b/examples/lighting-app/silabs/efr32/src/AppTask.cpp
@@ -281,7 +281,6 @@ void AppTask::ButtonEventHandler(uint8_t button, uint8_t btnAction)
         AppTask::GetAppTask().PostEvent(&button_event);
     }
 }
-
 #endif
 
 void AppTask::ActionInitiated(LightingManager::Action_t aAction, int32_t aActor)

diff --git a/examples/lock-app/silabs/efr32/include/AppTask.h b/examples/lock-app/silabs/efr32/include/AppTask.h
@@ -30,7 +30,6 @@
 #include "BaseApplication.h"
 #include "FreeRTOS.h"
 #include "LockManager.h"
-#include "sl_simple_button_instances.h"
 #include "timers.h" // provides FreeRTOS timer support
 #include <app/clusters/identify-server/identify-server.h>
 #include <ble/BLEEndPoint.h>

diff --git a/examples/lock-app/silabs/efr32/src/AppTask.cpp b/examples/lock-app/silabs/efr32/src/AppTask.cpp
@@ -143,7 +143,9 @@ CHIP_ERROR AppTask::Init()
 {
     CHIP_ERROR err = CHIP_NO_ERROR;
 
+#ifdef SL_CATALOG_SIMPLE_BUTTON_PRESENT
     chip::DeviceLayer::Silabs::GetPlatform().SetButtonsCb(AppTask::ButtonEventHandler);
+#endif // SL_CATALOG_SIMPLE_BUTTON_PRESENT
 
 #ifdef DISPLAY_ENABLED
     GetLCD().Init((uint8_t *) "Lock-App", true);
@@ -359,6 +361,7 @@ void AppTask::LockActionEventHandler(AppEvent * aEvent)
     }
 }
 
+#ifdef SL_CATALOG_SIMPLE_BUTTON_PRESENT
 void AppTask::ButtonEventHandler(uint8_t button, uint8_t btnAction)
 {
     AppEvent button_event           = {};
@@ -376,6 +379,7 @@ void AppTask::ButtonEventHandler(uint8_t button, uint8_t btnAction)
         sAppTask.PostEvent(&button_event);
     }
 }
+#endif // SL_CATALOG_SIMPLE_BUTTON_PRESENT
 
 void AppTask::ActionInitiated(LockManager::Action_t aAction, int32_t aActor)
 {

diff --git a/examples/thermostat/silabs/efr32/BUILD.gn b/examples/thermostat/silabs/efr32/BUILD.gn
@@ -46,7 +46,8 @@ declare_args() {
 
   # Enable the temperature sensor
   # Some boards do not have a temperature sensor
-  use_temp_sensor = silabs_board != "BRD2703A" && silabs_board != "BRD4319A"
+  use_temp_sensor = silabs_board != "BRD2703A" && silabs_board != "BRD4319A" &&
+                    silabs_board != "BRD2704A"
 }
 
 efr32_sdk("sdk") {

diff --git a/examples/thermostat/silabs/efr32/include/AppTask.h b/examples/thermostat/silabs/efr32/include/AppTask.h
@@ -35,7 +35,6 @@
 #include "FreeRTOS.h"
 #include "SensorManager.h"
 #include "TemperatureManager.h"
-#include "sl_simple_button_instances.h"
 #include "timers.h" // provides FreeRTOS timer support
 #include <app/clusters/identify-server/identify-server.h>
 #include <ble/BLEEndPoint.h>

diff --git a/examples/thermostat/silabs/efr32/src/AppTask.cpp b/examples/thermostat/silabs/efr32/src/AppTask.cpp
@@ -142,8 +142,9 @@ AppTask AppTask::sAppTask;
 CHIP_ERROR AppTask::Init()
 {
     CHIP_ERROR err = CHIP_NO_ERROR;
-
+#ifdef SL_CATALOG_SIMPLE_BUTTON_PRESENT
     chip::DeviceLayer::Silabs::GetPlatform().SetButtonsCb(AppTask::ButtonEventHandler);
+#endif // SL_CATALOG_SIMPLE_BUTTON_PRESENT
 
 #ifdef DISPLAY_ENABLED
     GetLCD().Init((uint8_t *) "Thermostat-App");
@@ -245,6 +246,7 @@ void AppTask::UpdateThermoStatUI()
 #endif // DISPLAY_ENABLED
 }
 
+#ifdef SL_CATALOG_SIMPLE_BUTTON_PRESENT
 void AppTask::ButtonEventHandler(uint8_t button, uint8_t btnAction)
 {
     AppEvent aEvent           = {};
@@ -257,3 +259,4 @@ void AppTask::ButtonEventHandler(uint8_t button, uint8_t btnAction)
         sAppTask.PostEvent(&aEvent);
     }
 }
+#endif // SL_CATALOG_SIMPLE_BUTTON_PRESENT
diff --git a/examples/window-app/silabs/efr32/include/WindowAppImpl.h b/examples/window-app/silabs/efr32/include/WindowAppImpl.h
@@ -24,7 +24,6 @@
 #include <WindowApp.h>
 #include <queue.h>
 #include <setup_payload/QRCodeSetupPayloadGenerator.h>
-#include <sl_simple_button_instances.h>
 #include <string>
 #include <task.h>
 #include <timers.h>

diff --git a/examples/window-app/silabs/efr32/src/WindowAppImpl.cpp b/examples/window-app/silabs/efr32/src/WindowAppImpl.cpp
@@ -31,7 +31,9 @@
 #include <setup_payload/QRCodeSetupPayloadGenerator.h>
 #include <setup_payload/SetupPayload.h>
 #endif // QR_CODE_ENABLED
+#ifdef SL_CATALOG_SIMPLE_BUTTON_PRESENT
 #include <sl_simple_button_instances.h>
+#endif
 
 #include <sl_system_kernel.h>
 
@@ -516,8 +518,8 @@ WindowAppImpl::Button::Button(WindowApp::Button::Id id, const char * name) : Win
 
 void WindowAppImpl::OnButtonChange(uint8_t button, uint8_t btnAction)
 {
+#ifdef SL_CATALOG_SIMPLE_BUTTON_PRESENT
     WindowApp::Button * btn = static_cast<Button *>((button == 0) ? sInstance.mButtonUp : sInstance.mButtonDown);
-
     if (btnAction == SL_SIMPLE_BUTTON_PRESSED)
     {
         btn->Press();
@@ -526,6 +528,7 @@ void WindowAppImpl::OnButtonChange(uint8_t button, uint8_t btnAction)
     {
         btn->Release();
     }
+#endif
 }
 
 // Silabs button callback from button event ISR

diff --git a/examples/window-app/silabs/efr32/src/main.cpp b/examples/window-app/silabs/efr32/src/main.cpp
@@ -21,7 +21,6 @@
 #include <WindowApp.h>
 
 #include "init_efrPlatform.h"
-#include "sl_simple_button_instances.h"
 #include "sl_system_kernel.h"
 #include <DeviceInfoProviderImpl.h>
 #include <app/server/Server.h>

diff --git a/third_party/silabs/efr32_sdk.gni b/third_party/silabs/efr32_sdk.gni
@@ -179,7 +179,7 @@ template("efr32_sdk") {
       "${silabs_gen_folder}/autogen",
     ]
 
-    if (slc_generate || slc_reuse_files) {
+    if (slc_generate || slc_reuse_files || silabs_family == "mgm24") {
       _include_dirs +=
           [ "${efr32_sdk_root}/hardware/driver/configuration_over_swo/inc/" ]
     }
@@ -721,7 +721,7 @@ template("efr32_sdk") {
       ]
     }
 
-    if (slc_generate || slc_reuse_files) {
+    if (slc_generate || slc_reuse_files || silabs_family == "mgm24") {
       sources += [
         "${efr32_sdk_root}/hardware/driver/configuration_over_swo/src/sl_cos.c",
         "${efr32_sdk_root}/platform/driver/debug/src/sl_debug_swo.c",
-Original file line number
+Diff line change
@@ Expand Up @@
             AppTask::GetAppTask().PostEvent(&button_event);
         }
     }
     #endif
     void AppTask::ActionInitiated(LightingManager::Action_t aAction, int32_t aActor)
@@ Expand Down @@