Revert "[esp32]: Do not setup DeviceInfoProvider within PlatformMgr (#…

…21039)" (#21097) This reverts commit e15af6f.
project-chip · Jul 22, 2022 · c3e71f2 · c3e71f2
1 parent ce075ee
commit c3e71f2
Show file tree

Hide file tree

Showing 12 changed files with 492 additions and 33 deletions.
diff --git a/examples/all-clusters-app/esp32/main/AppTask.cpp b/examples/all-clusters-app/esp32/main/AppTask.cpp
@@ -26,7 +26,6 @@
 #include "esp_log.h"
 #include "esp_spi_flash.h"
 #include "freertos/FreeRTOS.h"
-#include <DeviceInfoProviderImpl.h>
 #include <app/server/OnboardingCodesUtil.h>
 
 #define APP_TASK_NAME "APP"
@@ -40,8 +39,6 @@ namespace {
 QueueHandle_t sAppEventQueue;
 TaskHandle_t sAppTaskHandle;
 
-chip::DeviceLayer::DeviceInfoProviderImpl gExampleDeviceInfoProvider;
-
 } // namespace
 
 AppTask AppTask::sAppTask;
@@ -87,9 +84,6 @@ CHIP_ERROR AppTask::Init()
 #if CONFIG_HAVE_DISPLAY
  InitDeviceDisplay();
 #endif
-
- chip::DeviceLayer::SetDeviceInfoProvider(&gExampleDeviceInfoProvider);
-
  return err;
 }
 

diff --git a/examples/all-clusters-app/esp32/main/CMakeLists.txt b/examples/all-clusters-app/esp32/main/CMakeLists.txt
@@ -21,15 +21,13 @@ set(PRIV_INCLUDE_DIRS_LIST
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/all-clusters-app"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/all-clusters-app/all-clusters-common/include"
  "${CMAKE_CURRENT_LIST_DIR}/include"
- "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/providers" 
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32"
 )
 set(SRC_DIRS_LIST
  "${CMAKE_CURRENT_LIST_DIR}"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/all-clusters-app/zap-generated"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/app-common/app-common/zap-generated/attributes"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/app-common/app-common/zap-generated"
- "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/providers" 
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/route_hook"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/ota"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/common"

diff --git a/examples/light-switch-app/esp32/main/AppTask.cpp b/examples/light-switch-app/esp32/main/AppTask.cpp
@@ -20,7 +20,6 @@
 #include "BindingHandler.h"
 #include "esp_log.h"
 #include "freertos/FreeRTOS.h"
-#include <DeviceInfoProviderImpl.h>
 
 #define APP_TASK_NAME "APP"
 #define APP_EVENT_QUEUE_SIZE 10
@@ -38,7 +37,6 @@ namespace {
 
 QueueHandle_t sAppEventQueue;
 TaskHandle_t sAppTaskHandle;
-chip::DeviceLayer::DeviceInfoProviderImpl gExampleDeviceInfoProvider;
 
 } // namespace
 
@@ -64,8 +62,8 @@ CHIP_ERROR AppTask::Init()
  CHIP_ERROR err = CHIP_NO_ERROR;
 
  AppButton.Init();
+
  AppButton.SetButtonPressCallback(ButtonPressCallback);
- chip::DeviceLayer::SetDeviceInfoProvider(&gExampleDeviceInfoProvider);
 
  return err;
 }

diff --git a/examples/light-switch-app/esp32/main/CMakeLists.txt b/examples/light-switch-app/esp32/main/CMakeLists.txt
@@ -19,14 +19,12 @@
 idf_component_register(PRIV_INCLUDE_DIRS
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/light-switch-app"
  "${CMAKE_CURRENT_LIST_DIR}/include"
- "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/providers" 
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32"
  SRC_DIRS
  "${CMAKE_CURRENT_LIST_DIR}"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/app-common/app-common/zap-generated/attributes"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/app-common/app-common/zap-generated"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/light-switch-app/zap-generated"
- "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/providers" 
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/route_hook"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/ota"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/common"

diff --git a/examples/lighting-app/esp32/main/AppTask.cpp b/examples/lighting-app/esp32/main/AppTask.cpp
@@ -20,7 +20,6 @@
 #include "esp_log.h"
 #include "freertos/FreeRTOS.h"
 
-#include <DeviceInfoProviderImpl.h>
 #include <app-common/zap-generated/attribute-id.h>
 #include <app-common/zap-generated/attribute-type.h>
 #include <app-common/zap-generated/attributes/Accessors.h>
@@ -47,8 +46,6 @@ constexpr EndpointId kLightEndpointId = 1;
 QueueHandle_t sAppEventQueue;
 TaskHandle_t sAppTaskHandle;
 
-chip::DeviceLayer::DeviceInfoProviderImpl gExampleDeviceInfoProvider;
-
 } // namespace
 
 AppTask AppTask::sAppTask;
