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

[Telink] Ability to disable/enable PM during BLE operation #25769

Merged
merged 4 commits into from
Mar 24, 2023
Merged

[Telink] Ability to disable/enable PM during BLE operation #25769

merged 4 commits into from
Mar 24, 2023

Conversation

serhiiSalamakha
Copy link
Contributor

Change overview:

  • Ability to disable/enable PM during BLE operation
  • Remove tlsr9518adk80d.overlay

Testing:

  • Tested manually with chip-tool.

Steps:

  • Run: $ chip-tool pairing ble-thread <...>
  • Wait till success

@pullapprove pullapprove bot requested a review from franck-apple March 21, 2023 17:19
@pullapprove pullapprove bot requested a review from gjc13 March 21, 2023 17:19
@github-actions
Copy link

PR #25769: Size comparison from 1b2ef05 to 4d46400

Increases (1 build for cc13x2_26x2)
platform target config section 1b2ef05 4d46400 change % change
cc13x2_26x2 lock-mtd LP_CC2652R7 (read/write) 178140 178148 8 0.0
Decreases (5 builds for bl702, cc13x2_26x2, cc32xx, psoc6)
platform target config section 1b2ef05 4d46400 change % change
bl702 lighting-app bl702 .debug_info 40912568 40912567 -1 -0.0
.text 960262 960260 -2 -0.0
cc13x2_26x2 lock-mtd LP_CC2652R7 (read only) 665739 665731 -8 -0.0
.text 561640 561632 -8 -0.0
cc32xx lock CC3235SF_LAUNCHXL .debug_info 20239635 20239634 -1 -0.0
psoc6 light cy8ckit_062s2_43012 .debug_info 22843835 22843834 -1 -0.0
lock cy8ckit_062s2_43012 .debug_info 23095238 23095237 -1 -0.0
Full report (31 builds for bl602, bl702, cc13x2_26x2, cc32xx, cyw30739, efr32, k32w, linux, mbed, psoc6, qpg)
platform target config section 1b2ef05 4d46400 change % change
bl602 lighting-app bl602 (read/write) 1356990 1356990 0 0.0
.bss 94834 94834 0 0.0
.data 9744 9744 0 0.0
.text 1029366 1029366 0 0.0
bl602+rpc (read/write) 1402438 1402438 0 0.0
.bss 102882 102882 0 0.0
.data 10136 10136 0 0.0
.text 1060300 1060300 0 0.0
bl702 lighting-app bl702 0 0 0 0.0
(read only) 3358 3358 0 0.0
(read/write) 1189595 1189595 0 0.0
.bleromro 6342 6342 0 0.0
.bleromrw 124 124 0 0.0
.boot2 292 292 0 0.0
.bss 70113 70113 0 0.0
.bss_psram 26064 26064 0 0.0
.comment 48 48 0 0.0
.data 4072 4072 0 0.0
.debug_abbrev 1555684 1555684 0 0.0
.debug_aranges 134752 134752 0 0.0
.debug_frame 493880 493880 0 0.0
.debug_info 40912568 40912567 -1 -0.0
.debug_line 5293586 5293586 0 0.0
.debug_loc 3430620 3430620 0 0.0
.debug_ranges 374160 374160 0 0.0
.debug_str 3591724 3591724 0 0.0
.hbn 536 536 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 144 144 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 107984 107984 0 0.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 577941 577941 0 0.0
.symtab 174384 174384 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3358 3358 0 0.0
.text 960262 960260 -2 -0.0
bl702+rpc 0 0 0 0.0
(read only) 3358 3358 0 0.0
(read/write) 1281159 1281159 0 0.0
.bleromro 6342 6342 0 0.0
.bleromrw 124 124 0 0.0
.boot2 292 292 0 0.0
.bss 78161 78161 0 0.0
.bss_psram 26464 26464 0 0.0
.comment 48 48 0 0.0
.data 4616 4616 0 0.0
.debug_abbrev 1707899 1707899 0 0.0
.debug_aranges 143112 143112 0 0.0
.debug_frame 522048 522048 0 0.0
.debug_info 45562857 45562857 0 0.0
.debug_line 5700059 5700059 0 0.0
.debug_loc 3629593 3629593 0 0.0
.debug_ranges 398160 398160 0 0.0
.debug_str 3996391 3996391 0 0.0
.hbn 536 536 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 160 160 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 122288 122288 0 0.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 639751 639751 0 0.0
.symtab 193008 193008 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3358 3358 0 0.0
.text 1036280 1036280 0 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 682607 682607 0 0.0
(read/write) 168648 168648 0 0.0
.bss 80948 80948 0 0.0
.data 3352 3352 0 0.0
.rodata 88567 88567 0 0.0
.text 593724 593724 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 645911 645911 0 0.0
(read/write) 157616 157616 0 0.0
.bss 80148 80148 0 0.0
.data 3352 3352 0 0.0
.rodata 78439 78439 0 0.0
.text 567152 567152 0 0.0
lock-ftd LP_CC2652R7 (read only) 679519 679519 0 0.0
(read/write) 169200 169200 0 0.0
.bss 78500 78500 0 0.0
.data 3316 3316 0 0.0
.rodata 76815 76815 0 0.0
.text 602224 602224 0 0.0
lock-mtd LP_CC2652R7 (read only) 665739 665731 -8 -0.0
(read/write) 178140 178148 8 0.0
.bss 73660 73660 0 0.0
.data 3316 3316 0 0.0
.rodata 103619 103619 0 0.0
.text 561640 561632 -8 -0.0
pump-app LP_CC2652R7 (read only) 692291 692291 0 0.0
(read/write) 157172 157172 0 0.0
.bss 78476 78476 0 0.0
.data 3280 3280 0 0.0
.rodata 91003 91003 0 0.0
.text 600808 600808 0 0.0
pump-controller-app LP_CC2652R7 (read only) 677251 677251 0 0.0
(read/write) 172348 172348 0 0.0
.bss 78612 78612 0 0.0
.data 3304 3304 0 0.0
.rodata 86875 86875 0 0.0
.text 589896 589896 0 0.0
shell LP_CC2652R7 (read only) 674062 674062 0 0.0
(read/write) 179264 179264 0 0.0
.bss 83020 83020 0 0.0
.data 3348 3348 0 0.0
.rodata 85478 85478 0 0.0
.text 588272 588272 0 0.0
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 645825 645825 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 930286 930286 0 0.0
.debug_aranges 87400 87400 0 0.0
.debug_frame 300336 300336 0 0.0
.debug_info 20239635 20239634 -1 -0.0
.debug_line 2661958 2661958 0 0.0
.debug_loc 2806731 2806731 0 0.0
.debug_ranges 283424 283424 0 0.0
.debug_str 3027815 3027815 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105993 105993 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 380469 380469 0 0.0
.symtab 257408 257408 0 0.0
.text 537712 537712 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 589378 589378 0 0.0
.app_xip_area 465972 465972 0 0.0
.bss 65832 65832 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) 593618 593618 0 0.0
.app_xip_area 464916 464916 0 0.0
.bss 71120 71120 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) 554134 554134 0 0.0
.app_xip_area 436328 436328 0 0.0
.bss 60280 60280 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) 1017024 1017024 0 0.0
.bss 178708 178708 0 0.0
.data 2128 2128 0 0.0
.text 836168 836168 0 0.0
lock-app BRD4161A+wf200 (read/write) 1071832 1071832 0 0.0
.bss 173140 173140 0 0.0
.data 2144 2144 0 0.0
.text 896528 896528 0 0.0
lighting-app BRD4187C (read/write) 1014756 1014756 0 0.0
.bss 148336 148336 0 0.0
.data 2776 2776 0 0.0
.text 839048 839048 0 0.0
window-app BRD4187C (read/write) 1131444 1131444 0 0.0
.bss 132760 132760 0 0.0
.data 2668 2668 0 0.0
.text 971420 971420 0 0.0
k32w contact k32w0+release (read/write) 674688 674688 0 0.0
.bss 65532 65532 0 0.0
.data 2224 2224 0 0.0
.text 575532 575532 0 0.0
light k32w0+release (read/write) 673732 673732 0 0.0
.bss 65196 65196 0 0.0
.data 2212 2212 0 0.0
.text 574924 574924 0 0.0
lock k32w0+release (read/write) 628420 628420 0 0.0
.bss 63268 63268 0 0.0
.data 2136 2136 0 0.0
.text 548000 548000 0 0.0
linux chip-tool-ipv6only arm64 (read only) 12132580 12132580 0 0.0
(read/write) 742648 742648 0 0.0
.bss 34392 34392 0 0.0
.data 3008 3008 0 0.0
.data.rel.ro 684520 684520 0 0.0
.dynamic 560 560 0 0.0
.got 15512 15512 0 0.0
.init 24 24 0 0.0
.init_array 216 216 0 0.0
.rodata 585588 585588 0 0.0
.text 9784676 9784676 0 0.0
thermostat-no-ble arm64 (read only) 2524524 2524524 0 0.0
(read/write) 145240 145240 0 0.0
.bss 56344 56344 0 0.0
.data 1784 1784 0 0.0
.data.rel.ro 77696 77696 0 0.0
.dynamic 560 560 0 0.0
.got 5368 5368 0 0.0
.init 24 24 0 0.0
.init_array 432 432 0 0.0
.rodata 150904 150904 0 0.0
.text 2111232 2111232 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2468552 2468552 0 0.0
.bss 215964 215964 0 0.0
.data 5880 5880 0 0.0
.text 1431196 1431196 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 0 0 0 0.0
(read only) 840840 840840 0 0.0
(read/write) 1764188 1764188 0 0.0
.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 1253737 1253737 0 0.0
.debug_aranges 111536 111536 0 0.0
.debug_frame 374952 374952 0 0.0
.debug_info 27943578 27943578 0 0.0
.debug_line 3797454 3797454 0 0.0
.debug_loc 3704108 3704108 0 0.0
.debug_ranges 365160 365160 0 0.0
.debug_str 3537815 3537815 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 583978 583978 0 0.0
.symtab 426848 426848 0 0.0
.text 1563296 1563296 0 0.0
.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) 1704508 1704508 0 0.0
.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 1239070 1239070 0 0.0
.debug_aranges 110768 110768 0 0.0
.debug_frame 377288 377288 0 0.0
.debug_info 27489047 27489047 0 0.0
.debug_line 3804953 3804953 0 0.0
.debug_loc 3687311 3687311 0 0.0
.debug_ranges 363096 363096 0 0.0
.debug_str 3524937 3524937 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 543218 543218 0 0.0
.symtab 411680 411680 0 0.0
.text 1504424 1504424 0 0.0
.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) 1617564 1617564 0 0.0
.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 1074368 1074368 0 0.0
.debug_aranges 102976 102976 0 0.0
.debug_frame 347568 347568 0 0.0
.debug_info 22843835 22843834 -1 -0.0
.debug_line 3357397 3357397 0 0.0
.debug_loc 3362089 3362089 0 0.0
.debug_ranges 321632 321632 0 0.0
.debug_str 3323530 3323530 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 478343 478343 0 0.0
.symtab 379312 379312 0 0.0
.text 1425744 1425744 0 0.0
.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) 1674492 1674492 0 0.0
.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 1076316 1076316 0 0.0
.debug_aranges 103376 103376 0 0.0
.debug_frame 349308 349308 0 0.0
.debug_info 23095238 23095237 -1 -0.0
.debug_line 3359677 3359677 0 0.0
.debug_loc 3386004 3386004 0 0.0
.debug_ranges 324232 324232 0 0.0
.debug_str 3342041 3342041 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 481740 481740 0 0.0
.symtab 381504 381504 0 0.0
.text 1454896 1454896 0 0.0
.zero.table 8 8 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1152888 1152888 0 0.0
.bss 96036 96036 0 0.0
.data 852 852 0 0.0
.text 599984 599984 0 0.0
lock-app qpg6105+debug (read/write) 1121448 1121448 0 0.0
.bss 91172 91172 0 0.0
.data 856 856 0 0.0
.text 568548 568548 0 0.0

