Skip to content

Commit 20a6ead

Browse files
Update ZAP to v2024.03.14-nightly.
This fixes the endpoint_config representation of "endpoint with no specific parent endpoint set" to match the SDK internal representation, so we can remove some code that was working around the representation mismatch.
1 parent 84970a0 commit 20a6ead

File tree

6 files changed

+9
-16
lines changed

6 files changed

+9
-16
lines changed

scripts/setup/zap.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -8,13 +8,13 @@
88
"mac-amd64",
99
"windows-amd64"
1010
],
11-
"tags": ["version:2@v2024.02.29-nightly.1"]
11+
"tags": ["version:2@v2024.03.14-nightly.1"]
1212
},
1313
{
1414
"_comment": "Always get the amd64 version on mac until usable arm64 zap build is available",
1515
"path": "fuchsia/third_party/zap/mac-amd64",
1616
"platforms": ["mac-arm64"],
17-
"tags": ["version:2@v2024.02.29-nightly.1"]
17+
"tags": ["version:2@v2024.03.14-nightly.1"]
1818
}
1919
]
2020
}

scripts/setup/zap.version

+1-1
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
v2024.02.29-nightly
1+
v2024.03.14-nightly

scripts/tools/zap/tests/outputs/all-clusters-app/app-templates/endpoint_config.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -3072,5 +3072,5 @@ static_assert(ATTRIBUTE_LARGEST <= CHIP_CONFIG_MAX_ATTRIBUTE_STORE_ELEMENT_SIZE,
30723072
// Array of parent endpoints for each endpoint
30733073
#define FIXED_PARENT_ENDPOINTS \
30743074
{ \
3075-
0, 0, 0, 0 \
3075+
kInvalidEndpointId, kInvalidEndpointId, kInvalidEndpointId, kInvalidEndpointId \
30763076
}

scripts/tools/zap/tests/outputs/lighting-app/app-templates/endpoint_config.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -1210,5 +1210,5 @@ static_assert(ATTRIBUTE_LARGEST <= CHIP_CONFIG_MAX_ATTRIBUTE_STORE_ELEMENT_SIZE,
12101210
// Array of parent endpoints for each endpoint
12111211
#define FIXED_PARENT_ENDPOINTS \
12121212
{ \
1213-
0, 0 \
1213+
kInvalidEndpointId, kInvalidEndpointId \
12141214
}

scripts/tools/zap/zap_execution.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
# Use scripts/tools/zap/version_update.py to manage ZAP versioning as many
2424
# files may need updating for versions
2525
#
26-
MIN_ZAP_VERSION = '2024.2.29'
26+
MIN_ZAP_VERSION = '2024.3.14'
2727

2828

2929
class ZapTool:

src/app/util/attribute-storage.cpp

+3-10
Original file line numberDiff line numberDiff line change
@@ -214,16 +214,9 @@ void emberAfEndpointConfigure()
214214
emAfEndpoints[ep].endpoint = fixedEndpoints[ep];
215215
emAfEndpoints[ep].deviceTypeList =
216216
Span<const EmberAfDeviceType>(&fixedDeviceTypeList[fixedDeviceTypeListOffsets[ep]], fixedDeviceTypeListLengths[ep]);
217-
emAfEndpoints[ep].endpointType = &generatedEmberAfEndpointTypes[fixedEmberAfEndpointTypes[ep]];
218-
emAfEndpoints[ep].dataVersions = currentDataVersions;
219-
if (fixedParentEndpoints[ep] == 0)
220-
{
221-
emAfEndpoints[ep].parentEndpointId = kInvalidEndpointId;
222-
}
223-
else
224-
{
225-
emAfEndpoints[ep].parentEndpointId = fixedParentEndpoints[ep];
226-
}
217+
emAfEndpoints[ep].endpointType = &generatedEmberAfEndpointTypes[fixedEmberAfEndpointTypes[ep]];
218+
emAfEndpoints[ep].dataVersions = currentDataVersions;
219+
emAfEndpoints[ep].parentEndpointId = fixedParentEndpoints[ep];
227220

228221
emAfEndpoints[ep].bitmask.Set(EmberAfEndpointOptions::isEnabled);
229222
emAfEndpoints[ep].bitmask.Set(EmberAfEndpointOptions::isFlatComposition);

0 commit comments

Comments
 (0)