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

Removed Optional General Diagnostics Cluster Events For Apps that Don't Support Them #25261

Conversation

emargolis
Copy link
Contributor

Those optional events are currently only implemented in lighting-app, lock-app and all-clusters-app.

…'t Support Them.

Those optional events are currently only implemented in lighting-app,
lock-app and all-clusters-app.
@github-actions
Copy link

PR #25261: Size comparison from c6cadc8 to 95c1247

Full report (2 builds for cc32xx, mbed)
platform target config section c6cadc8 95c1247 change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 642545 642545 0 0.0
(read/write) 203664 203664 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197064 197064 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 930168 930168 0 0.0
.debug_aranges 87272 87272 0 0.0
.debug_frame 299752 299752 0 0.0
.debug_info 20254045 20254045 0 0.0
.debug_line 2655138 2655138 0 0.0
.debug_loc 2795342 2795342 0 0.0
.debug_ranges 281592 281592 0 0.0
.debug_str 3017877 3017877 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105817 105817 0 0.0
.shstrtab 232 232 0 0.0
.stab 204 204 0 0.0
.stabstr 441 441 0 0.0
.stack 2048 2048 0 0.0
.strtab 377630 377630 0 0.0
.symtab 256192 256192 0 0.0
.text 534604 534604 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2466784 2466784 0 0.0
.bss 215780 215780 0 0.0
.data 5880 5880 0 0.0
.text 1429428 1429428 0 0.0

@yufengwangca yufengwangca merged commit 2f462e3 into project-chip:master Feb 22, 2023
lecndav pushed a commit to lecndav/connectedhomeip that referenced this pull request Mar 22, 2023
…'t Support Them. (project-chip#25261)

Those optional events are currently only implemented in lighting-app,
lock-app and all-clusters-app.
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.

3 participants