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.

Sign up for GitHub

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

Jump to bottom

[Silabs] : Cleanup and applying coding guideline on Silabs code #25977

Merged
merged 10 commits into from
Apr 13, 2023
Merged

[Silabs] : Cleanup and applying coding guideline on Silabs code #25977

merged 10 commits into from
Apr 13, 2023

Conversation

sharad-patil24
Copy link
Contributor

This PR is the first PR in the series of PR's to cleanup and Update our code to follow coding style guideline provided for Silabs.

In this PR we focused on the necessary ( Must do ) things.

@github-actions
Copy link

github-actions bot commented Apr 4, 2023

PR #25977: Size comparison from a8096b1 to 3d9e0b5

Full report (1 build for cc32xx)
platform target config section a8096b1b 3d9e0b5 change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 644409 644409 0 0.0
(read/write) 203848 203848 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197248 197248 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 933099 933099 0 0.0
.debug_aranges 87608 87608 0 0.0
.debug_frame 301328 301328 0 0.0
.debug_info 20302987 20302987 0 0.0
.debug_line 2679748 2679748 0 0.0
.debug_loc 2824679 2824679 0 0.0
.debug_ranges 286200 286200 0 0.0
.debug_str 3039406 3039406 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105953 105953 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 377422 377422 0 0.0
.symtab 256768 256768 0 0.0
.text 536336 536336 0 0.0

@github-actions
Copy link

github-actions bot commented Apr 5, 2023

PR #25977: Size comparison from f0f8931 to 49a510a

Increases above 0.2%:

platform target config section f0f8931 49a510a change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read/write) 169040 170560 1520 0.9
lock-ftd LP_CC2652R7 (read/write) 169552 170648 1096 0.6
lock-mtd LP_CC2652R7 (read/write) 178492 179580 1088 0.6
pump-app LP_CC2652R7 (read/write) 157484 158716 1232 0.8
pump-controller-app LP_CC2652R7 (read/write) 172612 173956 1344 0.8
Increases (10 builds for cc13x2_26x2, telink)
platform target config section f0f8931 49a510a change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read/write) 169040 170560 1520 0.9
lock-ftd LP_CC2652R7 (read/write) 169552 170648 1096 0.6
lock-mtd LP_CC2652R7 (read/write) 178492 179580 1088 0.6
pump-app LP_CC2652R7 (read/write) 157484 158716 1232 0.8
pump-controller-app LP_CC2652R7 (read/write) 172612 173956 1344 0.8
shell LP_CC2652R7 (read/write) 179648 179768 120 0.1
.text 587928 587936 8 0.0
telink light-switch-app tlsr9518adk80d text 597418 597422 4 0.0
lock-app tlsr9518adk80d text 611220 611224 4 0.0
ota-requestor-app tlsr9518adk80d text 607952 607956 4 0.0
pump-app tlsr9518adk80d text 589048 589050 2 0.0
Decreases (48 builds for bl602, bl702, cc13x2_26x2, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
platform target config section f0f8931 49a510a change % change
bl602 lighting-app bl602 (read/write) 1356554 1355210 -1344 -0.1
bl602+rpc (read/write) 1401722 1400382 -1340 -0.1
.text 1060512 1060510 -2 -0.0
bl702 lighting-app bl702 (read/write) 1184727 1183687 -1040 -0.1
.rodata 106704 105664 -1040 -1.0
bl702+rpc (read/write) 1276275 1275171 -1104 -0.1
.rodata 120992 119888 -1104 -0.9
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 682215 680695 -1520 -0.2
.rodata 88535 87015 -1520 -1.7
all-clusters-minimal-app LP_CC2652R7 (read only) 645567 643967 -1600 -0.2
.rodata 78407 76807 -1600 -2.0
lock-ftd LP_CC2652R7 (read only) 679167 678071 -1096 -0.2
.rodata 76783 75687 -1096 -1.4
lock-mtd LP_CC2652R7 (read only) 665387 664299 -1088 -0.2
.rodata 103595 102507 -1088 -1.1
pump-app LP_CC2652R7 (read only) 691979 690747 -1232 -0.2
.rodata 90979 89747 -1232 -1.4
pump-controller-app LP_CC2652R7 (read only) 676987 675643 -1344 -0.2
.rodata 86843 85499 -1344 -1.5
shell LP_CC2652R7 (read only) 673678 673558 -120 -0.0
.rodata 85438 85310 -128 -0.1
cc32xx lock CC3235SF_LAUNCHXL (read only) 644409 642745 -1664 -0.3
.rodata 105953 104289 -1664 -1.6
cyw30739 light cyw930739m2evb_01 (read/write) 533954 533506 -448 -0.1
.app_xip_area 414524 414076 -448 -0.1
lock cyw930739m2evb_01 (read/write) 538234 537882 -352 -0.1
.app_xip_area 413516 413164 -352 -0.1
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 555246 554766 -480 -0.1
.app_xip_area 436576 436096 -480 -0.1
efr32 lighting-app BRD4161A+rs9116 (read/write) 1018648 1017712 -936 -0.1
.text 837800 836864 -936 -0.1
lock-app BRD4161A+wf200 (read/write) 1071336 1070728 -608 -0.1
.text 896032 895424 -608 -0.1
lighting-app BRD4187C (read/write) 1025948 1025324 -624 -0.1
.text 839376 838752 -624 -0.1
window-app BRD4187C (read/write) 1142492 1141548 -944 -0.1
.text 971596 970652 -944 -0.1
esp32 all-clusters-app c3devkit (read/write) 1585994 1585466 -528 -0.0
.flash.rodata 222496 221968 -528 -0.2
m5stack (read/write) 501947 501351 -596 -0.1
.flash.rodata 251112 250516 -596 -0.2
k32w contact k32w0+release (read/write) 674416 673736 -680 -0.1
.text 575260 574580 -680 -0.1
light k32w0+release (read/write) 673508 672876 -632 -0.1
.text 574700 574068 -632 -0.1
lock k32w0+release (read/write) 628180 627588 -592 -0.1
.text 547760 547168 -592 -0.1
linux chip-tool-ipv6only arm64 (read only) 12129740 12113692 -16048 -0.1
.rodata 582204 566156 -16048 -2.8
thermostat-no-ble arm64 (read only) 2518364 2500492 -17872 -0.7
.rodata 147272 129400 -17872 -12.1
mbed lock-app CY8CPROTO_062_4343W+release (read/write) 2466152 2465472 -680 -0.0
.text 1428796 1428116 -680 -0.0
nrfconnect all-clusters-app nrf7002dk_nrf5340_cpuapp text 775536 775532 -4 -0.0
psoc6 all-clusters cy8ckit_062s2_43012 (read/write) 1763828 1760444 -3384 -0.2
.debug_info 28088663 28088661 -2 -0.0
.text 1562936 1559552 -3384 -0.2
all-clusters-minimal cy8ckit_062s2_43012 (read/write) 1704148 1700484 -3664 -0.2
.text 1504064 1500400 -3664 -0.2
light cy8ckit_062s2_43012 (read/write) 1617204 1614212 -2992 -0.2
.debug_info 22906490 22906489 -1 -0.0
.text 1425384 1422392 -2992 -0.2
lock cy8ckit_062s2_43012 (read/write) 1674116 1671364 -2752 -0.2
.debug_info 23157847 23157846 -1 -0.0
.text 1454520 1451768 -2752 -0.2
qpg lighting-app qpg6105+debug (read/write) 1153680 1153440 -240 -0.0
.text 600776 600536 -240 -0.0
lock-app qpg6105+debug (read/write) 1122216 1122048 -168 -0.0
.text 569316 569148 -168 -0.0
telink all-clusters-app tlsr9518adk80d (read/write) 1022736 1022724 -12 -0.0
all-clusters-minimal-app tlsr9518adk80d (read/write) 957788 957768 -20 -0.0
text 653102 653100 -2 -0.0
contact-sensor-app tlsr9518adk80d (read/write) 861696 861676 -20 -0.0
text 583222 583220 -2 -0.0
light-switch-app tlsr9518adk80d (read/write) 877552 877548 -4 -0.0
lighting-app tlsr9518adk80d (read/write) 956236 956224 -12 -0.0
text 663796 663794 -2 -0.0
lighting-app-factory-data tlsr9518adk80d (read/write) 967588 967576 -12 -0.0
lighting-app-rpc tlsr9518adk80d (read/write) 1024408 1024396 -12 -0.0
text 706900 706898 -2 -0.0
lock-app tlsr9518adk80d (read/write) 916040 916036 -4 -0.0
ota-requestor-app tlsr9518adk80d (read/write) 890444 890440 -4 -0.0
pump-app tlsr9518adk80d (read/write) 867128 867124 -4 -0.0
pump-controller-app tlsr9518adk80d (read/write) 854404 854392 -12 -0.0
text 579388 579386 -2 -0.0
temperature-measurement-app tlsr9518adk80d (read/write) 850192 850180 -12 -0.0
thermostat tlsr9518adk80d (read/write) 881704 881692 -12 -0.0
text 598952 598950 -2 -0.0
window-covering tlsr9518adk80d (read/write) 944056 944044 -12 -0.0
text 650026 650024 -2 -0.0
Full report (50 builds for bl602, bl702, cc13x2_26x2, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
platform target config section f0f8931 49a510a change % change
bl602 lighting-app bl602 (read/write) 1356554 1355210 -1344 -0.1
.bss 94778 94778 0 0.0
.data 9776 9776 0 0.0
.text 1029832 1029832 0 0.0
bl602+rpc (read/write) 1401722 1400382 -1340 -0.1
.bss 102826 102826 0 0.0
.data 10168 10168 0 0.0
.text 1060512 1060510 -2 -0.0
bl702 lighting-app bl702 0 0 0 0.0
(read only) 3358 3358 0 0.0
(read/write) 1184727 1183687 -1040 -0.1
.bleromro 6342 6342 0 0.0
.bleromrw 124 124 0 0.0
.boot2 292 292 0 0.0
.bss 70721 70721 0 0.0
.bss_psram 24368 24368 0 0.0
.comment 48 48 0 0.0
.data 4088 4088 0 0.0
.debug_abbrev 1551688 1551688 0 0.0
.debug_aranges 133944 133944 0 0.0
.debug_frame 492632 492632 0 0.0
.debug_info 40755236 40755236 0 0.0
.debug_line 5298432 5298432 0 0.0
.debug_loc 3433903 3433903 0 0.0
.debug_ranges 374376 374376 0 0.0
.debug_str 3593197 3593197 0 0.0
.hbn 536 536 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 140 140 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 106704 105664 -1040 -1.0
.rsvd 2960 2960 0 0.0
.sha_ocram 72 72 0 0.0
.shstrtab 304 304 0 0.0
.stack 2048 2048 0 0.0
.strtab 572622 572622 0 0.0
.symtab 173744 173744 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3358 3358 0 0.0
.text 957738 957738 0 0.0
bl702+rpc 0 0 0 0.0
(read only) 3358 3358 0 0.0
(read/write) 1276275 1275171 -1104 -0.1
.bleromro 6342 6342 0 0.0
.bleromrw 124 124 0 0.0
.boot2 292 292 0 0.0
.bss 78769 78769 0 0.0
.bss_psram 24768 24768 0 0.0
.comment 48 48 0 0.0
.data 4648 4648 0 0.0
.debug_abbrev 1703908 1703908 0 0.0
.debug_aranges 142304 142304 0 0.0
.debug_frame 520800 520800 0 0.0
.debug_info 45402319 45402319 0 0.0
.debug_line 5704695 5704695 0 0.0
.debug_loc 3632905 3632905 0 0.0
.debug_ranges 398376 398376 0 0.0
.debug_str 3997864 3997864 0 0.0
.hbn 536 536 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 156 156 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 120992 119888 -1104 -0.9
.rsvd 2960 2960 0 0.0
.sha_ocram 72 72 0 0.0
.shstrtab 304 304 0 0.0
.stack 2048 2048 0 0.0
.strtab 634432 634432 0 0.0
.symtab 192368 192368 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3358 3358 0 0.0
.text 1033758 1033758 0 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 682215 680695 -1520 -0.2
(read/write) 169040 170560 1520 0.9
.bss 80948 80948 0 0.0
.data 3352 3352 0 0.0
.rodata 88535 87015 -1520 -1.7
.text 593364 593364 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 645567 643967 -1600 -0.2
(read/write) 157616 157616 0 0.0
.bss 80148 80148 0 0.0
.data 3352 3352 0 0.0
.rodata 78407 76807 -1600 -2.0
.text 566840 566840 0 0.0
lock-ftd LP_CC2652R7 (read only) 679167 678071 -1096 -0.2
(read/write) 169552 170648 1096 0.6
.bss 78500 78500 0 0.0
.data 3316 3316 0 0.0
.rodata 76783 75687 -1096 -1.4
.text 601904 601904 0 0.0
lock-mtd LP_CC2652R7 (read only) 665387 664299 -1088 -0.2
(read/write) 178492 179580 1088 0.6
.bss 73660 73660 0 0.0
.data 3316 3316 0 0.0
.rodata 103595 102507 -1088 -1.1
.text 561312 561312 0 0.0
pump-app LP_CC2652R7 (read only) 691979 690747 -1232 -0.2
(read/write) 157484 158716 1232 0.8
.bss 78476 78476 0 0.0
.data 3280 3280 0 0.0
.rodata 90979 89747 -1232 -1.4
.text 600520 600520 0 0.0
pump-controller-app LP_CC2652R7 (read only) 676987 675643 -1344 -0.2
(read/write) 172612 173956 1344 0.8
.bss 78612 78612 0 0.0
.data 3304 3304 0 0.0
.rodata 86843 85499 -1344 -1.5
.text 589664 589664 0 0.0
shell LP_CC2652R7 (read only) 673678 673558 -120 -0.0
(read/write) 179648 179768 120 0.1
.bss 83020 83020 0 0.0
.data 3348 3348 0 0.0
.rodata 85438 85310 -128 -0.1
.text 587928 587936 8 0.0
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 644409 642745 -1664 -0.3
(read/write) 203848 203848 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197248 197248 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 933099 933099 0 0.0
.debug_aranges 87608 87608 0 0.0
.debug_frame 301328 301328 0 0.0
.debug_info 20302987 20302987 0 0.0
.debug_line 2679748 2679748 0 0.0
.debug_loc 2824679 2824679 0 0.0
.debug_ranges 286200 286200 0 0.0
.debug_str 3039406 3039406 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105953 104289 -1664 -1.6
.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 377422 377422 0 0.0
.symtab 256768 256768 0 0.0
.text 536336 536336 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 533954 533506 -448 -0.1
.app_xip_area 414524 414076 -448 -0.1
.bss 61848 61848 0 0.0
.data 740 740 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 538234 537882 -352 -0.1
.app_xip_area 413516 413164 -352 -0.1
.bss 67128 67128 0 0.0
.data 748 748 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 555246 554766 -480 -0.1
.app_xip_area 436576 436096 -480 -0.1
.bss 61136 61136 0 0.0
.data 696 696 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A+rs9116 (read/write) 1018648 1017712 -936 -0.1
.bss 178700 178700 0 0.0
.data 2128 2128 0 0.0
.text 837800 836864 -936 -0.1
lock-app BRD4161A+wf200 (read/write) 1071336 1070728 -608 -0.1
.bss 173140 173140 0 0.0
.data 2144 2144 0 0.0
.text 896032 895424 -608 -0.1
lighting-app BRD4187C (read/write) 1025948 1025324 -624 -0.1
.bss 159200 159200 0 0.0
.data 2776 2776 0 0.0
.text 839376 838752 -624 -0.1
window-app BRD4187C (read/write) 1142492 1141548 -944 -0.1
.bss 143632 143632 0 0.0
.data 2668 2668 0 0.0
.text 971596 970652 -944 -0.1
esp32 all-clusters-app c3devkit (read only) 1052034 1052034 0 0.0
(read/write) 1585994 1585466 -528 -0.0
.dram0.bss 77984 77984 0 0.0
.dram0.data 13752 13752 0 0.0
.flash.rodata 222496 221968 -528 -0.2
.flash.text 1052034 1052034 0 0.0
.iram0.text 72896 72896 0 0.0
m5stack (read only) 1102047 1102047 0 0.0
(read/write) 501947 501351 -596 -0.1
.dram0.bss 83024 83024 0 0.0
.dram0.data 34040 34040 0 0.0
.flash.rodata 251112 250516 -596 -0.2
.flash.text 1096663 1096663 0 0.0
.iram0.text 124855 124855 0 0.0
k32w contact k32w0+release (read/write) 674416 673736 -680 -0.1
.bss 65532 65532 0 0.0
.data 2224 2224 0 0.0
.text 575260 574580 -680 -0.1
light k32w0+release (read/write) 673508 672876 -632 -0.1
.bss 65196 65196 0 0.0
.data 2212 2212 0 0.0
.text 574700 574068 -632 -0.1
lock k32w0+release (read/write) 628180 627588 -592 -0.1
.bss 63268 63268 0 0.0
.data 2136 2136 0 0.0
.text 547760 547168 -592 -0.1
linux chip-tool-ipv6only arm64 (read only) 12129740 12113692 -16048 -0.1
(read/write) 742616 742616 0 0.0
.bss 34392 34392 0 0.0
.data 3008 3008 0 0.0
.data.rel.ro 684496 684496 0 0.0
.dynamic 560 560 0 0.0
.got 15504 15504 0 0.0
.init 24 24 0 0.0
.init_array 216 216 0 0.0
.rodata 582204 566156 -16048 -2.8
.text 9785268 9785268 0 0.0
thermostat-no-ble arm64 (read only) 2518364 2500492 -17872 -0.7
(read/write) 145192 145192 0 0.0
.bss 56344 56344 0 0.0
.data 1784 1784 0 0.0
.data.rel.ro 77648 77648 0 0.0
.dynamic 560 560 0 0.0
.got 5360 5360 0 0.0
.init 24 24 0 0.0
.init_array 432 432 0 0.0
.rodata 147272 129400 -17872 -12.1
.text 2108784 2108784 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2466152 2465472 -680 -0.0
.bss 215964 215964 0 0.0
.data 5880 5880 0 0.0
.text 1428796 1428116 -680 -0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read only) 4 4 0 0.0
(read/write) 1173596 1173596 0 0.0
bss 155557 155557 0 0.0
rodata 132652 132652 0 0.0
text 804616 804616 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read only) 4 4 0 0.0
(read/write) 1118828 1118828 0 0.0
bss 154713 154713 0 0.0
rodata 109468 109468 0 0.0
text 774008 774008 0 0.0
all-clusters-app nrf7002dk_nrf5340_cpuapp (read only) 4 4 0 0.0
(read/write) 1433604 1433604 0 0.0
bss 135297 135297 0 0.0
rodata 228692 228692 0 0.0
text 775536 775532 -4 -0.0
psoc6 all-clusters cy8ckit_062s2_43012 0 0 0 0.0
(read only) 840840 840840 0 0.0
(read/write) 1763828 1760444 -3384 -0.2
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 189864 189864 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2640 2640 0 0.0
.debug_abbrev 1259797 1259797 0 0.0
.debug_aranges 111816 111816 0 0.0
.debug_frame 376196 376196 0 0.0
.debug_info 28088663 28088661 -2 -0.0
.debug_line 3822069 3822069 0 0.0
.debug_loc 3723184 3723184 0 0.0
.debug_ranges 368016 368016 0 0.0
.debug_str 3550474 3550474 0 0.0
.heap 840840 840840 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 582791 582791 0 0.0
.symtab 426640 426640 0 0.0
.text 1562936 1559552 -3384 -0.2
.zero.table 8 8 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 0 0 0 0.0
(read only) 841648 841648 0 0.0
(read/write) 1704148 1700484 -3664 -0.2
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 189064 189064 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2632 2632 0 0.0
.debug_abbrev 1241661 1241661 0 0.0
.debug_aranges 110968 110968 0 0.0
.debug_frame 378280 378280 0 0.0
.debug_info 27548067 27548067 0 0.0
.debug_line 3825472 3825472 0 0.0
.debug_loc 3705254 3705254 0 0.0
.debug_ranges 365864 365864 0 0.0
.debug_str 3536400 3536400 0 0.0
.heap 841648 841648 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 541993 541993 0 0.0
.symtab 411456 411456 0 0.0
.text 1504064 1500400 -3664 -0.2
.zero.table 8 8 0 0.0
light cy8ckit_062s2_43012 0 0 0 0.0
(read only) 849912 849912 0 0.0
(read/write) 1617204 1614212 -2992 -0.2
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 180968 180968 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2464 2464 0 0.0
.debug_abbrev 1076959 1076959 0 0.0
.debug_aranges 103176 103176 0 0.0
.debug_frame 348560 348560 0 0.0
.debug_info 22906490 22906489 -1 -0.0
.debug_line 3376393 3376393 0 0.0
.debug_loc 3380061 3380061 0 0.0
.debug_ranges 324400 324400 0 0.0
.debug_str 3335079 3335079 0 0.0
.heap 849912 849912 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 477080 477080 0 0.0
.symtab 379088 379088 0 0.0
.text 1425384 1422392 -2992 -0.2
.zero.table 8 8 0 0.0
lock cy8ckit_062s2_43012 0 0 0 0.0
(read only) 822136 822136 0 0.0
(read/write) 1674116 1671364 -2752 -0.2
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 208728 208728 0 0.0
.comment 200 200 0 0.0
.copy.table 24 24 0 0.0
.cy_m0p_image 6216 6216 0 0.0
.cy_sharedmem 8 8 0 0.0
.data 2480 2480 0 0.0
.debug_abbrev 1079191 1079191 0 0.0
.debug_aranges 103576 103576 0 0.0
.debug_frame 350300 350300 0 0.0
.debug_info 23157847 23157846 -1 -0.0
.debug_line 3378118 3378118 0 0.0
.debug_loc 3403928 3403928 0 0.0
.debug_ranges 327000 327000 0 0.0
.debug_str 3353596 3353596 0 0.0
.heap 822136 822136 0 0.0
.noinit 148 148 0 0.0
.ramVectors 736 736 0 0.0
.shstrtab 288 288 0 0.0
.stab 156 156 0 0.0
.stabstr 335 335 0 0.0
.stack_dummy 4096 4096 0 0.0
.strtab 480477 480477 0 0.0
.symtab 381280 381280 0 0.0
.text 1454520 1451768 -2752 -0.2
.zero.table 8 8 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1153680 1153440 -240 -0.0
.bss 97404 97404 0 0.0
.data 852 852 0 0.0
.text 600776 600536 -240 -0.0
lock-app qpg6105+debug (read/write) 1122216 1122048 -168 -0.0
.bss 92540 92540 0 0.0
.data 856 856 0 0.0
.text 569316 569148 -168 -0.0
telink all-clusters-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 1022736 1022724 -12 -0.0
bss 98068 98068 0 0.0
text 691972 691972 0 0.0
all-clusters-minimal-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 957788 957768 -20 -0.0
bss 97120 97120 0 0.0
text 653102 653100 -2 -0.0
contact-sensor-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 861696 861676 -20 -0.0
bss 89284 89284 0 0.0
text 583222 583220 -2 -0.0
light-switch-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 877552 877548 -4 -0.0
bss 89372 89372 0 0.0
text 597418 597422 4 0.0
lighting-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 956236 956224 -12 -0.0
bss 97500 97500 0 0.0
text 663796 663794 -2 -0.0
lighting-app-factory-data tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 967588 967576 -12 -0.0
bss 101748 101748 0 0.0
text 671366 671366 0 0.0
lighting-app-rpc tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 1024408 1024396 -12 -0.0
bss 99064 99064 0 0.0
text 706900 706898 -2 -0.0
lock-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 916040 916036 -4 -0.0
bss 89936 89936 0 0.0
text 611220 611224 4 0.0
ota-requestor-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 890444 890440 -4 -0.0
bss 90296 90296 0 0.0
text 607952 607956 4 0.0
pump-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 867128 867124 -4 -0.0
bss 89400 89400 0 0.0
text 589048 589050 2 0.0
pump-controller-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 854404 854392 -12 -0.0
bss 89200 89200 0 0.0
text 579388 579386 -2 -0.0
temperature-measurement-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 850192 850180 -12 -0.0
bss 89060 89060 0 0.0
text 574756 574756 0 0.0
thermostat tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 881704 881692 -12 -0.0
bss 90764 90764 0 0.0
text 598952 598950 -2 -0.0
window-covering tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 944056 944044 -12 -0.0
bss 99172 99172 0 0.0
text 650026 650024 -2 -0.0