@@ -77,8 +74,6 @@ CHIP_ERROR AppTask::Init()
 
  AppButton.SetButtonPressCallback(ButtonPressCallback);
 
- chip::DeviceLayer::SetDeviceInfoProvider(&gExampleDeviceInfoProvider);
-
  return err;
 }
 

diff --git a/examples/lighting-app/esp32/main/CMakeLists.txt b/examples/lighting-app/esp32/main/CMakeLists.txt
@@ -18,7 +18,6 @@
 
 idf_component_register(PRIV_INCLUDE_DIRS
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/lighting-app"
- "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/providers"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/lighting-app/lighting-common/include"
  "${CMAKE_CURRENT_LIST_DIR}/include"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32"
@@ -27,12 +26,11 @@ idf_component_register(PRIV_INCLUDE_DIRS
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/app-common/app-common/zap-generated/attributes"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/app-common/app-common/zap-generated"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/lighting-app/zap-generated"
- "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/providers"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/lighting-app/lighting-common/src"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/route_hook"
-  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/ota"
+  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/ota"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/common"
-  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/shell_extension"
+  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/shell_extension"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/src/app/server"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/src/app/util"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/src/app/reporting"

diff --git a/examples/lock-app/esp32/main/AppTask.cpp b/examples/lock-app/esp32/main/AppTask.cpp
@@ -21,7 +21,6 @@
 #include "Button.h"
 #include "LEDWidget.h"
 #include "esp_log.h"
-#include <DeviceInfoProviderImpl.h>
 #include <app-common/zap-generated/attribute-id.h>
 #include <app-common/zap-generated/attribute-type.h>
 #include <app-common/zap-generated/cluster-id.h>
@@ -57,8 +56,6 @@ QueueHandle_t sAppEventQueue;
 bool sHaveBLEConnections = false;
 
 StackType_t appStack[APP_TASK_STACK_SIZE / sizeof(StackType_t)];
-
-chip::DeviceLayer::DeviceInfoProviderImpl gExampleDeviceInfoProvider;
 } // namespace
 
 using namespace ::chip::DeviceLayer;
@@ -106,8 +103,6 @@ CHIP_ERROR AppTask::Init()
 
  sLockLED.Set(!BoltLockMgr().IsUnlocked());
 
- chip::DeviceLayer::SetDeviceInfoProvider(&gExampleDeviceInfoProvider);
-
  chip::DeviceLayer::SystemLayer().ScheduleWork(UpdateClusterState, nullptr);
 
  ConfigurationMgr().LogDeviceConfig();

diff --git a/examples/lock-app/esp32/main/CMakeLists.txt b/examples/lock-app/esp32/main/CMakeLists.txt
@@ -21,7 +21,6 @@ idf_component_register(INCLUDE_DIRS
  "${CMAKE_CURRENT_LIST_DIR}"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/pw_sys_io/public"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32"
- "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/providers"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/common"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/common/pigweed"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/common/pigweed/esp32"
@@ -31,10 +30,9 @@ idf_component_register(INCLUDE_DIRS
  "${IDF_PATH}/components/freertos/include/freertos"
  SRC_DIRS
  "${CMAKE_CURRENT_LIST_DIR}"
- "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/providers"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/common"
-  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/common/pigweed"
+  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/common/pigweed"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/common/pigweed/esp32"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/app-common/app-common/zap-generated/attributes"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/app-common/app-common/zap-generated"
@@ -143,17 +141,15 @@ target_compile_options(${COMPONENT_LIB} PRIVATE
 else (CONFIG_ENABLE_PW_RPC)
 idf_component_register(PRIV_INCLUDE_DIRS
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/lock-app/"
- "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/providers" 
  "${CMAKE_CURRENT_LIST_DIR}/include"
  SRC_DIRS
  "${CMAKE_CURRENT_LIST_DIR}"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/app-common/app-common/zap-generated/attributes"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/app-common/app-common/zap-generated"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/zzz_generated/lock-app/zap-generated"
- "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/providers" 
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/route_hook"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/shell_extension"
-  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/common"
+  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/examples/platform/esp32/common"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/src/app/server"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/src/app/util"
  "${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/src/app/reporting"

diff --git a/src/platform/ESP32/BUILD.gn b/src/platform/ESP32/BUILD.gn
@@ -34,6 +34,8 @@ static_library("ESP32") {
  "ConfigurationManagerImpl.h",
  "ConnectivityManagerImpl.cpp",
  "ConnectivityManagerImpl.h",
+ "DeviceInfoProviderImpl.cpp",
+ "DeviceInfoProviderImpl.h",
  "DeviceNetworkProvisioningDelegateImpl.cpp",
  "DeviceNetworkProvisioningDelegateImpl.h",
  "DiagnosticDataProviderImpl.cpp",