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

[QPG] Use OpenThread endpoint + Re-enable check_system-includes [Issue #17404] #17548

Merged
merged 4 commits into from
Apr 21, 2022

Conversation

dvdm-qorvo
Copy link
Contributor

Problem

Change overview

  • System include testing is activated again
  • Remove LwIP usage. Use OpenThread endpoint implementation instead.

Testing

How was this tested? (at least one bullet point required)

  • Manual build test
  • Manual functional testing of the lighting and shell application

@github-actions
Copy link

github-actions bot commented Apr 20, 2022

PR #17548: Size comparison from 66333a9 to 52207ee

Full report (32 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section 66333a9 52207ee change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 684595 684595 0 0.0
(read/write) 168308 168308 0 0.0
.bss 75888 75888 0 0.0
.data 3380 3380 0 0.0
.rodata 103323 103323 0 0.0
.text 580792 580792 0 0.0
lock-ftd LP_CC2652R7 (read only) 676947 676947 0 0.0
(read/write) 166924 166924 0 0.0
.bss 73912 73912 0 0.0
.data 3212 3212 0 0.0
.rodata 99027 99027 0 0.0
.text 577436 577436 0 0.0
lock-mtd LP_CC2652R7 (read only) 625691 625691 0 0.0
(read/write) 146700 146700 0 0.0
.bss 69632 69632 0 0.0
.data 3212 3212 0 0.0
.rodata 98907 98907 0 0.0
.text 526292 526292 0 0.0
pump-app LP_CC2652R7 (read only) 648715 648715 0 0.0
(read/write) 152212 152212 0 0.0
.bss 74344 74344 0 0.0
.data 3244 3244 0 0.0
.rodata 75491 75491 0 0.0
.text 572736 572736 0 0.0
pump-controller-app LP_CC2652R7 (read only) 642019 642019 0 0.0
(read/write) 151880 151880 0 0.0
.bss 74048 74048 0 0.0
.data 3208 3208 0 0.0
.rodata 78771 78771 0 0.0
.text 562760 562760 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 617842 617842 0 0.0
.app_xip_area 524880 524880 0 0.0
.bss 75628 75628 0 0.0
.data 684 684 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 612534 612534 0 0.0
.app_xip_area 521052 521052 0 0.0
.bss 74180 74180 0 0.0
.data 648 648 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 565322 565322 0 0.0
.app_xip_area 464252 464252 0 0.0
.bss 83472 83472 0 0.0
.data 564 564 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 905540 905540 0 0.0
(read/write) 128628 128628 0 0.0
.bss 126592 126592 0 0.0
.data 2036 2036 0 0.0
.text 905532 905532 0 0.0
BRD4161A+rpc (read only) 939900 939900 0 0.0
(read/write) 145312 145312 0 0.0
.bss 143072 143072 0 0.0
.data 2240 2240 0 0.0
.text 939892 939892 0 0.0
window-app BRD4161A (read only) 842996 842996 0 0.0
(read/write) 126712 126712 0 0.0
.bss 124768 124768 0 0.0
.data 1944 1944 0 0.0
.text 842988 842988 0 0.0
esp32 all-clusters-app c3devkit (read only) 980922 980922 0 0.0
(read/write) 1397714 1397714 0 0.0
.dram0.bss 62352 62352 0 0.0
.dram0.data 14412 14412 0 0.0
.flash.rodata 202040 202040 0 0.0
.flash.text 980922 980922 0 0.0
.iram0.text 62020 62020 0 0.0
m5stack (read only) 1036355 1036355 0 0.0
(read/write) 465456 465456 0 0.0
.dram0.bss 67872 67872 0 0.0
.dram0.data 34152 34152 0 0.0
.flash.rodata 231596 231596 0 0.0
.flash.text 1030971 1030971 0 0.0
.iram0.text 123107 123107 0 0.0
k32w light k32w061+release (read/write) 683720 683720 0 0.0
.bss 77632 77632 0 0.0
.data 1992 1992 0 0.0
.text 598296 598296 0 0.0
lock k32w061+release (read/write) 724704 724704 0 0.0
.bss 78264 78264 0 0.0
.data 1952 1952 0 0.0
.text 638688 638688 0 0.0
linux all-clusters-app debug (read only) 2706137 2706137 0 0.0
(read/write) 149536 149536 0 0.0
.bss 59872 59872 0 0.0
.data 1920 1920 0 0.0
.data.rel.ro 81672 81672 0 0.0
.dynamic 608 608 0 0.0
.got 4448 4448 0 0.0
.init 27 27 0 0.0
.init_array 984 984 0 0.0
.rodata 233349 233349 0 0.0
.text 2297922 2297922 0 0.0
bridge-app debug+rpc (read only) 1839053 1839053 0 0.0
(read/write) 91856 91856 0 0.0
.bss 44136 44136 0 0.0
.data 2912 2912 0 0.0
.data.rel.ro 39744 39744 0 0.0
.dynamic 592 592 0 0.0
.got 3920 3920 0 0.0
.init 27 27 0 0.0
.init_array 544 544 0 0.0
.rodata 149089 149089 0 0.0
.text 1570821 1570821 0 0.0
chip-tool debug (read only) 10878005 10878005 0 0.0
(read/write) 374192 374192 0 0.0
.bss 22592 22592 0 0.0
.data 1136 1136 0 0.0
.data.rel.ro 344232 344232 0 0.0
.dynamic 624 624 0 0.0
.got 4936 4936 0 0.0
.init 27 27 0 0.0
.init_array 648 648 0 0.0
.rodata 545933 545933 0 0.0
.text 9496357 9496357 0 0.0
chip-tool-no-interactive-ipv6only arm64 (read only) 10495100 10495100 0 0.0
(read/write) 495713 495713 0 0.0
.bss 40865 40865 0 0.0
.data 1184 1184 0 0.0
.data.rel.ro 391400 391400 0 0.0
.dynamic 560 560 0 0.0
.got 58464 58464 0 0.0
.init 24 24 0 0.0
.init_array 184 184 0 0.0
.rodata 519596 519596 0 0.0
.text 8862644 8862644 0 0.0
lighting-app debug+rpc (read only) 2316233 2316233 0 0.0
(read/write) 127952 127952 0 0.0
.bss 49920 49920 0 0.0
.data 1952 1952 0 0.0
.data.rel.ro 70360 70360 0 0.0
.dynamic 608 608 0 0.0
.got 4304 4304 0 0.0
.init 27 27 0 0.0
.init_array 776 776 0 0.0
.rodata 183249 183249 0 0.0
.text 1966322 1966322 0 0.0
lock-app debug (read only) 2139897 2139897 0 0.0
(read/write) 120216 120216 0 0.0
.bss 47744 47744 0 0.0
.data 1472 1472 0 0.0
.data.rel.ro 65416 65416 0 0.0
.dynamic 592 592 0 0.0
.got 4248 4248 0 0.0
.init 27 27 0 0.0
.init_array 720 720 0 0.0
.rodata 189633 189633 0 0.0
.text 1793298 1793298 0 0.0
ota-provider-app debug (read only) 2049145 2049145 0 0.0
(read/write) 115584 115584 0 0.0
.bss 47936 47936 0 0.0
.data 1640 1640 0 0.0
.data.rel.ro 60264 60264 0 0.0
.dynamic 608 608 0 0.0
.got 4448 4448 0 0.0
.init 27 27 0 0.0
.init_array 632 632 0 0.0
.rodata 173595 173595 0 0.0
.text 1718338 1718338 0 0.0
ota-requestor-app debug (read only) 2080025 2080025 0 0.0
(read/write) 118392 118392 0 0.0
.bss 48608 48608 0 0.0
.data 1896 1896 0 0.0
.data.rel.ro 62296 62296 0 0.0
.dynamic 592 592 0 0.0
.got 4288 4288 0 0.0
.init 27 27 0 0.0
.init_array 656 656 0 0.0
.rodata 169980 169980 0 0.0
.text 1751346 1751346 0 0.0
shell debug (read only) 2533065 2533065 0 0.0
(read/write) 150280 150280 0 0.0
.bss 67272 67272 0 0.0
.data 1296 1296 0 0.0
.data.rel.ro 76000 76000 0 0.0
.dynamic 592 592 0 0.0
.got 4168 4168 0 0.0
.init 27 27 0 0.0
.init_array 920 920 0 0.0
.rodata 215442 215442 0 0.0
.text 2155666 2155666 0 0.0
thermostat-no-ble arm64 (read only) 2358260 2358260 0 0.0
(read/write) 151073 151073 0 0.0
.bss 62849 62849 0 0.0
.data 1440 1440 0 0.0
.data.rel.ro 79040 79040 0 0.0
.dynamic 560 560 0 0.0
.got 4728 4728 0 0.0
.init 24 24 0 0.0
.init_array 368 368 0 0.0
.rodata 145156 145156 0 0.0
.text 1983968 1983968 0 0.0
tv-app debug (read only) 2833705 2833705 0 0.0
(read/write) 253104 253104 0 0.0
.bss 165880 165880 0 0.0
.data 4448 4448 0 0.0
.data.rel.ro 76568 76568 0 0.0
.dynamic 592 592 0 0.0
.got 4680 4680 0 0.0
.init 27 27 0 0.0
.init_array 912 912 0 0.0
.rodata 215371 215371 0 0.0
.text 2435922 2435922 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2409748 2409748 0 0.0
.bss 184972 184972 0 0.0
.data 5840 5840 0 0.0
.text 1372348 1372348 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1164331 1164331 0 0.0
bss 136240 136240 0 0.0
rodata 147484 147484 0 0.0
text 801948 801948 0 0.0
p6 all-clusters-app default (read/write) 2517448 2517448 0 0.0
.bss 118344 118344 0 0.0
.data 2768 2768 0 0.0
.text 1475712 1475712 0 0.0
light-app default (read/write) 2416120 2416120 0 0.0
.bss 111824 111824 0 0.0
.data 2576 2576 0 0.0
.text 1374384 1374384 0 0.0
lock-app default (read/write) 2420024 2420024 0 0.0
.bss 111624 111624 0 0.0
.data 2536 2536 0 0.0
.text 1378288 1378288 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 800756 800756 0 0.0
bss 69672 69672 0 0.0
noinit 40416 40416 0 0.0
text 569896 569896 0 0.0

@github-actions
Copy link

github-actions bot commented Apr 20, 2022

PR #17548: Size comparison from bd2c21a to bc59ea4

Full report (32 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section bd2c21a bc59ea4 change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 691147 691147 0 0.0
(read/write) 161780 161780 0 0.0
.bss 75912 75912 0 0.0
.data 3396 3396 0 0.0
.rodata 106163 106163 0 0.0
.text 584504 584504 0 0.0
lock-ftd LP_CC2652R7 (read only) 677587 677587 0 0.0
(read/write) 166308 166308 0 0.0
.bss 73936 73936 0 0.0
.data 3220 3220 0 0.0
.rodata 99011 99011 0 0.0
.text 578092 578092 0 0.0
lock-mtd LP_CC2652R7 (read only) 626331 626331 0 0.0
(read/write) 146732 146732 0 0.0
.bss 69656 69656 0 0.0
.data 3220 3220 0 0.0
.rodata 98891 98891 0 0.0
.text 526948 526948 0 0.0
pump-app LP_CC2652R7 (read only) 649339 649339 0 0.0
(read/write) 152500 152500 0 0.0
.bss 74368 74368 0 0.0
.data 3252 3252 0 0.0
.rodata 75531 75531 0 0.0
.text 573320 573320 0 0.0
pump-controller-app LP_CC2652R7 (read only) 642395 642395 0 0.0
(read/write) 151912 151912 0 0.0
.bss 74072 74072 0 0.0
.data 3216 3216 0 0.0
.rodata 78715 78715 0 0.0
.text 563192 563192 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 618266 618266 0 0.0
.app_xip_area 525272 525272 0 0.0
.bss 75652 75652 0 0.0
.data 692 692 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 613230 613230 0 0.0
.app_xip_area 521716 521716 0 0.0
.bss 74204 74204 0 0.0
.data 656 656 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 565770 565770 0 0.0
.app_xip_area 464668 464668 0 0.0
.bss 83496 83496 0 0.0
.data 568 568 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 906156 906156 0 0.0
(read/write) 128660 128660 0 0.0
.bss 126616 126616 0 0.0
.data 2044 2044 0 0.0
.text 906148 906148 0 0.0
BRD4161A+rpc (read only) 940548 940548 0 0.0
(read/write) 145344 145344 0 0.0
.bss 143096 143096 0 0.0
.data 2248 2248 0 0.0
.text 940540 940540 0 0.0
window-app BRD4161A (read only) 843612 843612 0 0.0
(read/write) 126748 126748 0 0.0
.bss 124792 124792 0 0.0
.data 1956 1956 0 0.0
.text 843604 843604 0 0.0
esp32 all-clusters-app c3devkit (read only) 985210 985210 0 0.0
(read/write) 1466170 1466170 0 0.0
.dram0.bss 62376 62376 0 0.0
.dram0.data 14428 14428 0 0.0
.flash.rodata 204912 204912 0 0.0
.flash.text 985210 985210 0 0.0
.iram0.text 62020 62020 0 0.0
m5stack (read only) 1040491 1040491 0 0.0
(read/write) 468336 468336 0 0.0
.dram0.bss 67896 67896 0 0.0
.dram0.data 34168 34168 0 0.0
.flash.rodata 234436 234436 0 0.0
.flash.text 1035107 1035107 0 0.0
.iram0.text 123107 123107 0 0.0
k32w light k32w061+release (read/write) 684184 684184 0 0.0
.bss 77656 77656 0 0.0
.data 2000 2000 0 0.0
.text 598728 598728 0 0.0
lock k32w061+release (read/write) 725360 725360 0 0.0
.bss 78288 78288 0 0.0
.data 1960 1960 0 0.0
.text 639312 639312 0 0.0
linux all-clusters-app debug (read only) 2728713 2728713 0 0.0
(read/write) 149664 149664 0 0.0
.bss 59936 59936 0 0.0
.data 1984 1984 0 0.0
.data.rel.ro 81672 81672 0 0.0
.dynamic 608 608 0 0.0
.got 4448 4448 0 0.0
.init 27 27 0 0.0
.init_array 984 984 0 0.0
.rodata 233221 233221 0 0.0
.text 2320626 2320626 0 0.0
bridge-app debug+rpc (read only) 1842637 1842637 0 0.0
(read/write) 92016 92016 0 0.0
.bss 44136 44136 0 0.0
.data 3072 3072 0 0.0
.data.rel.ro 39744 39744 0 0.0
.dynamic 592 592 0 0.0
.got 3920 3920 0 0.0
.init 27 27 0 0.0
.init_array 544 544 0 0.0
.rodata 148961 148961 0 0.0
.text 1574533 1574533 0 0.0
chip-tool debug (read only) 10916205 10916205 0 0.0
(read/write) 375120 375120 0 0.0
.bss 22592 22592 0 0.0
.data 1136 1136 0 0.0
.data.rel.ro 345152 345152 0 0.0
.dynamic 624 624 0 0.0
.got 4936 4936 0 0.0
.init 27 27 0 0.0
.init_array 648 648 0 0.0
.rodata 546005 546005 0 0.0
.text 9531797 9531797 0 0.0
chip-tool-no-interactive-ipv6only arm64 (read only) 10530764 10530764 0 0.0
(read/write) 496897 496897 0 0.0
.bss 40865 40865 0 0.0
.data 1192 1192 0 0.0
.data.rel.ro 392424 392424 0 0.0
.dynamic 560 560 0 0.0
.got 58608 58608 0 0.0
.init 24 24 0 0.0
.init_array 184 184 0 0.0
.rodata 519948 519948 0 0.0
.text 8894740 8894740 0 0.0
lighting-app debug+rpc (read only) 2320089 2320089 0 0.0
(read/write) 128080 128080 0 0.0
.bss 50016 50016 0 0.0
.data 1984 1984 0 0.0
.data.rel.ro 70360 70360 0 0.0
.dynamic 608 608 0 0.0
.got 4304 4304 0 0.0
.init 27 27 0 0.0
.init_array 776 776 0 0.0
.rodata 183121 183121 0 0.0
.text 1970306 1970306 0 0.0
lock-app debug (read only) 2146585 2146585 0 0.0
(read/write) 120312 120312 0 0.0
.bss 47808 47808 0 0.0
.data 1504 1504 0 0.0
.data.rel.ro 65416 65416 0 0.0
.dynamic 592 592 0 0.0
.got 4264 4264 0 0.0
.init 27 27 0 0.0
.init_array 720 720 0 0.0
.rodata 189473 189473 0 0.0
.text 1799970 1799970 0 0.0
ota-provider-app debug (read only) 2053241 2053241 0 0.0
(read/write) 115680 115680 0 0.0
.bss 48000 48000 0 0.0
.data 1672 1672 0 0.0
.data.rel.ro 60264 60264 0 0.0
.dynamic 608 608 0 0.0
.got 4464 4464 0 0.0
.init 27 27 0 0.0
.init_array 632 632 0 0.0
.rodata 173627 173627 0 0.0
.text 1722226 1722226 0 0.0
ota-requestor-app debug (read only) 2084425 2084425 0 0.0
(read/write) 118488 118488 0 0.0
.bss 48672 48672 0 0.0
.data 1928 1928 0 0.0
.data.rel.ro 62296 62296 0 0.0
.dynamic 592 592 0 0.0
.got 4304 4304 0 0.0
.init 27 27 0 0.0
.init_array 656 656 0 0.0
.rodata 169916 169916 0 0.0
.text 1755634 1755634 0 0.0
shell debug (read only) 2555289 2555289 0 0.0
(read/write) 150408 150408 0 0.0
.bss 67336 67336 0 0.0
.data 1360 1360 0 0.0
.data.rel.ro 76000 76000 0 0.0
.dynamic 592 592 0 0.0
.got 4168 4168 0 0.0
.init 27 27 0 0.0
.init_array 920 920 0 0.0
.rodata 215442 215442 0 0.0
.text 2177890 2177890 0 0.0
thermostat-no-ble arm64 (read only) 2362044 2362044 0 0.0
(read/write) 151185 151185 0 0.0
.bss 62897 62897 0 0.0
.data 1488 1488 0 0.0
.data.rel.ro 79040 79040 0 0.0
.dynamic 560 560 0 0.0
.got 4744 4744 0 0.0
.init 24 24 0 0.0
.init_array 368 368 0 0.0
.rodata 145180 145180 0 0.0
.text 1987680 1987680 0 0.0
tv-app debug (read only) 2839073 2839073 0 0.0
(read/write) 253392 253392 0 0.0
.bss 165944 165944 0 0.0
.data 4640 4640 0 0.0
.data.rel.ro 76600 76600 0 0.0
.dynamic 592 592 0 0.0
.got 4680 4680 0 0.0
.init 27 27 0 0.0
.init_array 912 912 0 0.0
.rodata 215371 215371 0 0.0
.text 2441170 2441170 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2410308 2410308 0 0.0
.bss 184996 184996 0 0.0
.data 5848 5848 0 0.0
.text 1372908 1372908 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1170879 1170879 0 0.0
bss 136264 136264 0 0.0
rodata 150320 150320 0 0.0
text 805648 805648 0 0.0
p6 all-clusters-app default (read/write) 2524888 2524888 0 0.0
.bss 118368 118368 0 0.0
.data 2784 2784 0 0.0
.text 1483152 1483152 0 0.0
light-app default (read/write) 2416608 2416608 0 0.0
.bss 111848 111848 0 0.0
.data 2584 2584 0 0.0
.text 1374872 1374872 0 0.0
lock-app default (read/write) 2420792 2420792 0 0.0
.bss 111648 111648 0 0.0
.data 2544 2544 0 0.0
.text 1379056 1379056 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 801272 801272 0 0.0
bss 69696 69696 0 0.0
noinit 40416 40416 0 0.0
text 570414 570414 0 0.0

@Damian-Nordic Damian-Nordic merged commit 2b06717 into project-chip:master Apr 21, 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.

[QPG] restore check_system_includes to True
5 participants