@github-actions
Copy link

PR #25977: Size comparison from ed5ebd2 to 395e4ca

Full report (1 build for cc32xx)
platform target config section ed5ebd2 395e4ca change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 642945 642945 0 0.0
(read/write) 203848 203848 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197248 197248 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 933129 933129 0 0.0
.debug_aranges 87616 87616 0 0.0
.debug_frame 301352 301352 0 0.0
.debug_info 20303341 20303341 0 0.0
.debug_line 2679963 2679963 0 0.0
.debug_loc 2824782 2824782 0 0.0
.debug_ranges 286208 286208 0 0.0
.debug_str 3039557 3039557 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 104353 104353 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 377533 377533 0 0.0
.symtab 256800 256800 0 0.0
.text 536472 536472 0 0.0

@github-actions
Copy link

PR #25977: Size comparison from ed5ebd2 to 2eb7b4f

Increases (1 build for cc32xx)
platform target config section ed5ebd2 2eb7b4f change % change
cc32xx lock CC3235SF_LAUNCHXL (read only) 642945 642953 8 0.0
.debug_info 20303341 20303347 6 0.0
.debug_line 2679963 2679969 6 0.0
.text 536472 536480 8 0.0
Decreases (1 build for qpg)
platform target config section ed5ebd2 2eb7b4f change % change
qpg lighting-app qpg6105+debug (read/write) 1153624 1153616 -8 -0.0
.text 600720 600712 -8 -0.0
Full report (4 builds for cc32xx, mbed, qpg)
platform target config section ed5ebd2 2eb7b4f change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 642945 642953 8 0.0
(read/write) 203848 203848 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197248 197248 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 933129 933129 0 0.0
.debug_aranges 87616 87616 0 0.0
.debug_frame 301352 301352 0 0.0
.debug_info 20303341 20303347 6 0.0
.debug_line 2679963 2679969 6 0.0
.debug_loc 2824782 2824782 0 0.0
.debug_ranges 286208 286208 0 0.0
.debug_str 3039557 3039557 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 104353 104353 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 377533 377533 0 0.0
.symtab 256800 256800 0 0.0
.text 536472 536480 8 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2466248 2466248 0 0.0
.bss 215960 215960 0 0.0
.data 5888 5888 0 0.0
.text 1428892 1428892 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1153624 1153616 -8 -0.0
.bss 97404 97404 0 0.0
.data 852 852 0 0.0
.text 600720 600712 -8 -0.0
lock-app qpg6105+debug (read/write) 1122240 1122240 0 0.0
.bss 92540 92540 0 0.0
.data 856 856 0 0.0
.text 569340 569340 0 0.0

