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

[pump-app] Add PCC Event example #13081

Merged
merged 12 commits into from
Dec 17, 2021

Conversation

andersbangGF
Copy link
Contributor

Problem

  • The Pump Configuration and Control cluster supports a series of events that will be injected from the user application. This is not currently shown in the pump-app sample application.

Change overview

  • Added en example of event posting from the pump-app controlled with the launchpad button
  • Disabled unnecessary Wifi and Ethernet diagnostics clusters to free up enough FLASH to allow the app to build again.

Testing

*Built the pump-app for the TI platform and verified that there is enough memory for the app to link and run
*Tested on hardware. Commissioned pump-app on thread network and verified that the GeneralFault event is posted when the button is pressed.

@github-actions
Copy link

github-actions bot commented Dec 16, 2021

PR #13081: Size comparison from 91a1a5c to d300980

Full report (14 builds for efr32, k32w, linux, p6, qpg, telink)
platform target config section 91a1a5c d300980 change % change
efr32 lighting-app BRD4161A (read only) 829820 829820 0 0.0
(read/write) 127348 127348 0 0.0
.bss 125472 125472 0 0.0
.data 1876 1876 0 0.0
.text 829812 829812 0 0.0
BRD4161A+rpc (read only) 817440 817440 0 0.0
(read/write) 144016 144016 0 0.0
.bss 142040 142040 0 0.0
.data 1976 1976 0 0.0
.text 817432 817432 0 0.0
window-app BRD4161A (read only) 803228 803228 0 0.0
(read/write) 126288 126288 0 0.0
.bss 124456 124456 0 0.0
.data 1832 1832 0 0.0
.text 803220 803220 0 0.0
k32w light k32w061+release (read/write) 648300 648300 0 0.0
.bss 76480 76480 0 0.0
.data 1904 1904 0 0.0
.text 564116 564116 0 0.0
lock k32w061+release (read/write) 633048 633048 0 0.0
.bss 76200 76200 0 0.0
.data 1860 1860 0 0.0
.text 549188 549188 0 0.0
linux chip-tool-ipv6only arm64 (read only) 6930700 6930700 0 0.0
(read/write) 323633 323633 0 0.0
.bss 54577 54577 0 0.0
.data 1096 1096 0 0.0
.data.rel.ro 208096 208096 0 0.0
.dynamic 560 560 0 0.0
.got 56160 56160 0 0.0
.init 24 24 0 0.0
.init_array 168 168 0 0.0
.rodata 379476 379476 0 0.0
.text 5863652 5863652 0 0.0
thermostat-no-ble arm64 (read only) 1993780 1993780 0 0.0
(read/write) 143937 143937 0 0.0
.bss 64321 64321 0 0.0
.data 880 880 0 0.0
.data.rel.ro 72000 72000 0 0.0
.dynamic 560 560 0 0.0
.got 3840 3840 0 0.0
.init 24 24 0 0.0
.init_array 288 288 0 0.0
.rodata 128196 128196 0 0.0
.text 1654112 1654112 0 0.0
p6 all-clusters-app default (read/write) 2384624 2384624 0 0.0
.bss 117260 117260 0 0.0
.data 2544 2544 0 0.0
.text 1342888 1342888 0 0.0
light-app default (read/write) 2324008 2324008 0 0.0
.bss 106152 106152 0 0.0
.data 2384 2384 0 0.0
.text 1282272 1282272 0 0.0
lock-app default (read/write) 2296216 2296216 0 0.0
.bss 105032 105032 0 0.0
.data 2336 2336 0 0.0
.text 1254480 1254480 0 0.0
qpg lighting-app qpg6105+debug (read only) 531904 531904 0 0.0
(read/write) 146936 146936 0 0.0
.bss 86816 86816 0 0.0
.data 1004 1004 0 0.0
.text 526584 526584 0 0.0
lock-app qpg6105+debug (read only) 503600 503600 0 0.0
(read/write) 146940 146940 0 0.0
.bss 85952 85952 0 0.0
.data 952 952 0 0.0
.text 498280 498280 0 0.0
persistent-storage-app qpg6105+debug (read only) 106448 106448 0 0.0
(read/write) 146938 146938 0 0.0
.bss 36146 36146 0 0.0
.data 288 288 0 0.0
.text 101128 101128 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 830222 830222 0 0.0
bss 87040 87040 0 0.0
noinit 37160 37160 0 0.0
text 578410 578410 0 0.0

@bzbarsky-apple bzbarsky-apple merged commit 866e3e0 into project-chip:master Dec 17, 2021
@tlykkeberg-grundfos tlykkeberg-grundfos deleted the feature/AddPumpAppEvents branch March 2, 2022 07:28
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.

4 participants