-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Implements the Commissioner Control Cluster in Matter SDK #34375
Implements the Commissioner Control Cluster in Matter SDK #34375
Conversation
PR #34375: Size comparison from a30dd12 to d8d794d Increases above 0.2%:
Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
src/app/clusters/commissioner-control-server/commissioner-control-server.h
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.h
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.h
Outdated
Show resolved
Hide resolved
PR #34375: Size comparison from c7659cb to 2970c78 Full report (4 builds for cc32xx, mbed, stm32)
|
PR #34375: Size comparison from 3418614 to 4fa28cd Increases above 0.2%:
Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
PR #34375: Size comparison from c024e4f to 10a55bb Increases above 0.2%:
Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
…rol-server.cpp Co-authored-by: Terence Hampson <[email protected]>
…rol-server.cpp Co-authored-by: Boris Zbarsky <[email protected]>
…rol-server.h Co-authored-by: Boris Zbarsky <[email protected]>
PR #34375: Size comparison from d62f5a6 to d96cfbc Increases above 0.2%:
Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
src/app/clusters/commissioner-control-server/commissioner-control-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.h
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.h
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.h
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.h
Show resolved
Hide resolved
src/app/clusters/commissioner-control-server/commissioner-control-server.h
Outdated
Show resolved
Hide resolved
@@ -1647,6 +1698,21 @@ endpoint 0 { | |||
handle command KeySetReadAllIndices; | |||
handle command KeySetReadAllIndicesResponse; | |||
} | |||
|
|||
server cluster CommissionerControl { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tehampson Is that OK, that the CommissionerControl is on endpoint 0? In the spec, the CommissionerControl should be on Aggregator endpoint, which is endpoint 1 here.
…ip#34375) Co-authored-by: Terence Hampson <[email protected]> Co-authored-by: Boris Zbarsky <[email protected]> Co-authored-by: Restyled.io <[email protected]>
…ip#34375) Co-authored-by: Terence Hampson <[email protected]> Co-authored-by: Boris Zbarsky <[email protected]> Co-authored-by: Restyled.io <[email protected]>
Implements https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/9089
This PR contains the implementation of the Commissioner Control Cluster server itself. The actual usage of this cluster across multiple example applications (Fabric Admin and Fabric Bridge), will be addressed in a follow-up PR.
Fixes #33222