@sharad-patil24 sharad-patil24 force-pushed the feature/coding_standard branch from 2eb7b4f to 9b8710e Compare April 11, 2023 18:17
@github-actions
Copy link

PR #25977: Size comparison from 9f4cbc6 to 9b8710e

Decreases (1 build for cc32xx)
platform target config section 9f4cbc6 9b8710e change % change
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20303347 20303346 -1 -0.0
Full report (1 build for cc32xx)
platform target config section 9f4cbc6 9b8710e change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 642953 642953 0 0.0
(read/write) 203848 203848 0 0.0
.ARM.attributes 44 44 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 197248 197248 0 0.0
.comment 194 194 0 0.0
.data 1480 1480 0 0.0
.debug_abbrev 933129 933129 0 0.0
.debug_aranges 87616 87616 0 0.0
.debug_frame 301352 301352 0 0.0
.debug_info 20303347 20303346 -1 -0.0
.debug_line 2679969 2679969 0 0.0
.debug_loc 2824782 2824782 0 0.0
.debug_ranges 286208 286208 0 0.0
.debug_str 3039557 3039557 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 104353 104353 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 377533 377533 0 0.0
.symtab 256800 256800 0 0.0
.text 536480 536480 0 0.0

@andy31415 andy31415 merged commit c6e4a16 into project-chip:master Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mkardous-silabs mkardous-silabs mkardous-silabs approved these changes

