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

Pull request/power source configuration cluster #12209

Conversation

fessehaeve
Copy link
Contributor

Problem

Implement Power source configuration cluster

Change overview

Created XML file and included cluster in all-clusters-app and controller-cluster

Testing

Tested manually with Nordic dev kit and chip-tool.

@bzbarsky-apple bzbarsky-apple force-pushed the pull-request/power-source-configuration-cluster branch from a6bb87c to fc7d23e Compare December 3, 2021 15:22
@github-actions
Copy link

github-actions bot commented Dec 3, 2021

PR #12209: Size comparison from e8da685 to fc7d23e

Increases above 0.2%:

platform target config section e8da685 fc7d23e change % change
k32w lock-app k32w061+debug .bss 69752 69976 224 0.3
.data 1824 1828 4 0.2
shell k32w061+debug .bss 81400 81624 224 0.3
.data 1796 1800 4 0.2
qpg lock-app qpg6100+debug .bss 79184 79408 224 0.3
.data 916 920 4 0.4
Increases (3 builds for k32w, qpg)
platform target config section e8da685 fc7d23e change % change
k32w lock-app k32w061+debug (read/write) 617716 618660 944 0.2
.bss 69752 69976 224 0.3
.data 1824 1828 4 0.2
.text 540340 541056 716 0.1
shell k32w061+debug (read/write) 683608 684568 960 0.1
.bss 81400 81624 224 0.3
.data 1796 1800 4 0.2
.text 594612 595344 732 0.1
qpg lock-app qpg6100+debug (read only) 483316 484048 732 0.2
(read/write) 122332 122336 4 0.0
.bss 79184 79408 224 0.3
.data 916 920 4 0.4
.text 477996 478728 732 0.2
Full report (7 builds for k32w, qpg, telink)
platform target config section e8da685 fc7d23e change % change
k32w lighting-app k32w061+se05x+release (read/write) 728604 728604 0 0.0
.bss 79312 79312 0 0.0
.data 1860 1860 0 0.0
.text 641632 641632 0 0.0
lock-app k32w061+debug (read/write) 617716 618660 944 0.2
.bss 69752 69976 224 0.3
.data 1824 1828 4 0.2
.text 540340 541056 716 0.1
shell k32w061+debug (read/write) 683608 684568 960 0.1
.bss 81400 81624 224 0.3
.data 1796 1800 4 0.2
.text 594612 595344 732 0.1
qpg lighting-app qpg6100+debug (read only) 510404 510404 0 0.0
(read/write) 122332 122332 0 0.0
.bss 80272 80272 0 0.0
.data 964 964 0 0.0
.text 505084 505084 0 0.0
lock-app qpg6100+debug (read only) 483316 484048 732 0.2
(read/write) 122332 122336 4 0.0
.bss 79184 79408 224 0.3
.data 916 920 4 0.4
.text 477996 478728 732 0.2
persistent-storage-app qpg6100+debug (read only) 108208 108208 0 0.0
(read/write) 122332 122332 0 0.0
.bss 36696 36696 0 0.0
.data 292 292 0 0.0
.text 102888 102888 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 794710 794710 0 0.0
bss 80332 80332 0 0.0
noinit 37160 37160 0 0.0
text 554486 554486 0 0.0

@bzbarsky-apple bzbarsky-apple force-pushed the pull-request/power-source-configuration-cluster branch from fc7d23e to 38fb3fc Compare December 3, 2021 16:42

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
- A read on Sources attribute searches for all power source server clusters and decodes it in the response
Regenerate all after rebasing on ToT
Regenerate after changing descriptor cluster tests
Rebase and regenerate

Rebase and regenerate
@bzbarsky-apple bzbarsky-apple force-pushed the pull-request/power-source-configuration-cluster branch from 38fb3fc to 4ef05f9 Compare December 3, 2021 17:50
@bzbarsky-apple
Copy link
Contributor

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

Successfully merging this pull request may close these issues.

None yet

9 participants