@github-actions
Copy link

PR #25769: Size comparison from a4a407b to 301af5d

Full report (1 build for cc32xx)
platform target config section a4a407b 301af5d change % change
cc32xx lock CC3235SF_LAUNCHXL 0 0 0 0.0
(read only) 645825 645825 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 930266 930266 0 0.0
.debug_aranges 87376 87376 0 0.0
.debug_frame 300288 300288 0 0.0
.debug_info 20229260 20229260 0 0.0
.debug_line 2661728 2661728 0 0.0
.debug_loc 2806516 2806516 0 0.0
.debug_ranges 283400 283400 0 0.0
.debug_str 3028246 3028246 0 0.0
.ramVecs 780 780 0 0.0
.resetVecs 64 64 0 0.0
.rodata 105993 105993 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 380469 380469 0 0.0
.symtab 257408 257408 0 0.0
.text 537712 537712 0 0.0

@andy31415 andy31415 merged commit bfc3000 into project-chip:master Mar 24, 2023
@s07641069 s07641069 deleted the telink_matter_pm_update branch March 27, 2023 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andy31415 andy31415 andy31415 approved these changes

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

@jmartinez-silabs jmartinez-silabs jmartinez-silabs 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

@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

@robszewczyk robszewczyk Awaiting requested review from robszewczyk

@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

@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

@yufengwangca yufengwangca Awaiting requested review from yufengwangca

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

Successfully merging this pull request may close these issues.
4 participants
@serhiiSalamakha @andy31415 @bzbarsky-apple @jmartinez-silabs