@jmartinez-silabs jmartinez-silabs jmartinez-silabs approved these changes

@andy31415 andy31415 andy31415 approved these changes

@amitnj amitnj Awaiting requested review from amitnj

@anush-apple anush-apple Awaiting requested review from anush-apple

@arkq arkq Awaiting requested review from arkq

@bzbarsky-apple bzbarsky-apple Awaiting requested review from bzbarsky-apple

@carol-apple carol-apple Awaiting requested review from carol-apple

@chrisdecenzo chrisdecenzo Awaiting requested review from chrisdecenzo

@chshu chshu Awaiting requested review from chshu

@chulspro chulspro Awaiting requested review from chulspro

@cliffamzn cliffamzn Awaiting requested review from cliffamzn

@CodeChronos928 CodeChronos928 Awaiting requested review from CodeChronos928

@Damian-Nordic Damian-Nordic Awaiting requested review from Damian-Nordic

@dhrishi dhrishi Awaiting requested review from dhrishi

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@franck-apple franck-apple Awaiting requested review from franck-apple

@gjc13 gjc13 Awaiting requested review from gjc13

@harimau-qirex harimau-qirex Awaiting requested review from harimau-qirex

@harsha-rajendran harsha-rajendran Awaiting requested review from harsha-rajendran

@hawk248 hawk248 Awaiting requested review from hawk248

