-
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
Decouple ember functions from general commissioning cluster #36836
Decouple ember functions from general commissioning cluster #36836
Conversation
2c7b528
to
41ef3ea
Compare
PR #36836: Size comparison from 8ea6b8d to 41ef3ea Full report (69 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
src/app/clusters/general-commissioning-server/general-commissioning-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/general-commissioning-server/general-commissioning-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/general-commissioning-server/general-commissioning-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/general-commissioning-server/general-commissioning-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/general-commissioning-server/general-commissioning-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/general-commissioning-server/general-commissioning-server.cpp
Show resolved
Hide resolved
src/app/clusters/general-commissioning-server/general-commissioning-server.cpp
Outdated
Show resolved
Hide resolved
PR #36836: Size comparison from 8ea6b8d to 7e9f238 Full report (69 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
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.
Wait for #34065 to land to avoid conflicting it, since it's for 1.4.1
PR #36836: Size comparison from 8ea6b8d to a16388d Full report (69 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
Other PR has been merged, we can update this PR now.
PR #36836: Size comparison from 4ae6882 to e16195e Full report (69 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
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.
Was this expected to lead to several hundred bytes of Flash increase on constrained platforms? If not, what can we do to reduce that?
PR #36836: Size comparison from 4ae6882 to 9ba0d4b Full report (25 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, nrfconnect, qpg, stm32, telink, tizen)
|
PR #36836: Size comparison from 4ae6882 to 90de8a1 Full report (5 builds for cc32xx, stm32, tizen)
|
This reverts commit 90de8a1.
PR #36836: Size comparison from 4ae6882 to 9d6a54a Full report (69 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
The majority of the flash increase comes from the new debug logs added. If a command is not configured in ZAP, the corresponding dispatch logic is not generated. However, after transitioning to CHI, this increase is unavoidable. Specifically, on QPG platforms, we observe an additional 40 bytes of flash and 24 bytes of RAM increase. And flash decrease on some other constrained platforms. |
src/app/clusters/general-commissioning-server/general-commissioning-server.cpp
Outdated
Show resolved
Hide resolved
src/app/clusters/general-commissioning-server/general-commissioning-server.cpp
Show resolved
Hide resolved
PR #36836: Size comparison from 4ae6882 to 2c042ff Full report (69 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
|
Decouple General Commissioning Cluster from ember and migrate to use CHI (CommandHandlerInterface) instead
Fix: #36757
Verified with manual pairing and CI