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

Revert "Mtrpayloadparser check type" #22026

Merged
merged 1 commit into from
Aug 19, 2022

Conversation

woody-apple
Copy link
Contributor

Reverts #22024

@woody-apple woody-apple merged commit b506a86 into master Aug 19, 2022
@woody-apple woody-apple deleted the revert-22024-mtrpayloadparser_check_type branch August 19, 2022 01:34
@github-actions
Copy link

github-actions bot commented Aug 19, 2022

PR #22026: Size comparison from 2a5f7fe to 0143a8e

Increases (6 builds for cc13x2_26x2, nrfconnect, telink)
platform target config section 2a5f7fe 0143a8e change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 673019 673027 8 0.0
.text 583868 583876 8 0.0
lock-mtd LP_CC2652R7 (read/write) 180552 180560 8 0.0
pump-app LP_CC2652R7 (read/write) 157752 157760 8 0.0
nrfconnect all-clusters-minimal-app nrf52840dk_nrf52840 text 803068 803072 4 0.0
telink light-switch-app tlsr9518adk80d (read/write) 808736 808744 8 0.0
text 571316 571318 2 0.0
lighting-app tlsr9518adk80d (read/write) 830672 830680 8 0.0
text 589404 589408 4 0.0
Decreases (4 builds for cc13x2_26x2, esp32)
platform target config section 2a5f7fe 0143a8e change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read/write) 178476 178468 -8 -0.0
lock-mtd LP_CC2652R7 (read only) 656831 656823 -8 -0.0
.text 554592 554584 -8 -0.0
pump-app LP_CC2652R7 (read only) 684751 684743 -8 -0.0
.text 594308 594300 -8 -0.0
esp32 all-clusters-app c3devkit (read only) 1031272 1031268 -4 -0.0
.flash.text 1031272 1031268 -4 -0.0
Full report (16 builds for cc13x2_26x2, esp32, linux, mbed, nrfconnect, telink)
platform target config section 2a5f7fe 0143a8e change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 673019 673027 8 0.0
(read/write) 178476 178468 -8 -0.0
.bss 74388 74388 0 0.0
.data 3372 3372 0 0.0
.rodata 88835 88835 0 0.0
.text 583868 583876 8 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 637747 637747 0 0.0
(read/write) 157948 157948 0 0.0
.bss 73660 73660 0 0.0
.data 3372 3372 0 0.0
.rodata 77979 77979 0 0.0
.text 559444 559444 0 0.0
lock-ftd LP_CC2652R7 (read only) 674079 674079 0 0.0
(read/write) 167616 167616 0 0.0
.bss 71476 71476 0 0.0
.data 3296 3296 0 0.0
.rodata 76671 76671 0 0.0
.text 596928 596928 0 0.0
lock-mtd LP_CC2652R7 (read only) 656831 656823 -8 -0.0
(read/write) 180552 180560 8 0.0
.bss 67164 67164 0 0.0
.data 3296 3296 0 0.0
.rodata 101759 101759 0 0.0
.text 554592 554584 -8 -0.0
pump-app LP_CC2652R7 (read only) 684751 684743 -8 -0.0
(read/write) 157752 157760 8 0.0
.bss 71516 71516 0 0.0
.data 3296 3296 0 0.0
.rodata 89959 89959 0 0.0
.text 594308 594300 -8 -0.0
pump-controller-app LP_CC2652R7 (read only) 669243 669243 0 0.0
(read/write) 173380 173380 0 0.0
.bss 71636 71636 0 0.0
.data 3292 3292 0 0.0
.rodata 85515 85515 0 0.0
.text 583248 583248 0 0.0
shell LP_CC2652R7 (read only) 665710 665710 0 0.0
(read/write) 181304 181304 0 0.0
.bss 76708 76708 0 0.0
.data 3376 3376 0 0.0
.rodata 85782 85782 0 0.0
.text 579612 579612 0 0.0
esp32 all-clusters-app c3devkit (read only) 1031272 1031268 -4 -0.0
(read/write) 1489610 1489610 0 0.0
.dram0.bss 71136 71136 0 0.0
.dram0.data 14600 14600 0 0.0
.flash.rodata 218440 218440 0 0.0
.flash.text 1031272 1031268 -4 -0.0
.iram0.text 62902 62902 0 0.0
m5stack (read only) 1084371 1084371 0 0.0
(read/write) 491544 491544 0 0.0
.dram0.bss 76640 76640 0 0.0
.dram0.data 34144 34144 0 0.0
.flash.rodata 248764 248764 0 0.0
.flash.text 1078987 1078987 0 0.0
.iram0.text 123267 123267 0 0.0
linux chip-tool-ipv6only arm64 (read only) 10255980 10255980 0 0.0
(read/write) 699105 699105 0 0.0
.bss 33297 33297 0 0.0
.data 3272 3272 0 0.0
.data.rel.ro 643792 643792 0 0.0
.dynamic 560 560 0 0.0
.got 13784 13784 0 0.0
.init 24 24 0 0.0
.init_array 192 192 0 0.0
.rodata 493340 493340 0 0.0
.text 8124596 8124596 0 0.0
thermostat-no-ble arm64 (read only) 2357300 2357300 0 0.0
(read/write) 141825 141825 0 0.0
.bss 55345 55345 0 0.0
.data 1672 1672 0 0.0
.data.rel.ro 75984 75984 0 0.0
.dynamic 560 560 0 0.0
.got 5048 5048 0 0.0
.init 24 24 0 0.0
.init_array 408 408 0 0.0
.rodata 140620 140620 0 0.0
.text 1978960 1978960 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2454328 2454328 0 0.0
.bss 215044 215044 0 0.0
.data 5872 5872 0 0.0
.text 1416972 1416972 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1180603 1180603 0 0.0
bss 143737 143737 0 0.0
rodata 143356 143356 0 0.0
text 814664 814664 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1159799 1159799 0 0.0
bss 142964 142964 0 0.0
rodata 134944 134944 0 0.0
text 803068 803072 4 0.0
telink light-switch-app tlsr9518adk80d (read/write) 808736 808744 8 0.0
bss 71448 71448 0 0.0
noinit 43488 43488 0 0.0
text 571316 571318 2 0.0
lighting-app tlsr9518adk80d (read/write) 830672 830680 8 0.0
bss 72304 72304 0 0.0
noinit 43488 43488 0 0.0
text 589404 589408 4 0.0

isiu-apple pushed a commit to isiu-apple/connectedhomeip that referenced this pull request Sep 16, 2022
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.

1 participant