@jelderton jelderton Awaiting requested review from jelderton

@jepenven-silabs jepenven-silabs Awaiting requested review from jepenven-silabs

@jmeg-sfy jmeg-sfy Awaiting requested review from jmeg-sfy

@joonhaengHeo joonhaengHeo Awaiting requested review from joonhaengHeo

@jtung-apple jtung-apple Awaiting requested review from jtung-apple

@kcoppock kcoppock Awaiting requested review from kcoppock

@kkasperczyk-no kkasperczyk-no Awaiting requested review from kkasperczyk-no

@ksperling-apple ksperling-apple Awaiting requested review from ksperling-apple

@lazarkov lazarkov Awaiting requested review from lazarkov

@lpbeliveau-silabs lpbeliveau-silabs Awaiting requested review from lpbeliveau-silabs

@LuDuda LuDuda Awaiting requested review from LuDuda

@mrjerryjohns mrjerryjohns Awaiting requested review from mrjerryjohns

@msandstedt msandstedt Awaiting requested review from msandstedt

@mspang mspang Awaiting requested review from mspang

@nivi-apple nivi-apple Awaiting requested review from nivi-apple

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@sharadb-amazon sharadb-amazon Awaiting requested review from sharadb-amazon

@tcarmelveilleux tcarmelveilleux Awaiting requested review from tcarmelveilleux

@tecimovic tecimovic Awaiting requested review from tecimovic

@tehampson tehampson Awaiting requested review from tehampson

@turon turon Awaiting requested review from turon

@vijs vijs Awaiting requested review from vijs

@vivien-apple vivien-apple Awaiting requested review from vivien-apple

@woody-apple woody-apple Awaiting requested review from woody-apple

@xylophone21 xylophone21 Awaiting requested review from xylophone21

@younghak-hwang younghak-hwang Awaiting requested review from younghak-hwang

@yunhanw-google yunhanw-google Awaiting requested review from yunhanw-google

Assignees
No one assigned
Labels
examples platform review - approved
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@sharad-patil24 @andy31415 @jmartinez-silabs @mkardous-silabs @chirag-silabs @restyled-commits