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

[QPG] LED handling switched from polling to event-based #23526

Merged
merged 11 commits into from
Nov 9, 2022
Merged

[QPG] LED handling switched from polling to event-based #23526

merged 11 commits into from
Nov 9, 2022

Conversation

adamb-q
Copy link
Contributor

@adamb-q adamb-q commented Nov 7, 2022

AppTask used to poll for current Thread and BLE state and on this basis was controlling the LEDs. It caused more stack usage and NVM API calls. Now the LED handling is done using events.

@adamb-q adamb-q marked this pull request as ready for review November 7, 2022 12:45
@github-actions
Copy link

github-actions bot commented Nov 8, 2022 •
edited
Loading

PR #23526: Size comparison from 93efa99 to 23cedb3

Increases (1 build for esp32)
platform target config section 93efa99 23cedb3 change % change
esp32 all-clusters-app c3devkit (read only) 1224238 1224240 2 0.0
.flash.text 1224238 1224240 2 0.0
Decreases (9 builds for bl702, cc13x2_26x2, esp32, psoc6, qpg, telink)
platform target config section 93efa99 23cedb3 change % change
bl702 lighting-app bl702 .debug_info 38074143 38074142 -1 -0.0
.text 957112 957110 -2 -0.0
cc13x2_26x2 all-clusters-minimal-app LP_CC2652R7 (read only) 641667 641659 -8 -0.0
.text 562664 562656 -8 -0.0
esp32 all-clusters-app c3devkit (read/write) 1788526 1788518 -8 -0.0
.flash.rodata 258096 258088 -8 -0.0
m5stack (read/write) 564340 564332 -8 -0.0
.flash.rodata 315056 315048 -8 -0.0
psoc6 all-clusters-minimal cy8ckit_062s2_43012 .debug_info 26760327 26760326 -1 -0.0
light cy8ckit_062s2_43012 .debug_info 22225736 22225735 -1 -0.0
qpg lighting-app qpg6105+debug (read/write) 1149560 1149528 -32 -0.0
.bss 110596 109572 -1024 -0.9
.text 596660 596628 -32 -0.0
telink all-clusters-app tlsr9518adk80d text 648604 648602 -2 -0.0
ota-requestor-app tlsr9518adk80d text 565484 565482 -2 -0.0
Full report (51 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
platform target config section 93efa99 23cedb3 change % change
bl602 lighting-app bl602 (read/write) 1388982 1388982 0 0.0
.bss 90729 90729 0 0.0
.data 9936 9936 0 0.0
.text 1068756 1068756 0 0.0
bl602+rpc (read/write) 1434430 1434430 0 0.0
.bss 98161 98161 0 0.0
.data 10320 10320 0 0.0
.text 1100360 1100360 0 0.0
bl702 lighting-app bl702 (read only) 3262 3262 0 0.0
(read/write) 1188179 1188179 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 67126 67126 0 0.0
.bss_psram 29696 29696 0 0.0
.comment 48 48 0 0.0
.data 4280 4280 0 0.0
.debug_abbrev 1514432 1514432 0 0.0
.debug_aranges 132032 132032 0 0.0
.debug_frame 484600 484600 0 0.0
.debug_info 38074143 38074142 -1 -0.0
.debug_line 5271016 5271016 0 0.0
.debug_loc 3373259 3373259 0 0.0
.debug_ranges 359736 359736 0 0.0
.debug_str 3447472 3447472 0 0.0
.hbn 509 509 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 116152 116152 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 565752 565752 0 0.0
.symtab 171984 171984 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 0 0 0 0.0
957112 957110 -2 -0.0
bl702+rpc (read only) 3262 3262 0 0.0
(read/write) 1284123 1284123 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 75158 75158 0 0.0
.bss_psram 29936 29936 0 0.0
.comment 48 48 0 0.0
.data 4816 4816 0 0.0
.debug_abbrev 1651848 1651848 0 0.0
.debug_aranges 139536 139536 0 0.0
.debug_frame 509972 509972 0 0.0
.debug_info 4199138 4199138 0 0.0
.debug_line 5647163 5647163 0 0.0
.debug_loc 3566043 3566043 0 0.0
.debug_ranges 382536 382536 0 0.0
.debug_str 3843622 3843622 0 0.0
.hbn 509 509 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 129560 129560 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 624925 624925 0 0.0
.symtab 189792 189792 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 0 0 0 0.0
1030818 1030818 0 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 677403 677403 0 0.0
(read/write) 174140 174140 0 0.0
.bss 81236 81236 0 0.0
.data 3380 3380 0 0.0
.rodata 89547 89547 0 0.0
.text 587544 587544 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 641667 641659 -8 -0.0
(read/write) 158004 158004 0 0.0
.bss 80508 80508 0 0.0
.data 3380 3380 0 0.0
.rodata 78683 78683 0 0.0
.text 562664 562656 -8 -0.0
lock-ftd LP_CC2652R7 (read only) 676167 676167 0 0.0
(read/write) 172512 172512 0 0.0
.bss 78460 78460 0 0.0
.data 3304 3304 0 0.0
.rodata 77071 77071 0 0.0
.text 598616 598616 0 0.0
lock-mtd LP_CC2652R7 (read only) 659955 659955 0 0.0
(read/write) 184412 184412 0 0.0
.bss 74148 74148 0 0.0
.data 3304 3304 0 0.0
.rodata 102923 102923 0 0.0
.text 556552 556552 0 0.0
pump-app LP_CC2652R7 (read only) 688631 688631 0 0.0
(read/write) 160784 160784 0 0.0
.bss 78428 78428 0 0.0
.data 3296 3296 0 0.0
.rodata 90559 90559 0 0.0
.text 597588 597588 0 0.0
pump-controller-app LP_CC2652R7 (read only) 672915 672915 0 0.0
(read/write) 176596 176596 0 0.0
.bss 78524 78524 0 0.0
.data 3292 3292 0 0.0
.rodata 86107 86107 0 0.0
.text 586328 586328 0 0.0
shell LP_CC2652R7 (read only) 668638 668638 0 0.0
(read/write) 185224 185224 0 0.0
.bss 83556 83556 0 0.0
.data 3376 3376 0 0.0
.rodata 86294 86294 0 0.0
.text 582028 582028 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 586814 586814 0 0.0
.app_xip_area 463424 463424 0 0.0
.bss 65808 65808 0 0.0
.data 728 728 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 590870 590870 0 0.0
.app_xip_area 462184 462184 0 0.0
.bss 71096 71096 0 0.0
.data 736 736 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 543034 543034 0 0.0
.app_xip_area 424716 424716 0 0.0
.bss 60776 60776 0 0.0
.data 684 684 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A+rpc (read/write) 974252 974252 0 0.0
.bss 152260 152260 0 0.0
.data 2248 2248 0 0.0
.text 819724 819724 0 0.0
BRD4161A+rs911x (read/write) 1034800 1034800 0 0.0
.bss 186728 186728 0 0.0
.data 2092 2092 0 0.0
.text 845956 845956 0 0.0
BRD4187C (read/write) 1146632 1146632 0 0.0
.bss 138648 138648 0 0.0
.data 2596 2596 0 0.0
.text 980792 980792 0 0.0
lock-app BRD4161A+wf200 (read/write) 1160088 1160088 0 0.0
.bss 158256 158256 0 0.0
.data 2100 2100 0 0.0
.text 999708 999708 0 0.0
window-app BRD4187C (read/write) 1139852 1139852 0 0.0
.bss 140064 140064 0 0.0
.data 2620 2620 0 0.0
.text 972572 972572 0 0.0
esp32 all-clusters-app c3devkit (read only) 1224238 1224240 2 0.0
(read/write) 1788526 1788518 -8 -0.0
.dram0.bss 76944 76944 0 0.0
.dram0.data 13840 13840 0 0.0
.flash.rodata 258096 258088 -8 -0.0
.flash.text 1224238 1224240 2 0.0
.iram0.text 65204 65204 0 0.0
m5stack (read only) 1234123 1234123 0 0.0
(read/write) 564340 564332 -8 -0.0
.dram0.bss 82320 82320 0 0.0
.dram0.data 34296 34296 0 0.0
.flash.rodata 315056 315048 -8 -0.0
.flash.text 1228739 1228739 0 0.0
.iram0.text 123939 123939 0 0.0
k32w contact k32w0+release (read/write) 661244 661244 0 0.0
.bss 77024 77024 0 0.0
.data 2104 2104 0 0.0
.text 563004 563004 0 0.0
light k32w0+release (read/write) 671288 671288 0 0.0
.bss 74824 74824 0 0.0
.data 2060 2060 0 0.0
.text 591676 591676 0 0.0
lock k32w0+release (read/write) 632764 632764 0 0.0
.bss 75584 75584 0 0.0
.data 2080 2080 0 0.0
.text 552372 552372 0 0.0
linux all-clusters-app debug (read only) 3085025 3085025 0 0.0
(read/write) 158280 158280 0 0.0
.bss 62464 62464 0 0.0
.data 2272 2272 0 0.0
.data.rel.ro 86984 86984 0 0.0
.dynamic 608 608 0 0.0
.got 4760 4760 0 0.0
.init 27 27 0 0.0
.init_array 1184 1184 0 0.0
.rodata 280843 280843 0 0.0
.text 2620082 2620082 0 0.0
all-clusters-minimal-app debug (read only) 2910721 2910721 0 0.0
(read/write) 149712 149712 0 0.0
.bss 61664 61664 0 0.0
.data 2240 2240 0 0.0
.data.rel.ro 79352 79352 0 0.0
.dynamic 608 608 0 0.0
.got 4672 4672 0 0.0
.init 27 27 0 0.0
.init_array 1160 1160 0 0.0
.rodata 280331 280331 0 0.0
.text 2449330 2449330 0 0.0
bridge-app debug+rpc (read only) 2409945 2409945 0 0.0
(read/write) 129448 129448 0 0.0
.bss 51008 51008 0 0.0
.data 3792 3792 0 0.0
.data.rel.ro 68616 68616 0 0.0
.dynamic 608 608 0 0.0
.got 4560 4560 0 0.0
.init 27 27 0 0.0
.init_array 832 832 0 0.0
.rodata 210688 210688 0 0.0
.text 2032290 2032290 0 0.0
chip-tool debug (read only) 11079121 11079121 0 0.0
(read/write) 658304 658304 0 0.0
.bss 25912 25912 0 0.0
.data 2754 2754 0 0.0
.data.rel.ro 623040 623040 0 0.0
.dynamic 608 608 0 0.0
.got 5176 5176 0 0.0
.init 27 27 0 0.0
.init_array 768 768 0 0.0
.rodata 591837 591837 0 0.0
.text 8963972 8963972 0 0.0
chip-tool-ipv6only arm64 (read only) 10433484 10433484 0 0.0
(read/write) 706337 706337 0 0.0
.bss 33905 33905 0 0.0
.data 2768 2768 0 0.0
.data.rel.ro 650672 650672 0 0.0
.dynamic 560 560 0 0.0
.got 13912 13912 0 0.0
.init 24 24 0 0.0
.init_array 208 208 0 0.0
.rodata 518220 518220 0 0.0
.text 8259988 8259988 0 0.0
lighting-app debug+rpc (read only) 2640961 2640961 0 0.0
(read/write) 132296 132296 0 0.0
.bss 49984 49984 0 0.0
.data 2288 2288 0 0.0
.data.rel.ro 73832 73832 0 0.0
.dynamic 608 608 0 0.0
.got 4624 4624 0 0.0
.init 27 27 0 0.0
.init_array 928 928 0 0.0
.rodata 227552 227552 0 0.0
.text 2237106 2237106 0 0.0
lock-app debug (read only) 2599953 2599953 0 0.0
(read/write) 127216 127216 0 0.0
.bss 48416 48416 0 0.0
.data 1904 1904 0 0.0
.data.rel.ro 70680 70680 0 0.0
.dynamic 608 608 0 0.0
.got 4656 4656 0 0.0
.init 27 27 0 0.0
.init_array 904 904 0 0.0
.rodata 243752 243752 0 0.0
.text 2184882 2184882 0 0.0
ota-provider-app debug (read only) 2396377 2396377 0 0.0
(read/write) 120744 120744 0 0.0
.bss 47936 47936 0 0.0
.data 2128 2128 0 0.0
.data.rel.ro 64600 64600 0 0.0
.dynamic 608 608 0 0.0
.got 4672 4672 0 0.0
.init 27 27 0 0.0
.init_array 768 768 0 0.0
.rodata 216328 216328 0 0.0
.text 2011890 2011890 0 0.0
ota-requestor-app debug (read only) 2561689 2561689 0 0.0
(read/write) 129056 129056 0 0.0
.bss 50464 50464 0 0.0
.data 2496 2496 0 0.0
.data.rel.ro 69960 69960 0 0.0
.dynamic 608 608 0 0.0
.got 4664 4664 0 0.0
.init 27 27 0 0.0
.init_array 856 856 0 0.0
.rodata 223408 223408 0 0.0
.text 2161026 2161026 0 0.0
shell debug (read only) 2621153 2621153 0 0.0
(read/write) 142808 142808 0 0.0
.bss 57864 57864 0 0.0
.data 1264 1264 0 0.0
.data.rel.ro 77880 77880 0 0.0
.dynamic 608 608 0 0.0
.got 4120 4120 0 0.0
.init 27 27 0 0.0
.init_array 1032 1032 0 0.0
.rodata 239186 239186 0 0.0
.text 2222850 2222850 0 0.0
thermostat-no-ble arm64 (read only) 2392092 2392092 0 0.0
(read/write) 143777 143777 0 0.0
.bss 55393 55393 0 0.0
.data 1816 1816 0 0.0
.data.rel.ro 77328 77328 0 0.0
.dynamic 560 560 0 0.0
.got 5176 5176 0 0.0
.init 24 24 0 0.0
.init_array 440 440 0 0.0
.rodata 144724 144724 0 0.0
.text 2004000 2004000 0 0.0
tv-app debug (read only) 3254761 3254761 0 0.0
(read/write) 261800 261800 0 0.0
.bss 170648 170648 0 0.0
.data 4256 4256 0 0.0
.data.rel.ro 80224 80224 0 0.0
.dynamic 608 608 0 0.0
.got 4992 4992 0 0.0
.init 27 27 0 0.0
.init_array 1064 1064 0 0.0
.rodata 268216 268216 0 0.0
.text 2792946 2792946 0 0.0
tv-casting-app debug (read only) 5580841 5580841 0 0.0
(read/write) 161720 161720 0 0.0
.bss 52120 52120 0 0.0
.data 1936 1936 0 0.0
.data.rel.ro 101056 101056 0 0.0
.dynamic 608 608 0 0.0
.got 4904 4904 0 0.0
.init 27 27 0 0.0
.init_array 1048 1048 0 0.0
.rodata 355001 355001 0 0.0
.text 4950562 4950562 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2452192 2452192 0 0.0
.bss 215012 215012 0 0.0
.data 5872 5872 0 0.0
.text 1414836 1414836 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1187759 1187759 0 0.0
bss 145285 145285 0 0.0
rodata 144436 144436 0 0.0
text 816956 816956 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1167107 1167107 0 0.0
bss 144512 144512 0 0.0
rodata 136224 136224 0 0.0
text 805446 805446 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 (read only) 841968 841968 0 0.0
(read/write) 1746028 1746028 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 188712 188712 0 0.0
.comment 204 204 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 2664 2664 0 0.0
.debug_abbrev 1235716 1235716 0 0.0
.debug_aranges 110704 110704 0 0.0
.debug_frame 371280 371280 0 0.0
.debug_info 27023696 27023696 0 0.0
.debug_line 3685729 3685729 0 0.0
.debug_loc 3603426 3603426 0 0.0
.debug_ranges 339528 339528 0 0.0
.debug_str 3431145 3431145 0 0.0
.heap 841968 841968 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 570258 570258 0 0.0
.symtab 421424 421424 0 0.0
.text 1546264 1546264 0 0.0
.zero.table 8 8 0 0.0
text 0 0 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 (read only) 842704 842704 0 0.0
(read/write) 1688628 1688628 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 187976 187976 0 0.0
.comment 204 204 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 2664 2664 0 0.0
.debug_abbrev 1227559 1227559 0 0.0
.debug_aranges 110176 110176 0 0.0
.debug_frame 374360 374360 0 0.0
.debug_info 26760327 26760326 -1 -0.0
.debug_line 3706713 3706713 0 0.0
.debug_loc 3591063 3591063 0 0.0
.debug_ranges 338144 338144 0 0.0
.debug_str 3420165 3420165 0 0.0
.heap 842704 842704 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 534347 534347 0 0.0
.symtab 407856 407856 0 0.0
.text 1489600 1489600 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
light cy8ckit_062s2_43012 (read only) 850888 850888 0 0.0
(read/write) 1606956 1606956 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 180000 180000 0 0.0
.comment 204 204 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 2456 2456 0 0.0
.debug_abbrev 1062020 1062020 0 0.0
.debug_aranges 102376 102376 0 0.0
.debug_frame 344676 344676 0 0.0
.debug_info 22225736 22225735 -1 -0.0
.debug_line 3276055 3276055 0 0.0
.debug_loc 3288807 3288807 0 0.0
.debug_ranges 303440 303440 0 0.0
.debug_str 3225600 3225600 0 0.0
.heap 850888 850888 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 470672 470672 0 0.0
.symtab 376240 376240 0 0.0
.text 1416112 1416112 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
lock cy8ckit_062s2_43012 (read only) 845896 845896 0 0.0
(read/write) 1640732 1640732 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 184976 184976 0 0.0
.comment 204 204 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 2472 2472 0 0.0
.debug_abbrev 1063995 1063995 0 0.0
.debug_aranges 102776 102776 0 0.0
.debug_frame 346548 346548 0 0.0
.debug_info 22459151 22459151 0 0.0
.debug_line 3272949 3272949 0 0.0
.debug_loc 3316401 3316401 0 0.0
.debug_ranges 305384 305384 0 0.0
.debug_str 3245071 3245071 0 0.0
.heap 845896 845896 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 473075 473075 0 0.0
.symtab 377920 377920 0 0.0
.text 1444896 1444896 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1149560 1149528 -32 -0.0
.bss 110596 109572 -1024 -0.9
.data 848 848 0 0.0
.text 596660 596628 -32 -0.0
lock-app qpg6105+debug (read/write) 1114352 1114352 0 0.0
.bss 106372 106372 0 0.0
.data 836 836 0 0.0
.text 561448 561448 0 0.0
telink all-clusters-app tlsr9518adk80d (read/write) 936764 936764 0 0.0
bss 81616 81616 0 0.0
noinit 43440 43440 0 0.0
text 648604 648602 -2 -0.0
all-clusters-minimal-app tlsr9518adk80d (read/write) 874748 874748 0 0.0
bss 80840 80840 0 0.0
noinit 43440 43440 0 0.0
text 612574 612574 0 0.0
light-switch-app tlsr9518adk80d (read/write) 790244 790244 0 0.0
bss 72464 72464 0 0.0
noinit 43520 43520 0 0.0
text 554102 554102 0 0.0
lighting-app tlsr9518adk80d (read/write) 874460 874460 0 0.0
bss 82232 82232 0 0.0
noinit 43520 43520 0 0.0
text 624080 624080 0 0.0
ota-requestor-app tlsr9518adk80d (read/write) 804736 804736 0 0.0
bss 74036 74036 0 0.0
noinit 43520 43520 0 0.0
text 565484 565482 -2 -0.0

@andy31415 andy31415 enabled auto-merge (squash) November 8, 2022 14:16
@andy31415 andy31415 merged commit ce2f487 into project-chip:master Nov 9, 2022
@adamb-q adamb-q mentioned this pull request Nov 14, 2022
Merged
adbridge pushed a commit to ARM-software/connectedhomeip that referenced this pull request Nov 18, 2022
@adamb-q @restyled-commits @adbridge
[QPG] LED handling switched from polling to event-based (project-chip…
74cf61e 
…#23526)

* Switched to callbacks

* LEDs are behaving the same

* Cleanup

* BLE is now also handled by events

* Better variable name

* Logs removed

* switch-case is more elegant here

* Moved LED handling to separate func

* AppTask stack reduced to 2kB

* Restyled by clang-format

* Updating LEDs only on connectivity change

Co-authored-by: Restyled.io <commits@restyled.io>
adbridge pushed a commit to ARM-software/connectedhomeip that referenced this pull request Nov 18, 2022
@adamb-q @restyled-commits @adbridge
[QPG] LED handling switched from polling to event-based (project-chip…
8f9889b 
…#23526)

* Switched to callbacks

* LEDs are behaving the same

* Cleanup

* BLE is now also handled by events

* Better variable name

* Logs removed

* switch-case is more elegant here

* Moved LED handling to separate func

* AppTask stack reduced to 2kB

* Restyled by clang-format

* Updating LEDs only on connectivity change

Co-authored-by: Restyled.io <commits@restyled.io>
dvdm-qorvo pushed a commit to Qorvo/connectedhomeip that referenced this pull request Dec 21, 2022
@adamb-q @restyled-commits @dvdm-qorvo
[QPG] LED handling switched from polling to event-based (project-chip…
3a0251d 
…#23526)

* Switched to callbacks

* LEDs are behaving the same

* Cleanup

* BLE is now also handled by events

* Better variable name

* Logs removed

* switch-case is more elegant here

* Moved LED handling to separate func

* AppTask stack reduced to 2kB

* Restyled by clang-format

* Updating LEDs only on connectivity change

Co-authored-by: Restyled.io <commits@restyled.io>
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

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

@arkq arkq Awaiting requested review from arkq

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

@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

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

@dhrishi dhrishi Awaiting requested review from dhrishi

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@erjiaqing erjiaqing Awaiting requested review from erjiaqing

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

@gjc13 gjc13 Awaiting requested review from gjc13

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

@hawk248 hawk248 Awaiting requested review from hawk248

@isiu-apple isiu-apple Awaiting requested review from isiu-apple

@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

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

@kghost kghost Awaiting requested review from kghost

@lazarkov lazarkov Awaiting requested review from lazarkov

@LuDuda LuDuda Awaiting requested review from LuDuda

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

@msandstedt msandstedt Awaiting requested review from msandstedt

@mspang mspang Awaiting requested review from mspang

@rgoliver rgoliver Awaiting requested review from rgoliver

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@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

@wbschiller wbschiller Awaiting requested review from wbschiller

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

@xylophone21 xylophone21 Awaiting requested review from xylophone21

@yufengwangca yufengwangca Awaiting requested review from yufengwangca

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

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

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.
4 participants
@adamb-q @andy31415 @bzbarsky-apple @restyled-commits