Skip to content

Commit 2902338

Browse files
bzbarsky-applepull[bot]
authored andcommitted
Remove EmberAfClusterCommand and a bunch of things it entrained. (#25145)
1 parent ea35d73 commit 2902338

31 files changed

+26
-3002
lines changed

examples/bridge-app/linux/main.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424
#include <app-common/zap-generated/ids/Clusters.h>
2525
#include <app/ConcreteAttributePath.h>
2626
#include <app/EventLogging.h>
27-
#include <app/chip-zcl-zpro-codec.h>
2827
#include <app/clusters/network-commissioning/network-commissioning.h>
2928
#include <app/reporting/reporting.h>
3029
#include <app/util/af-types.h>

examples/dynamic-bridge-app/linux/main.cpp

-1
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@
2323
#include <app-common/zap-generated/attribute-type.h>
2424
#include <app/ConcreteAttributePath.h>
2525
#include <app/EventLogging.h>
26-
#include <app/chip-zcl-zpro-codec.h>
2726
#include <app/reporting/reporting.h>
2827
#include <app/util/af-types.h>
2928
#include <app/util/af.h>

scripts/tools/zap/tests/outputs/all-clusters-app/app-templates/IMClusterCommandHandler.cpp

-7
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,6 @@
3030
#include <lib/core/CHIPSafeCasts.h>
3131
#include <lib/support/TypeTraits.h>
3232

33-
// Currently we need some work to keep compatible with ember lib.
34-
#include <app/util/ember-compatibility-functions.h>
35-
3633
namespace chip {
3734
namespace app {
3835

@@ -2052,8 +2049,6 @@ void DispatchServerCommand(CommandHandler * apCommandObj, const ConcreteCommandP
20522049

20532050
void DispatchSingleClusterCommand(const ConcreteCommandPath & aCommandPath, TLV::TLVReader & aReader, CommandHandler * apCommandObj)
20542051
{
2055-
Compatibility::SetupEmberAfCommandHandler(apCommandObj, aCommandPath);
2056-
20572052
switch (aCommandPath.mClusterId)
20582053
{
20592054
case Clusters::AdministratorCommissioning::Id:
@@ -2148,8 +2143,6 @@ void DispatchSingleClusterCommand(const ConcreteCommandPath & aCommandPath, TLV:
21482143
apCommandObj->AddStatus(aCommandPath, Protocols::InteractionModel::Status::UnsupportedCluster);
21492144
break;
21502145
}
2151-
2152-
Compatibility::ResetEmberAfObjects();
21532146
}
21542147

21552148
} // namespace app

scripts/tools/zap/tests/outputs/lighting-app/app-templates/IMClusterCommandHandler.cpp

-7
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,6 @@
3030
#include <lib/core/CHIPSafeCasts.h>
3131
#include <lib/support/TypeTraits.h>
3232

33-
// Currently we need some work to keep compatible with ember lib.
34-
#include <app/util/ember-compatibility-functions.h>
35-
3633
namespace chip {
3734
namespace app {
3835

@@ -1179,8 +1176,6 @@ void DispatchServerCommand(CommandHandler * apCommandObj, const ConcreteCommandP
11791176

11801177
void DispatchSingleClusterCommand(const ConcreteCommandPath & aCommandPath, TLV::TLVReader & aReader, CommandHandler * apCommandObj)
11811178
{
1182-
Compatibility::SetupEmberAfCommandHandler(apCommandObj, aCommandPath);
1183-
11841179
switch (aCommandPath.mClusterId)
11851180
{
11861181
case Clusters::AdministratorCommissioning::Id:
@@ -1239,8 +1234,6 @@ void DispatchSingleClusterCommand(const ConcreteCommandPath & aCommandPath, TLV:
12391234
apCommandObj->AddStatus(aCommandPath, Protocols::InteractionModel::Status::UnsupportedCluster);
12401235
break;
12411236
}
1242-
1243-
Compatibility::ResetEmberAfObjects();
12441237
}
12451238

12461239
} // namespace app

src/app/chip-zcl-zpro-codec.h

-44
This file was deleted.

src/app/server/Server.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -240,7 +240,7 @@ CHIP_ERROR Server::Init(const ServerInitParams & initParams)
240240
#endif // CHIP_CONFIG_ENABLE_SERVER_IM_EVENT
241241

242242
// This initializes clusters, so should come after lower level initialization.
243-
InitDataModelHandler(&mExchangeMgr);
243+
InitDataModelHandler();
244244

245245
#if defined(CHIP_APP_USE_ECHO)
246246
err = InitEchoHandler(&mExchangeMgr);

src/app/tests/TestCommissionManager.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ using chip::CommissioningWindowManager;
3535
using chip::Server;
3636

3737
// Mock function for linking
38-
void InitDataModelHandler(chip::Messaging::ExchangeManager * exchangeMgr) {}
38+
void InitDataModelHandler() {}
3939

4040
namespace {
4141
bool sAdminFabricIndexDirty = false;

src/app/util/DataModelHandler.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,11 @@
3131

3232
using namespace ::chip;
3333

34-
void InitDataModelHandler(chip::Messaging::ExchangeManager * exchangeManager)
34+
void InitDataModelHandler()
3535
{
3636
#ifdef USE_ZAP_CONFIG
3737
ChipLogProgress(Zcl, "Using ZAP configuration...");
3838
emberAfEndpointConfigure();
39-
emberAfInit(exchangeManager);
39+
emberAfInit();
4040
#endif
4141
}

src/app/util/DataModelHandler.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -32,4 +32,4 @@
3232
* data model messages.
3333
*
3434
*/
35-
void InitDataModelHandler(chip::Messaging::ExchangeManager * exchangeMgr);
35+
void InitDataModelHandler();

0 commit comments

Comments
 (0)