Skip to content

Commit 207005b

Browse files
ArekBalysNordiccarlescufi
authored andcommitted
samples: matter: Fix manufacturer-specific sample clusters
Fixed wrong configuration in clusters: - removed client Basic Information cluster. - removed client NordicDevKit cluster. Signed-off-by: Arkadiusz Balys <[email protected]> (cherry picked from commit af14898)
1 parent 33c1521 commit 207005b

File tree

4 files changed

+5
-83
lines changed

4 files changed

+5
-83
lines changed

samples/matter/manufacturer_specific/src/default_zap/NordicDevKitCluster.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141
<class>Simple</class>
4242
<scope>Endpoint</scope>
4343
<clusters lockOthers="true">
44-
<include cluster="NordicDevKit" client="true" server="true" clientLocked="false" serverLocked="false"/>
44+
<include cluster="NordicDevKit" client="false" server="true" clientLocked="false" serverLocked="false"/>
4545
</clusters>
4646
</deviceType>
4747
</configurator>

samples/matter/manufacturer_specific/src/default_zap/manufacturer_specific.zap

Lines changed: 0 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -425,24 +425,6 @@
425425
}
426426
]
427427
},
428-
{
429-
"name": "Basic Information",
430-
"code": 40,
431-
"mfgCode": null,
432-
"define": "BASIC_INFORMATION_CLUSTER",
433-
"side": "client",
434-
"enabled": 1,
435-
"commands": [
436-
{
437-
"name": "GenerateRandom",
438-
"code": 0,
439-
"mfgCode": null,
440-
"source": "client",
441-
"isIncoming": 0,
442-
"isEnabled": 1
443-
}
444-
]
445-
},
446428
{
447429
"name": "Basic Information",
448430
"code": 40,
@@ -2679,24 +2661,6 @@
26792661
"deviceTypeCode": 1,
26802662
"deviceTypeProfileId": 4095,
26812663
"clusters": [
2682-
{
2683-
"name": "NordicDevKit",
2684-
"code": 4294048769,
2685-
"mfgCode": null,
2686-
"define": "NORDIC_DEV_KIT_CLUSTER",
2687-
"side": "client",
2688-
"enabled": 1,
2689-
"commands": [
2690-
{
2691-
"name": "SetLED",
2692-
"code": 4293984256,
2693-
"mfgCode": null,
2694-
"source": "client",
2695-
"isIncoming": 0,
2696-
"isEnabled": 1
2697-
}
2698-
]
2699-
},
27002664
{
27012665
"name": "NordicDevKit",
27022666
"code": 4294048769,

samples/matter/manufacturer_specific/src/default_zap/zap-generated/endpoint_config.h

Lines changed: 4 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -430,7 +430,7 @@
430430
// clang-format on
431431

432432
// This is an array of EmberAfCluster structures.
433-
#define GENERATED_CLUSTER_COUNT 14
433+
#define GENERATED_CLUSTER_COUNT 12
434434
// clang-format off
435435
#define GENERATED_CLUSTERS { \
436436
{ \
@@ -459,19 +459,6 @@
459459
.eventList = nullptr, \
460460
.eventCount = 0, \
461461
},\
462-
{ \
463-
/* Endpoint: 0, Cluster: Basic Information (client) */ \
464-
.clusterId = 0x00000028, \
465-
.attributes = ZAP_ATTRIBUTE_INDEX(13), \
466-
.attributeCount = 0, \
467-
.clusterSize = 0, \
468-
.mask = ZAP_CLUSTER_MASK(CLIENT), \
469-
.functions = NULL, \
470-
.acceptedCommandList = nullptr, \
471-
.generatedCommandList = nullptr, \
472-
.eventList = nullptr, \
473-
.eventCount = 0, \
474-
},\
475462
{ \
476463
/* Endpoint: 0, Cluster: Basic Information (server) */ \
477464
.clusterId = 0x00000028, \
@@ -589,19 +576,6 @@
589576
.eventList = nullptr, \
590577
.eventCount = 0, \
591578
},\
592-
{ \
593-
/* Endpoint: 1, Cluster: NordicDevKit (client) */ \
594-
.clusterId = 0xFFF1FC01, \
595-
.attributes = ZAP_ATTRIBUTE_INDEX(86), \
596-
.attributeCount = 0, \
597-
.clusterSize = 0, \
598-
.mask = ZAP_CLUSTER_MASK(CLIENT), \
599-
.functions = NULL, \
600-
.acceptedCommandList = nullptr, \
601-
.generatedCommandList = nullptr, \
602-
.eventList = nullptr, \
603-
.eventCount = 0, \
604-
},\
605579
{ \
606580
/* Endpoint: 1, Cluster: NordicDevKit (server) */ \
607581
.clusterId = 0xFFF1FC01, \
@@ -624,8 +598,8 @@
624598
// This is an array of EmberAfEndpointType structures.
625599
#define GENERATED_ENDPOINT_TYPES \
626600
{ \
627-
{ ZAP_CLUSTER_INDEX(0), 12, 86 }, \
628-
{ ZAP_CLUSTER_INDEX(12), 2, 263 }, \
601+
{ ZAP_CLUSTER_INDEX(0), 11, 86 }, \
602+
{ ZAP_CLUSTER_INDEX(11), 1, 263 }, \
629603
}
630604

631605
// Largest attribute size is needed for various buffers
@@ -651,13 +625,7 @@ static_assert(ATTRIBUTE_LARGEST <= CHIP_CONFIG_MAX_ATTRIBUTE_STORE_ELEMENT_SIZE,
651625
#define FIXED_PROFILE_IDS { 0x0103, 0x0FFF }
652626

653627
// Array of device types
654-
#define FIXED_DEVICE_TYPES \
655-
{ \
656-
{ 0x00000012, 1 }, { 0x00000016, 3 }, \
657-
{ \
658-
0x00000001, 1 \
659-
} \
660-
}
628+
#define FIXED_DEVICE_TYPES { { 0x00000012, 1 }, { 0x00000016, 3 }, { 0x00000001, 1 } }
661629

662630
// Array of device type offsets
663631
#define FIXED_DEVICE_TYPE_OFFSETS { 0, 2 }

samples/matter/manufacturer_specific/src/default_zap/zap-generated/gen_config.h

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@
2323
/**** Cluster endpoint counts ****/
2424
#define MATTER_DM_DESCRIPTOR_CLUSTER_SERVER_ENDPOINT_COUNT (1)
2525
#define MATTER_DM_ACCESS_CONTROL_CLUSTER_SERVER_ENDPOINT_COUNT (1)
26-
#define MATTER_DM_BASIC_INFORMATION_CLUSTER_CLIENT_ENDPOINT_COUNT (1)
2726
#define MATTER_DM_BASIC_INFORMATION_CLUSTER_SERVER_ENDPOINT_COUNT (1)
2827
#define MATTER_DM_OTA_SOFTWARE_UPDATE_PROVIDER_CLUSTER_CLIENT_ENDPOINT_COUNT (1)
2928
#define MATTER_DM_OTA_SOFTWARE_UPDATE_REQUESTOR_CLUSTER_SERVER_ENDPOINT_COUNT (1)
@@ -33,7 +32,6 @@
3332
#define MATTER_DM_ADMINISTRATOR_COMMISSIONING_CLUSTER_SERVER_ENDPOINT_COUNT (1)
3433
#define MATTER_DM_OPERATIONAL_CREDENTIALS_CLUSTER_SERVER_ENDPOINT_COUNT (1)
3534
#define MATTER_DM_GROUP_KEY_MANAGEMENT_CLUSTER_SERVER_ENDPOINT_COUNT (1)
36-
#define MATTER_DM_NORDIC_DEV_KIT_CLUSTER_CLIENT_ENDPOINT_COUNT (1)
3735
#define MATTER_DM_NORDIC_DEV_KIT_CLUSTER_SERVER_ENDPOINT_COUNT (1)
3836

3937
/**** Cluster Plugins ****/
@@ -48,10 +46,6 @@
4846
#define MATTER_DM_PLUGIN_ACCESS_CONTROL_SERVER
4947
#define MATTER_DM_PLUGIN_ACCESS_CONTROL
5048

51-
// Use this macro to check if the client side of the Basic Information cluster is included
52-
#define ZCL_USING_BASIC_INFORMATION_CLUSTER_CLIENT
53-
#define MATTER_DM_PLUGIN_BASIC_INFORMATION_CLIENT
54-
5549
// Use this macro to check if the server side of the Basic Information cluster is included
5650
#define ZCL_USING_BASIC_INFORMATION_CLUSTER_SERVER
5751
#define MATTER_DM_PLUGIN_BASIC_INFORMATION_SERVER
@@ -96,10 +90,6 @@
9690
#define MATTER_DM_PLUGIN_GROUP_KEY_MANAGEMENT_SERVER
9791
#define MATTER_DM_PLUGIN_GROUP_KEY_MANAGEMENT
9892

99-
// Use this macro to check if the client side of the NordicDevKit cluster is included
100-
#define ZCL_USING_NORDIC_DEV_KIT_CLUSTER_CLIENT
101-
#define MATTER_DM_PLUGIN_NORDIC_DEV_KIT_CLIENT
102-
10393
// Use this macro to check if the server side of the NordicDevKit cluster is included
10494
#define ZCL_USING_NORDIC_DEV_KIT_CLUSTER_SERVER
10595
#define MATTER_DM_PLUGIN_NORDIC_DEV_KIT_SERVER

0 commit comments

Comments
 (0)