[Ameba] Fix commissioning and hardfault issue (#16757)

* Update chipinterface to fix hardfault error * Fix ble-wifi commissioning issue, disable ipv4, reduce max fabric to 5 * Control rotating_id enable/disable from chip.cmake instead of chip_main.cmake * Restyled by whitespace * Restyled by clang-format Co-authored-by: Restyled.io <commits@restyled.io>
project-chip · Apr 7, 2022 · 3516440 · 3516440
1 parent 9c574de
commit 3516440
Show file tree

Hide file tree

Showing 11 changed files with 65 additions and 45 deletions.
diff --git a/config/ameba/chip.cmake b/config/ameba/chip.cmake
@@ -8,6 +8,8 @@ set(chip_c_flags "")
 set(chip_cpp_flags "")
 set(chip-gn chip-gn)
 
+set(matter_enable_rotating_id 1)
+
 get_filename_component(CHIP_ROOT ${chip_dir} REALPATH)
 get_filename_component(CHIP_OUTPUT ${chip_dir_output} REALPATH)
 get_filename_component(LIB_ROOT ${prj_root}/GCC-RELEASE/project_hp/asdk/lib/application REALPATH)
@@ -101,6 +103,7 @@ string(APPEND CHIP_GN_ARGS "ameba_ar = \"arm-none-eabi-ar\"\n")
 string(APPEND CHIP_GN_ARGS "ameba_cc = \"arm-none-eabi-gcc\"\n")
 string(APPEND CHIP_GN_ARGS "ameba_cxx = \"arm-none-eabi-c++\"\n")
 string(APPEND CHIP_GN_ARGS "ameba_cpu = \"ameba\"\n")
+string(APPEND CHIP_GN_ARGS "chip_inet_config_enable_ipv4 = false\n")
 
 # Build RPC
 if (matter_enable_rpc)
@@ -118,10 +121,13 @@ if (matter_enable_ota_requestor)
 string(APPEND CHIP_GN_ARGS "chip_enable_ota_requestor = true\n")
 endif (matter_enable_ota_requestor)
 
-if (CONFIG_ENABLE_ROTATING_DEVICE_ID)
- string(APPEND CHIP_GN_ARGS "chip_enable_additional_data_advertising" "true")
- string(APPEND CHIP_GN_ARGS "chip_enable_rotating_device_id" "true")
-endif()
+if (matter_enable_rotating_id)
+ string(APPEND CHIP_GN_ARGS "chip_enable_additional_data_advertising = true\n")
+ string(APPEND CHIP_GN_ARGS "chip_enable_rotating_device_id = true\n")
+else(matter_enable_rotating_id)
+ string(APPEND CHIP_GN_ARGS "chip_enable_additional_data_advertising = false\n")
+ string(APPEND CHIP_GN_ARGS "chip_enable_rotating_device_id = false\n")
+endif(matter_enable_rotating_id)
 
 file(GENERATE OUTPUT ${CHIP_OUTPUT}/args.gn CONTENT ${CHIP_GN_ARGS})
 

diff --git a/examples/all-clusters-app/ameba/chip_main.cmake b/examples/all-clusters-app/ameba/chip_main.cmake
@@ -218,13 +218,12 @@ endif (matter_enable_rpc)
 list(
  APPEND chip_main_flags
 
- -DINET_CONFIG_ENABLE_IPV4=1
+ -DINET_CONFIG_ENABLE_IPV4=0
  -DCHIP_PROJECT=1
  -DCHIP_DEVICE_LAYER_TARGET=Ameba
  -DUSE_ZAP_CONFIG
  -DCHIP_HAVE_CONFIG_H
  -DMBEDTLS_CONFIG_FILE=<mbedtls_config.h>
- -DCONFIG_ENABLE_ROTATING_DEVICE_ID=1
  -DMATTER_ALL_CLUSTERS_APP=1
 )
 

diff --git a/examples/all-clusters-app/ameba/main/chipinterface.cpp b/examples/all-clusters-app/ameba/main/chipinterface.cpp
@@ -28,6 +28,7 @@
 #include <app/clusters/network-commissioning/network-commissioning.h>
 #include <app/server/OnboardingCodesUtil.h>
 #include <app/server/Server.h>
+#include <app/util/af.h>
 #include <credentials/DeviceAttestationCredsProvider.h>
 #include <credentials/examples/DeviceAttestationCredsExample.h>
 #include <lib/support/ErrorStr.h>
@@ -56,14 +57,21 @@ using namespace ::chip::DeviceManager;
 using namespace ::chip::DeviceLayer;
 using namespace ::chip::System;
 
-namespace {
+namespace { // Network Commissioning
+constexpr EndpointId kNetworkCommissioningEndpointMain = 0;
+constexpr EndpointId kNetworkCommissioningEndpointSecondary = 0xFFFE;
+
 app::Clusters::NetworkCommissioning::Instance
- sWiFiNetworkCommissioningInstance(0 /* Endpoint Id */, &(NetworkCommissioning::AmebaWiFiDriver::GetInstance()));
+ sWiFiNetworkCommissioningInstance(kNetworkCommissioningEndpointMain /* Endpoint Id */,
+ &(NetworkCommissioning::AmebaWiFiDriver::GetInstance()));
 } // namespace
 
 void NetWorkCommissioningInstInit()
 {
  sWiFiNetworkCommissioningInstance.Init();
+
+ // We only have network commissioning on endpoint 0.
+ emberAfEndpointEnableDisable(kNetworkCommissioningEndpointSecondary, false);
 }
 
 Identify gIdentify0 = {
@@ -102,14 +110,13 @@ AmebaOTAImageProcessor gImageProcessor;
 extern "C" void amebaQueryImageCmdHandler()
 {
  ChipLogProgress(DeviceLayer, "Calling amebaQueryImageCmdHandler");
- static_cast<OTARequestor *>(GetRequestorInstance())->TriggerImmediateQuery();
+ PlatformMgr().ScheduleWork([](intptr_t) { GetRequestorInstance()->TriggerImmediateQuery(); });
 }
 
 extern "C" void amebaApplyUpdateCmdHandler()
 {
  ChipLogProgress(DeviceLayer, "Calling amebaApplyUpdateCmdHandler");
-
- static_cast<OTARequestor *>(GetRequestorInstance())->ApplyUpdate();
+ PlatformMgr().ScheduleWork([](intptr_t) { GetRequestorInstance()->ApplyUpdate(); });
 }
 
 static void InitOTARequestor(void)
@@ -134,6 +141,10 @@ static void InitOTARequestor(void)
 
 static void InitServer(intptr_t context)
 {
+#if CONFIG_ENABLE_OTA_REQUESTOR
+ InitOTARequestor();
+#endif
+
  // Init ZCL Data Model and CHIP App Server
  chip::Server::GetInstance().Init();
 
@@ -146,10 +157,6 @@ static void InitServer(intptr_t context)
  // QR code will be used with CHIP Tool
  PrintOnboardingCodes(chip::RendezvousInformationFlags(chip::RendezvousInformationFlag::kBLE));
  }
-
-#if CONFIG_ENABLE_OTA_REQUESTOR
- InitOTARequestor();
-#endif
 }
 
 extern "C" void ChipTest(void)

diff --git a/examples/lighting-app/ameba/chip_main.cmake b/examples/lighting-app/ameba/chip_main.cmake
@@ -75,7 +75,7 @@ target_include_directories(
 list(
  APPEND chip_main_flags
 
- -DINET_CONFIG_ENABLE_IPV4=1
+ -DINET_CONFIG_ENABLE_IPV4=0
  -DCHIP_PROJECT=1
  -DCHIP_DEVICE_LAYER_TARGET=Ameba
  -DUSE_ZAP_CONFIG

diff --git a/examples/lighting-app/ameba/main/chipinterface.cpp b/examples/lighting-app/ameba/main/chipinterface.cpp
@@ -32,6 +32,7 @@
 #include <app/clusters/identify-server/identify-server.h>
 #include <app/clusters/network-commissioning/network-commissioning.h>
 #include <app/server/OnboardingCodesUtil.h>
+#include <app/util/af.h>
 #include <lib/support/ErrorStr.h>
 #include <platform/Ameba/AmebaConfig.h>
 #include <platform/Ameba/NetworkCommissioningDriver.h>
@@ -55,14 +56,21 @@ using namespace ::chip::DeviceManager;
 using namespace ::chip::DeviceLayer;
 using namespace ::chip::System;
 
-namespace {
+namespace { // Network Commissioning
+constexpr EndpointId kNetworkCommissioningEndpointMain = 0;
+constexpr EndpointId kNetworkCommissioningEndpointSecondary = 0xFFFE;
+
 app::Clusters::NetworkCommissioning::Instance
- sWiFiNetworkCommissioningInstance(0 /* Endpoint Id */, &(NetworkCommissioning::AmebaWiFiDriver::GetInstance()));
+ sWiFiNetworkCommissioningInstance(kNetworkCommissioningEndpointMain /* Endpoint Id */,
+ &(NetworkCommissioning::AmebaWiFiDriver::GetInstance()));
 } // namespace
 
 void NetWorkCommissioningInstInit()
 {
  sWiFiNetworkCommissioningInstance.Init();
+
+ // We only have network commissioning on endpoint 0.
+ emberAfEndpointEnableDisable(kNetworkCommissioningEndpointSecondary, false);
 }
 
 #ifdef CONFIG_PLATFORM_8721D
@@ -87,14 +95,13 @@ AmebaOTAImageProcessor gImageProcessor;
 extern "C" void amebaQueryImageCmdHandler()
 {
  ChipLogProgress(DeviceLayer, "Calling amebaQueryImageCmdHandler");
- static_cast<OTARequestor *>(GetRequestorInstance())->TriggerImmediateQuery();
+ PlatformMgr().ScheduleWork([](intptr_t) { GetRequestorInstance()->TriggerImmediateQuery(); });
 }
 
 extern "C" void amebaApplyUpdateCmdHandler()
 {
  ChipLogProgress(DeviceLayer, "Calling amebaApplyUpdateCmdHandler");
-
- static_cast<OTARequestor *>(GetRequestorInstance())->ApplyUpdate();
+ PlatformMgr().ScheduleWork([](intptr_t) { GetRequestorInstance()->ApplyUpdate(); });
 }
 
 static void InitOTARequestor(void)
@@ -155,6 +162,10 @@ static Identify gIdentify1 = {
 
 static void InitServer(intptr_t context)
 {
+#if CONFIG_ENABLE_OTA_REQUESTOR
+ InitOTARequestor();
+#endif
+
  // Init ZCL Data Model and CHIP App Server
  chip::Server::GetInstance().Init();
 
@@ -167,10 +178,6 @@ static void InitServer(intptr_t context)
  // QR code will be used with CHIP Tool
  PrintOnboardingCodes(chip::RendezvousInformationFlags(chip::RendezvousInformationFlag::kBLE));
  }
-
-#if CONFIG_ENABLE_OTA_REQUESTOR
- InitOTARequestor();
-#endif
 }
 
 extern "C" void ChipTest(void)
@@ -195,10 +202,6 @@ extern "C" void ChipTest(void)
  chip::DeviceLayer::PlatformMgr().ScheduleWork(InitServer, 0);
 
  statusLED1.Init(STATUS_LED_GPIO_NUM);
-
-#if CONFIG_ENABLE_OTA_REQUESTOR
- InitOTARequestor();
-#endif
 }
 
 bool lowPowerClusterSleep()

diff --git a/examples/ota-requestor-app/ameba/chip_main.cmake b/examples/ota-requestor-app/ameba/chip_main.cmake
@@ -68,7 +68,7 @@ target_include_directories(
 list(
  APPEND chip_main_flags
 
- -DINET_CONFIG_ENABLE_IPV4=1
+ -DINET_CONFIG_ENABLE_IPV4=0
  -DCHIP_PROJECT=1
  -DCHIP_DEVICE_LAYER_TARGET=Ameba
  -DUSE_ZAP_CONFIG

diff --git a/examples/ota-requestor-app/ameba/main/chipinterface.cpp b/examples/ota-requestor-app/ameba/main/chipinterface.cpp
@@ -23,6 +23,7 @@
 
 #include <app/clusters/network-commissioning/network-commissioning.h>
 #include <app/server/Server.h>
+#include <app/util/af.h>
 #include <credentials/DeviceAttestationCredsProvider.h>
 #include <credentials/examples/DeviceAttestationCredsExample.h>
 
@@ -65,14 +66,21 @@ using namespace ::chip::Credentials;
 using namespace ::chip::DeviceManager;
 using namespace ::chip::DeviceLayer;
 
-namespace {
+namespace { // Network Commissioning
+constexpr EndpointId kNetworkCommissioningEndpointMain = 0;
+constexpr EndpointId kNetworkCommissioningEndpointSecondary = 0xFFFE;
+
 app::Clusters::NetworkCommissioning::Instance
- sWiFiNetworkCommissioningInstance(0 /* Endpoint Id */, &(NetworkCommissioning::AmebaWiFiDriver::GetInstance()));
+ sWiFiNetworkCommissioningInstance(kNetworkCommissioningEndpointMain /* Endpoint Id */,
+ &(NetworkCommissioning::AmebaWiFiDriver::GetInstance()));
 } // namespace
 
 void NetWorkCommissioningInstInit()
 {
  sWiFiNetworkCommissioningInstance.Init();
+
+ // We only have network commissioning on endpoint 0.
+ emberAfEndpointEnableDisable(kNetworkCommissioningEndpointSecondary, false);
 }
 
 static DeviceCallbacks EchoCallbacks;
@@ -86,14 +94,13 @@ AmebaOTAImageProcessor gImageProcessor;
 extern "C" void amebaQueryImageCmdHandler()
 {
  ChipLogProgress(DeviceLayer, "Calling amebaQueryImageCmdHandler");
- static_cast<OTARequestor *>(GetRequestorInstance())->TriggerImmediateQuery();
+ PlatformMgr().ScheduleWork([](intptr_t) { GetRequestorInstance()->TriggerImmediateQuery(); });
 }
 
 extern "C" void amebaApplyUpdateCmdHandler()
 {
  ChipLogProgress(DeviceLayer, "Calling amebaApplyUpdateCmdHandler");
-
- static_cast<OTARequestor *>(GetRequestorInstance())->ApplyUpdate();
+ PlatformMgr().ScheduleWork([](intptr_t) { GetRequestorInstance()->ApplyUpdate(); });
 }
 
 static void InitOTARequestor(void)
@@ -117,14 +124,14 @@ static void InitOTARequestor(void)
 
 static void InitServer(intptr_t context)
 {
+ InitOTARequestor();
+
  // Init ZCL Data Model and CHIP App Server
  chip::Server::GetInstance().Init();
 
  // Initialize device attestation config
  SetDeviceAttestationCredentialsProvider(Examples::GetExampleDACProvider());
  NetWorkCommissioningInstInit();
-
- InitOTARequestor();
 }
 
 extern "C" void ChipTest(void)

diff --git a/examples/pigweed-app/ameba/chip_main.cmake b/examples/pigweed-app/ameba/chip_main.cmake
@@ -72,7 +72,7 @@ link_directories(
 list(
  APPEND chip_main_flags
 
- -DINET_CONFIG_ENABLE_IPV4=1
+ -DINET_CONFIG_ENABLE_IPV4=0
  -DCHIP_PROJECT=1
  -DCHIP_DEVICE_LAYER_TARGET=Ameba
  -DUSE_ZAP_CONFIG

diff --git a/src/platform/Ameba/CHIPPlatformConfig.h b/src/platform/Ameba/CHIPPlatformConfig.h
@@ -50,6 +50,10 @@
 #define CHIP_CONFIG_MAX_PEER_NODES 16
 #endif // CHIP_CONFIG_MAX_PEER_NODES
 
+#ifndef CHIP_CONFIG_MAX_FABRICS
+#define CHIP_CONFIG_MAX_FABRICS 5
+#endif // CHIP_CONFIG_MAX_FABRICS
+
 #ifndef CHIP_CONFIG_MAX_UNSOLICITED_MESSAGE_HANDLERS
 #define CHIP_CONFIG_MAX_UNSOLICITED_MESSAGE_HANDLERS 16
 #endif // CHIP_CONFIG_MAX_UNSOLICITED_MESSAGE_HANDLERS

diff --git a/src/platform/Ameba/InetPlatformConfig.h b/src/platform/Ameba/InetPlatformConfig.h
@@ -43,5 +43,3 @@
 #ifndef INET_CONFIG_NUM_UDP_ENDPOINTS
 #define INET_CONFIG_NUM_UDP_ENDPOINTS 10
 #endif // INET_CONFIG_NUM_UDP_ENDPOINTS
-
-#define INET_CONFIG_ENABLE_IPV4 1
diff --git a/src/platform/Ameba/KeyValueStoreManagerImpl.cpp b/src/platform/Ameba/KeyValueStoreManagerImpl.cpp
@@ -54,14 +54,10 @@ CHIP_ERROR KeyValueStoreManagerImpl::_Get(const char * key, void * value, size_t
  if (TRUE == ret)
  {
  err = CHIP_NO_ERROR;
- if (read_bytes_size)
- {
- *read_bytes_size = value_size;
- }
  }
  else
  {
- err = CHIP_ERROR_INTERNAL;
+ err = CHIP_ERROR_PERSISTED_STORAGE_VALUE_NOT_FOUND;
  }
 
  return err;