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

Guard raw/Ble usage with CONFIG_NETWORK_LAYER_BLE #16651

Merged
merged 3 commits into from
Mar 26, 2022
Merged

Guard raw/Ble usage with CONFIG_NETWORK_LAYER_BLE #16651

merged 3 commits into from
Mar 26, 2022

Conversation

wendythewan
Copy link
Contributor

Problem

What is being fixed? Examples:
Guard raw/Ble usage with CONFIG_NETWORK_LAYER_BLE

Change overview

Guard raw/Ble usage with CONFIG_NETWORK_LAYER_BLE

Testing

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

  • Build Matter locally

@bzbarsky-apple
Copy link
Contributor

/rebase

@boring-cyborg boring-cyborg bot added the app label Mar 25, 2022
@github-actions
Copy link

github-actions bot commented Mar 25, 2022 •
edited
Loading

PR #16651: Size comparison from a49813a to 3ac2f95

Full report (27 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section a49813a 3ac2f95 change % change
cyw30739 light cyw930739m2evb_01 (read/write) 604482 604482 0 0.0
.app_xip_area 511652 511652 0 0.0
.bss 75576 75576 0 0.0
.data 604 604 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 562270 562270 0 0.0
.app_xip_area 470968 470968 0 0.0
.bss 74080 74080 0 0.0
.data 568 568 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 574634 574634 0 0.0
.app_xip_area 473684 473684 0 0.0
.bss 83408 83408 0 0.0
.data 508 508 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 925504 925504 0 0.0
(read/write) 128688 128688 0 0.0
.bss 126688 126688 0 0.0
.data 1996 1996 0 0.0
.text 925496 925496 0 0.0
BRD4161A+rpc (read only) 954320 954320 0 0.0
(read/write) 144640 144640 0 0.0
.bss 142464 142464 0 0.0
.data 2176 2176 0 0.0
.text 954312 954312 0 0.0
window-app BRD4161A (read only) 854900 854900 0 0.0
(read/write) 126648 126648 0 0.0
.bss 124776 124776 0 0.0
.data 1872 1872 0 0.0
.text 854892 854892 0 0.0
esp32 all-clusters-app c3devkit (read only) 965630 965630 0 0.0
(read/write) 1394162 1394162 0 0.0
.dram0.bss 61992 61992 0 0.0
.dram0.data 14212 14212 0 0.0
.flash.rodata 199032 199032 0 0.0
.flash.text 965630 965630 0 0.0
.iram0.text 62016 62016 0 0.0
m5stack (read only) 1022107 1022107 0 0.0
(read/write) 461908 461908 0 0.0
.dram0.bss 67520 67520 0 0.0
.dram0.data 34024 34024 0 0.0
.flash.rodata 228528 228528 0 0.0
.flash.text 1016723 1016723 0 0.0
.iram0.text 123107 123107 0 0.0
k32w light k32w061+release (read/write) 703000 703000 0 0.0
.bss 77568 77568 0 0.0
.data 1876 1876 0 0.0
.text 617756 617756 0 0.0
lock k32w061+release (read/write) 702068 702068 0 0.0
.bss 77544 77544 0 0.0
.data 1916 1916 0 0.0
.text 616808 616808 0 0.0
linux all-clusters-app debug (read only) 2487873 2487873 0 0.0
(read/write) 143344 143344 0 0.0
.bss 57408 57408 0 0.0
.data 1280 1280 0 0.0
.data.rel.ro 78920 78920 0 0.0
.dynamic 592 592 0 0.0
.got 4176 4176 0 0.0
.init 27 27 0 0.0
.init_array 936 936 0 0.0
.rodata 212229 212229 0 0.0
.text 2111922 2111922 0 0.0
bridge-app debug+rpc (read only) 1760133 1760133 0 0.0
(read/write) 89552 89552 0 0.0
.bss 44552 44552 0 0.0
.data 1984 1984 0 0.0
.data.rel.ro 37936 37936 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 144268 144268 0 0.0
.text 1500629 1500629 0 0.0
chip-tool debug (read only) 10219085 10219085 0 0.0
(read/write) 355720 355720 0 0.0
.bss 22496 22496 0 0.0
.data 1072 1072 0 0.0
.data.rel.ro 326056 326056 0 0.0
.dynamic 608 608 0 0.0
.got 4816 4816 0 0.0
.init 27 27 0 0.0
.init_array 640 640 0 0.0
.rodata 518901 518901 0 0.0
.text 8915653 8915653 0 0.0
chip-tool-ipv6only arm64 (read only) 9836652 9836652 0 0.0
(read/write) 473889 473889 0 0.0
.bss 40769 40769 0 0.0
.data 1144 1144 0 0.0
.data.rel.ro 372712 372712 0 0.0
.dynamic 560 560 0 0.0
.got 55464 55464 0 0.0
.init 24 24 0 0.0
.init_array 192 192 0 0.0
.rodata 494020 494020 0 0.0
.text 8289012 8289012 0 0.0
door-lock-app debug (read only) 2012145 2012145 0 0.0
(read/write) 116896 116896 0 0.0
.bss 47680 47680 0 0.0
.data 992 992 0 0.0
.data.rel.ro 62840 62840 0 0.0
.dynamic 592 592 0 0.0
.got 4120 4120 0 0.0
.init 27 27 0 0.0
.init_array 664 664 0 0.0
.rodata 180956 180956 0 0.0
.text 1681506 1681506 0 0.0
lighting-app debug+rpc (read only) 2185993 2185993 0 0.0
(read/write) 123888 123888 0 0.0
.bss 48960 48960 0 0.0
.data 1472 1472 0 0.0
.data.rel.ro 67928 67928 0 0.0
.dynamic 608 608 0 0.0
.got 4168 4168 0 0.0
.init 27 27 0 0.0
.init_array 744 744 0 0.0
.rodata 174972 174972 0 0.0
.text 1851954 1851954 0 0.0
ota-provider-app debug (read only) 1953337 1953337 0 0.0
(read/write) 112688 112688 0 0.0
.bss 47616 47616 0 0.0
.data 1256 1256 0 0.0
.data.rel.ro 58184 58184 0 0.0
.dynamic 608 608 0 0.0
.got 4376 4376 0 0.0
.init 27 27 0 0.0
.init_array 616 616 0 0.0
.rodata 166699 166699 0 0.0
.text 1634898 1634898 0 0.0
ota-requestor-app debug (read only) 1980961 1980961 0 0.0
(read/write) 115912 115912 0 0.0
.bss 48544 48544 0 0.0
.data 1480 1480 0 0.0
.data.rel.ro 60440 60440 0 0.0
.dynamic 592 592 0 0.0
.got 4184 4184 0 0.0
.init 27 27 0 0.0
.init_array 640 640 0 0.0
.rodata 162820 162820 0 0.0
.text 1665394 1665394 0 0.0
shell debug (read only) 2436633 2436633 0 0.0
(read/write) 147400 147400 0 0.0
.bss 67336 67336 0 0.0
.data 800 800 0 0.0
.data.rel.ro 73568 73568 0 0.0
.dynamic 592 592 0 0.0
.got 4152 4152 0 0.0
.init 27 27 0 0.0
.init_array 920 920 0 0.0
.rodata 209682 209682 0 0.0
.text 2070082 2070082 0 0.0
thermostat-no-ble arm64 (read only) 2269668 2269668 0 0.0
(read/write) 148385 148385 0 0.0
.bss 62833 62833 0 0.0
.data 1040 1040 0 0.0
.data.rel.ro 77000 77000 0 0.0
.dynamic 560 560 0 0.0
.got 4504 4504 0 0.0
.init 24 24 0 0.0
.init_array 360 360 0 0.0
.rodata 139716 139716 0 0.0
.text 1906544 1906544 0 0.0
tv-app debug (read only) 2685465 2685465 0 0.0
(read/write) 247712 247712 0 0.0
.bss 164352 164352 0 0.0
.data 3136 3136 0 0.0
.data.rel.ro 74160 74160 0 0.0
.dynamic 592 592 0 0.0
.got 4552 4552 0 0.0
.init 27 27 0 0.0
.init_array 888 888 0 0.0
.rodata 206027 206027 0 0.0
.text 2304546 2304546 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2354708 2354708 0 0.0
.bss 184572 184572 0 0.0
.data 5760 5760 0 0.0
.text 1317308 1317308 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1140943 1140943 0 0.0
bss 142500 142500 0 0.0
rodata 142040 142040 0 0.0
text 781524 781524 0 0.0
p6 all-clusters-app default (read/write) 2496728 2496728 0 0.0
.bss 117992 117992 0 0.0
.data 2640 2640 0 0.0
.text 1454992 1454992 0 0.0
light-app default (read/write) 2399104 2399104 0 0.0
.bss 111464 111464 0 0.0
.data 2496 2496 0 0.0
.text 1357368 1357368 0 0.0
lock-app default (read/write) 2362632 2362632 0 0.0
.bss 111208 111208 0 0.0
.data 2456 2456 0 0.0
.text 1320896 1320896 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 897470 897470 0 0.0
bss 87356 87356 0 0.0
noinit 37160 37160 0 0.0
text 634874 634874 0 0.0

@github-actions
Copy link

github-actions bot commented Mar 25, 2022 •
edited
Loading

PR #16651: Size comparison from a49813a to 9b23a11

Decreases (8 builds for cyw30739, efr32, k32w, linux, nrfconnect)
platform target config section a49813a 9b23a11 change % change
cyw30739 ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 574634 574618 -16 -0.0
.app_xip_area 473684 473668 -16 -0.0
efr32 lighting-app BRD4161A (read only) 925504 925488 -16 -0.0
.text 925496 925480 -16 -0.0
BRD4161A+rpc (read only) 954320 954304 -16 -0.0
.text 954312 954296 -16 -0.0
k32w light k32w061+release (read/write) 703000 702984 -16 -0.0
.text 617756 617740 -16 -0.0
linux chip-tool debug (read only) 10219085 10195021 -24064 -0.2
.text 8915653 8891589 -24064 -0.3
ota-requestor-app debug (read only) 1980961 1980929 -32 -0.0
.text 1665394 1665362 -32 -0.0
tv-app debug (read only) 2685465 2682961 -2504 -0.1
(read/write) 247712 247552 -160 -0.1
.bss 164352 164320 -32 -0.0
.data.rel.ro 74160 74048 -112 -0.2
.rodata 206027 205611 -416 -0.2
.text 2304546 2302722 -1824 -0.1
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1140943 1140927 -16 -0.0
text 781524 781512 -12 -0.0
Full report (25 builds for cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section a49813a 9b23a11 change % change
cyw30739 light cyw930739m2evb_01 (read/write) 604482 604482 0 0.0
.app_xip_area 511652 511652 0 0.0
.bss 75576 75576 0 0.0
.data 604 604 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 562270 562270 0 0.0
.app_xip_area 470968 470968 0 0.0
.bss 74080 74080 0 0.0
.data 568 568 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 574634 574618 -16 -0.0
.app_xip_area 473684 473668 -16 -0.0
.bss 83408 83408 0 0.0
.data 508 508 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 925504 925488 -16 -0.0
(read/write) 128688 128688 0 0.0
.bss 126688 126688 0 0.0
.data 1996 1996 0 0.0
.text 925496 925480 -16 -0.0
BRD4161A+rpc (read only) 954320 954304 -16 -0.0
(read/write) 144640 144640 0 0.0
.bss 142464 142464 0 0.0
.data 2176 2176 0 0.0
.text 954312 954296 -16 -0.0
window-app BRD4161A (read only) 854900 854900 0 0.0
(read/write) 126648 126648 0 0.0
.bss 124776 124776 0 0.0
.data 1872 1872 0 0.0
.text 854892 854892 0 0.0
esp32 all-clusters-app c3devkit (read only) 965630 965630 0 0.0
(read/write) 1394162 1394162 0 0.0
.dram0.bss 61992 61992 0 0.0
.dram0.data 14212 14212 0 0.0
.flash.rodata 199032 199032 0 0.0
.flash.text 965630 965630 0 0.0
.iram0.text 62016 62016 0 0.0
m5stack (read only) 1022107 1022107 0 0.0
(read/write) 461908 461908 0 0.0
.dram0.bss 67520 67520 0 0.0
.dram0.data 34024 34024 0 0.0
.flash.rodata 228528 228528 0 0.0
.flash.text 1016723 1016723 0 0.0
.iram0.text 123107 123107 0 0.0
k32w light k32w061+release (read/write) 703000 702984 -16 -0.0
.bss 77568 77568 0 0.0
.data 1876 1876 0 0.0
.text 617756 617740 -16 -0.0
lock k32w061+release (read/write) 702068 702068 0 0.0
.bss 77544 77544 0 0.0
.data 1916 1916 0 0.0
.text 616808 616808 0 0.0
linux all-clusters-app debug (read only) 2487873 2487873 0 0.0
(read/write) 143344 143344 0 0.0
.bss 57408 57408 0 0.0
.data 1280 1280 0 0.0
.data.rel.ro 78920 78920 0 0.0
.dynamic 592 592 0 0.0
.got 4176 4176 0 0.0
.init 27 27 0 0.0
.init_array 936 936 0 0.0
.rodata 212229 212229 0 0.0
.text 2111922 2111922 0 0.0
bridge-app debug+rpc (read only) 1760133 1760133 0 0.0
(read/write) 89552 89552 0 0.0
.bss 44552 44552 0 0.0
.data 1984 1984 0 0.0
.data.rel.ro 37936 37936 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 144268 144268 0 0.0
.text 1500629 1500629 0 0.0
chip-tool debug (read only) 10219085 10195021 -24064 -0.2
(read/write) 355720 355720 0 0.0
.bss 22496 22496 0 0.0
.data 1072 1072 0 0.0
.data.rel.ro 326056 326056 0 0.0
.dynamic 608 608 0 0.0
.got 4816 4816 0 0.0
.init 27 27 0 0.0
.init_array 640 640 0 0.0
.rodata 518901 518901 0 0.0
.text 8915653 8891589 -24064 -0.3
door-lock-app debug (read only) 2012145 2012145 0 0.0
(read/write) 116896 116896 0 0.0
.bss 47680 47680 0 0.0
.data 992 992 0 0.0
.data.rel.ro 62840 62840 0 0.0
.dynamic 592 592 0 0.0
.got 4120 4120 0 0.0
.init 27 27 0 0.0
.init_array 664 664 0 0.0
.rodata 180956 180956 0 0.0
.text 1681506 1681506 0 0.0
lighting-app debug+rpc (read only) 2185993 2185993 0 0.0
(read/write) 123888 123888 0 0.0
.bss 48960 48960 0 0.0
.data 1472 1472 0 0.0
.data.rel.ro 67928 67928 0 0.0
.dynamic 608 608 0 0.0
.got 4168 4168 0 0.0
.init 27 27 0 0.0
.init_array 744 744 0 0.0
.rodata 174972 174972 0 0.0
.text 1851954 1851954 0 0.0
ota-provider-app debug (read only) 1953337 1953337 0 0.0
(read/write) 112688 112688 0 0.0
.bss 47616 47616 0 0.0
.data 1256 1256 0 0.0
.data.rel.ro 58184 58184 0 0.0
.dynamic 608 608 0 0.0
.got 4376 4376 0 0.0
.init 27 27 0 0.0
.init_array 616 616 0 0.0
.rodata 166699 166699 0 0.0
.text 1634898 1634898 0 0.0
ota-requestor-app debug (read only) 1980961 1980929 -32 -0.0
(read/write) 115912 115912 0 0.0
.bss 48544 48544 0 0.0
.data 1480 1480 0 0.0
.data.rel.ro 60440 60440 0 0.0
.dynamic 592 592 0 0.0
.got 4184 4184 0 0.0
.init 27 27 0 0.0
.init_array 640 640 0 0.0
.rodata 162820 162820 0 0.0
.text 1665394 1665362 -32 -0.0
shell debug (read only) 2436633 2436633 0 0.0
(read/write) 147400 147400 0 0.0
.bss 67336 67336 0 0.0
.data 800 800 0 0.0
.data.rel.ro 73568 73568 0 0.0
.dynamic 592 592 0 0.0
.got 4152 4152 0 0.0
.init 27 27 0 0.0
.init_array 920 920 0 0.0
.rodata 209682 209682 0 0.0
.text 2070082 2070082 0 0.0
tv-app debug (read only) 2685465 2682961 -2504 -0.1
(read/write) 247712 247552 -160 -0.1
.bss 164352 164320 -32 -0.0
.data 3136 3136 0 0.0
.data.rel.ro 74160 74048 -112 -0.2
.dynamic 592 592 0 0.0
.got 4552 4552 0 0.0
.init 27 27 0 0.0
.init_array 888 888 0 0.0
.rodata 206027 205611 -416 -0.2
.text 2304546 2302722 -1824 -0.1
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2354708 2354708 0 0.0
.bss 184572 184572 0 0.0
.data 5760 5760 0 0.0
.text 1317308 1317308 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1140943 1140927 -16 -0.0
bss 142500 142500 0 0.0
rodata 142040 142040 0 0.0
text 781524 781512 -12 -0.0
p6 all-clusters-app default (read/write) 2496728 2496728 0 0.0
.bss 117992 117992 0 0.0
.data 2640 2640 0 0.0
.text 1454992 1454992 0 0.0
light-app default (read/write) 2399104 2399104 0 0.0
.bss 111464 111464 0 0.0
.data 2496 2496 0 0.0
.text 1357368 1357368 0 0.0
lock-app default (read/write) 2362632 2362632 0 0.0
.bss 111208 111208 0 0.0
.data 2456 2456 0 0.0
.text 1320896 1320896 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 897470 897470 0 0.0
bss 87356 87356 0 0.0
noinit 37160 37160 0 0.0
text 634874 634874 0 0.0

@woody-apple woody-apple merged commit 4479621 into project-chip:master Mar 26, 2022
rochaferraz pushed a commit to rochaferraz/connectedhomeip that referenced this pull request Mar 31, 2022
@wendythewan @rochaferraz
Guard raw/Ble usage with CONFIG_NETWORK_LAYER_BLE (project-chip#16651)
d309939 
* Guard raw/Ble usage with CONFIG_NETWORK_LAYER_BLE

* Guard raw/Ble usage with CONFIG_NETWORK_LAYER_BLE
andrei-menzopol pushed a commit to andrei-menzopol/connectedhomeip that referenced this pull request Apr 14, 2022
@wendythewan @andrei-menzopol
Guard raw/Ble usage with CONFIG_NETWORK_LAYER_BLE (project-chip#16651)
e591e89 
* Guard raw/Ble usage with CONFIG_NETWORK_LAYER_BLE

* Guard raw/Ble usage with CONFIG_NETWORK_LAYER_BLE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Damian-Nordic Damian-Nordic Damian-Nordic approved these changes

@bzbarsky-apple bzbarsky-apple bzbarsky-apple approved these changes

@msandstedt msandstedt msandstedt approved these changes

@Byungjoo-Lee Byungjoo-Lee Awaiting requested review from Byungjoo-Lee

@cecille cecille Awaiting requested review from cecille

@chrisdecenzo chrisdecenzo Awaiting requested review from chrisdecenzo

@chshu chshu Awaiting requested review from chshu

@chulspro chulspro Awaiting requested review from chulspro

@dhrishi dhrishi Awaiting requested review from dhrishi

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@erjiaqing erjiaqing Awaiting requested review from erjiaqing

@gjc13 gjc13 Awaiting requested review from gjc13

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

@hawk248 hawk248 Awaiting requested review from hawk248

@holbrookt holbrookt Awaiting requested review from holbrookt

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

@jelderton jelderton Awaiting requested review from jelderton

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

@jmartinez-silabs jmartinez-silabs Awaiting requested review from jmartinez-silabs

@kghost kghost Awaiting requested review from kghost

@kpschoedel kpschoedel Awaiting requested review from kpschoedel

@lazarkov lazarkov Awaiting requested review from lazarkov

@LuDuda LuDuda Awaiting requested review from LuDuda

@mlepage-google mlepage-google Awaiting requested review from mlepage-google

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@tecimovic tecimovic Awaiting requested review from tecimovic

@turon turon Awaiting requested review from turon

@vijs vijs Awaiting requested review from vijs

@wbschiller wbschiller Awaiting requested review from wbschiller

@xylophone21 xylophone21 Awaiting requested review from xylophone21

@yufengwangca yufengwangca Awaiting requested review from yufengwangca

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

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

Successfully merging this pull request may close these issues.
5 participants
@wendythewan @bzbarsky-apple @msandstedt @Damian-Nordic @woody-apple