Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SV2] Door Lock Zap File Enables Groups Cluster #22183

Closed
ifyall opened this issue Aug 26, 2022 · 0 comments · Fixed by #22954
Closed

[SV2] Door Lock Zap File Enables Groups Cluster #22183

ifyall opened this issue Aug 26, 2022 · 0 comments · Fixed by #22954

Comments

@ifyall
Copy link

ifyall commented Aug 26, 2022

Problem

The lock-app.zap file used for Matter's Door Lock example enable the Groups Cluster on endpoint 1. This should not be the case, as Groups is not a requirement for a Door Lock per the spec, section 8.1.4.

See device-library spec, version Version master 3dda765, 2022-08-23 15:57:15 -0700: Draft
https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/master/src/device_types/DoorLock.adoc#cluster-requirements

lock-app.zap file: https://github.com/project-chip/connectedhomeip/blob/sve-2/examples/lock-app/lock-common/lock-app.zap#L5485

Proposed Solution

Suggested fix is to either turn the enable value for the Groups server on endpoint 1 to "0" or remove all Groups Cluster references from the zap file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant