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

Apply API review fixes for MTRPersistentStorageDelegate.h #22609

Merged

Conversation

bzbarsky-apple
Copy link
Contributor

  • Rename the protocol (and header).
  • Fix some comments.

Addresses part of #22420

Issue Being Resolved

Change overview

See above.

* Rename the protocol (and header).
* Fix some comments.

Fixes project-chip#22542

Addresses part of project-chip#22420
@github-actions
Copy link

github-actions bot commented Sep 14, 2022

PR #22609: Size comparison from 8626d39 to aad9895

Increases (7 builds for bl602, bl702, cc13x2_26x2, esp32, nrfconnect, telink)
platform target config section 8626d39 aad9895 change % change
bl602 lighting-app bl602+rpc (read/write) 1458330 1458338 8 0.0
.text 1099184 1099186 2 0.0
bl702 lighting-app bl702 .debug_info 37894018 37894019 1 0.0
.text 957170 957172 2 0.0
bl702+rpc .debug_info 41800588 41800589 1 0.0
.text 1030872 1030874 2 0.0
cc13x2_26x2 lock-ftd LP_CC2652R7 (read only) 678055 678063 8 0.0
.text 600280 600288 8 0.0
esp32 all-clusters-app m5stack (read/write) 563988 563996 8 0.0
.flash.rodata 314712 314720 8 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 text 815932 815936 4 0.0
telink light-switch-app tlsr9518adk80d text 574710 574712 2 0.0
Decreases (3 builds for cc13x2_26x2, psoc6, qpg)
platform target config section 8626d39 aad9895 change % change
cc13x2_26x2 lock-ftd LP_CC2652R7 (read/write) 163680 163672 -8 -0.0
psoc6 all-clusters cy8ckit_062s2_43012 .debug_info 26813751 26813750 -1 -0.0
qpg lock-app qpg6105+debug (read/write) 1101888 1101880 -8 -0.0
.text 548988 548980 -8 -0.0
Full report (37 builds for bl602, bl702, cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
platform target config section 8626d39 aad9895 change % change
bl602 lighting-app bl602 (read/write) 1412862 1412862 0 0.0
.bss 121585 121585 0 0.0
.data 9824 9824 0 0.0
.text 1067580 1067580 0 0.0
bl602+rpc (read/write) 1458330 1458338 8 0.0
.bss 129017 129017 0 0.0
.data 10208 10208 0 0.0
.text 1099184 1099186 2 0.0
bl702 lighting-app bl702 (read only) 3262 3262 0 0.0
(read/write) 1188427 1188427 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 66958 66958 0 0.0
.bss_psram 29696 29696 0 0.0
.comment 48 48 0 0.0
.data 4280 4280 0 0.0
.debug_abbrev 1506731 1506731 0 0.0
.debug_aranges 133072 133072 0 0.0
.debug_frame 486348 486348 0 0.0
.debug_info 37894018 37894019 1 0.0
.debug_line 5253486 5253486 0 0.0
.debug_loc 3363590 3363590 0 0.0
.debug_ranges 360256 360256 0 0.0
.debug_str 3455308 3455308 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 116504 116504 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 564895 564895 0 0.0
.symtab 171616 171616 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 0 0 0 0.0
957170 957172 2 0.0
bl702+rpc (read only) 3262 3262 0 0.0
(read/write) 1284339 1284339 0 0.0
.bleromro 6296 6296 0 0.0
.bleromrw 124 124 0 0.0
.boot2 688 688 0 0.0
.bss 75006 75006 0 0.0
.bss_psram 29936 29936 0 0.0
.comment 48 48 0 0.0
.data 4800 4800 0 0.0
.debug_abbrev 1644310 1644310 0 0.0
.debug_aranges 140576 140576 0 0.0
.debug_frame 511724 511724 0 0.0
.debug_info 41800588 41800589 1 0.0
.debug_line 5628025 5628025 0 0.0
.debug_loc 3556258 3556258 0 0.0
.debug_ranges 382712 382712 0 0.0
.debug_str 3851275 3851275 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 129896 129896 0 0.0
.rsvd 3188 3188 0 0.0
.shstrtab 293 293 0 0.0
.stack 2048 2048 0 0.0
.strtab 624068 624068 0 0.0
.symtab 189424 189424 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3262 3262 0 0.0
.text 0 0 0 0.0
1030872 1030874 2 0.0
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 676939 676939 0 0.0
(read/write) 174484 174484 0 0.0
.bss 74316 74316 0 0.0
.data 3380 3380 0 0.0
.rodata 89451 89451 0 0.0
.text 587176 587176 0 0.0
all-clusters-minimal-app LP_CC2652R7 (read only) 641651 641651 0 0.0
(read/write) 157884 157884 0 0.0
.bss 73588 73588 0 0.0
.data 3380 3380 0 0.0
.rodata 78595 78595 0 0.0
.text 562736 562736 0 0.0
lock-ftd LP_CC2652R7 (read only) 678055 678063 8 0.0
(read/write) 163680 163672 -8 -0.0
.bss 71516 71516 0 0.0
.data 3304 3304 0 0.0
.rodata 77295 77295 0 0.0
.text 600280 600288 8 0.0
lock-mtd LP_CC2652R7 (read only) 661883 661883 0 0.0
(read/write) 175540 175540 0 0.0
.bss 67204 67204 0 0.0
.data 3304 3304 0 0.0
.rodata 103131 103131 0 0.0
.text 558272 558272 0 0.0
pump-app LP_CC2652R7 (read only) 687291 687291 0 0.0
(read/write) 155148 155148 0 0.0
.bss 71452 71452 0 0.0
.data 3296 3296 0 0.0
.rodata 90563 90563 0 0.0
.text 596244 596244 0 0.0
pump-controller-app LP_CC2652R7 (read only) 671783 671783 0 0.0
(read/write) 170768 170768 0 0.0
.bss 71564 71564 0 0.0
.data 3292 3292 0 0.0
.rodata 86119 86119 0 0.0
.text 585184 585184 0 0.0
shell LP_CC2652R7 (read only) 667982 667982 0 0.0
(read/write) 178952 178952 0 0.0
.bss 76628 76628 0 0.0
.data 3376 3376 0 0.0
.rodata 86174 86174 0 0.0
.text 581492 581492 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 587410 587410 0 0.0
.app_xip_area 464068 464068 0 0.0
.bss 65776 65776 0 0.0
.data 744 744 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 594498 594498 0 0.0
.app_xip_area 465828 465828 0 0.0
.bss 71096 71096 0 0.0
.data 752 752 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 543482 543482 0 0.0
.app_xip_area 425164 425164 0 0.0
.bss 60784 60784 0 0.0
.data 716 716 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read/write) 1110704 1110704 0 0.0
.bss 136332 136332 0 0.0
.data 2072 2072 0 0.0
.text 972280 972280 0 0.0
BRD4161A+rpc (read/write) 973588 973588 0 0.0
.bss 150844 150844 0 0.0
.data 2252 2252 0 0.0
.text 820472 820472 0 0.0
BRD4161A+rs911x (read/write) 1003968 1003968 0 0.0
.bss 169168 169168 0 0.0
.data 2064 2064 0 0.0
.text 832716 832716 0 0.0
lock-app BRD4161A+wf200 (read/write) 1151476 1151476 0 0.0
.bss 152248 152248 0 0.0
.data 2072 2072 0 0.0
.text 997136 997136 0 0.0
window-app BRD4161A (read/write) 1101928 1101928 0 0.0
.bss 137772 137772 0 0.0
.data 2096 2096 0 0.0
.text 962040 962040 0 0.0
esp32 all-clusters-app c3devkit (read only) 1223856 1223856 0 0.0
(read/write) 1788102 1788102 0 0.0
.dram0.bss 76952 76952 0 0.0
.dram0.data 13840 13840 0 0.0
.flash.rodata 257656 257656 0 0.0
.flash.text 1223856 1223856 0 0.0
.iram0.text 65204 65204 0 0.0
m5stack (read only) 1233723 1233723 0 0.0
(read/write) 563988 563996 8 0.0
.dram0.bss 82312 82312 0 0.0
.dram0.data 34296 34296 0 0.0
.flash.rodata 314712 314720 8 0.0
.flash.text 1228339 1228339 0 0.0
.iram0.text 123939 123939 0 0.0
k32w light k32w0+release (read/write) 649972 649972 0 0.0
.bss 70712 70712 0 0.0
.data 2068 2068 0 0.0
.text 574464 574464 0 0.0
lock k32w0+release (read/write) 706960 706960 0 0.0
.bss 71160 71160 0 0.0
.data 2076 2076 0 0.0
.text 630996 630996 0 0.0
linux chip-tool-ipv6only arm64 (read only) 10345860 10345860 0 0.0
(read/write) 706241 706241 0 0.0
.bss 33937 33937 0 0.0
.data 2864 2864 0 0.0
.data.rel.ro 650560 650560 0 0.0
.dynamic 560 560 0 0.0
.got 13904 13904 0 0.0
.init 24 24 0 0.0
.init_array 200 200 0 0.0
.rodata 503028 503028 0 0.0
.text 8188884 8188884 0 0.0
thermostat-no-ble arm64 (read only) 2368252 2368252 0 0.0
(read/write) 142497 142497 0 0.0
.bss 55233 55233 0 0.0
.data 1688 1688 0 0.0
.data.rel.ro 76704 76704 0 0.0
.dynamic 560 560 0 0.0
.got 5096 5096 0 0.0
.init 24 24 0 0.0
.init_array 416 416 0 0.0
.rodata 141588 141588 0 0.0
.text 1987664 1987664 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2455952 2455952 0 0.0
.bss 215044 215044 0 0.0
.data 5872 5872 0 0.0
.text 1418596 1418596 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1182723 1182723 0 0.0
bss 143641 143641 0 0.0
rodata 144212 144212 0 0.0
text 815932 815936 4 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read/write) 1161923 1161923 0 0.0
bss 142868 142868 0 0.0
rodata 135804 135804 0 0.0
text 804328 804328 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 (read only) 841960 841960 0 0.0
(read/write) 1744780 1744780 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 188720 188720 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 1229158 1229158 0 0.0
.debug_aranges 111800 111800 0 0.0
.debug_frame 373224 373224 0 0.0
.debug_info 26813751 26813750 -1 -0.0
.debug_line 3668978 3668978 0 0.0
.debug_loc 3582717 3582717 0 0.0
.debug_ranges 340576 340576 0 0.0
.debug_str 3439101 3439101 0 0.0
.heap 841960 841960 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 571696 571696 0 0.0
.symtab 422080 422080 0 0.0
.text 1545008 1545008 0 0.0
.zero.table 8 8 0 0.0
text 0 0 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 (read only) 842696 842696 0 0.0
(read/write) 1687972 1687972 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 187984 187984 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 1220957 1220957 0 0.0
.debug_aranges 111272 111272 0 0.0
.debug_frame 376304 376304 0 0.0
.debug_info 26550599 26550599 0 0.0
.debug_line 3689694 3689694 0 0.0
.debug_loc 3570354 3570354 0 0.0
.debug_ranges 339192 339192 0 0.0
.debug_str 3428114 3428114 0 0.0
.heap 842696 842696 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 536170 536170 0 0.0
.symtab 408672 408672 0 0.0
.text 1488936 1488936 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
light cy8ckit_062s2_43012 (read only) 850928 850928 0 0.0
(read/write) 1605236 1605236 0 0.0
.ARM.attributes 46 46 0 0.0
.ARM.exidx 8 8 0 0.0
.bss 179960 179960 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 1055116 1055116 0 0.0
.debug_aranges 103448 103448 0 0.0
.debug_frame 346572 346572 0 0.0
.debug_info 22016853 22016853 0 0.0
.debug_line 3259861 3259861 0 0.0
.debug_loc 3268724 3268724 0 0.0
.debug_ranges 304664 304664 0 0.0
.debug_str 3233368 3233368 0 0.0
.heap 850928 850928 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 469469 469469 0 0.0
.symtab 375760 375760 0 0.0
.text 1414432 1414432 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) 1642924 1642924 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 1062571 1062571 0 0.0
.debug_aranges 104120 104120 0 0.0
.debug_frame 349400 349400 0 0.0
.debug_info 22396229 22396229 0 0.0
.debug_line 3268581 3268581 0 0.0
.debug_loc 3308561 3308561 0 0.0
.debug_ranges 308008 308008 0 0.0
.debug_str 3260785 3260785 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 475672 475672 0 0.0
.symtab 378944 378944 0 0.0
.text 1447088 1447088 0 0.0
.zero.table 0 0 0 0.0
8 8 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1130876 1130876 0 0.0
.bss 106112 106112 0 0.0
.data 1028 1028 0 0.0
.text 577972 577972 0 0.0
lock-app qpg6105+debug (read/write) 1101888 1101880 -8 -0.0
.bss 102344 102344 0 0.0
.data 1032 1032 0 0.0
.text 548988 548980 -8 -0.0
telink light-switch-app tlsr9518adk80d (read/write) 813868 813868 0 0.0
bss 71372 71372 0 0.0
noinit 43488 43488 0 0.0
text 574710 574712 2 0.0
lighting-app tlsr9518adk80d (read/write) 835800 835800 0 0.0
bss 72228 72228 0 0.0
noinit 43488 43488 0 0.0
text 592850 592850 0 0.0
ota-requestor-app tlsr9518adk80d (read/write) 843916 843916 0 0.0
bss 73136 73136 0 0.0
noinit 43488 43488 0 0.0
text 599112 599112 0 0.0

@bzbarsky-apple
Copy link
Contributor Author

Fast-tracking platform-specific fix with platform owner review.

@bzbarsky-apple bzbarsky-apple merged commit 91db05d into project-chip:master Sep 14, 2022
@bzbarsky-apple bzbarsky-apple deleted the matter-storage-delegate branch September 14, 2022 12:52
isiu-apple pushed a commit to isiu-apple/connectedhomeip that referenced this pull request Sep 16, 2022
…ip#22609)

* Rename the protocol (and header).
* Fix some comments.

Fixes project-chip#22542

Addresses part of project-chip#22420
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this pull request Nov 3, 2022
This is a re-landing of PR project-chip#22609 but modified to preserve the old APIs.

* Rename the protocol (and header).
* Fix some comments.
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this pull request Nov 3, 2022
This is a re-landing of PR project-chip#22609 but modified to preserve the old APIs.

* Rename the protocol (and header).
* Fix some comments.
bzbarsky-apple added a commit that referenced this pull request Nov 3, 2022
* Apply API review fixes for MTRPersistentStorageDelegate.h.

This is a re-landing of PR #22609 but modified to preserve the old APIs.

* Rename the protocol (and header).
* Fix some comments.

* Address review comment.
adbridge pushed a commit to ARM-software/connectedhomeip that referenced this pull request Nov 18, 2022
…hip#23476)

* Apply API review fixes for MTRPersistentStorageDelegate.h.

This is a re-landing of PR project-chip#22609 but modified to preserve the old APIs.

* Rename the protocol (and header).
* Fix some comments.

* Address review comment.
adbridge pushed a commit to ARM-software/connectedhomeip that referenced this pull request Nov 18, 2022
…hip#23476)

* Apply API review fixes for MTRPersistentStorageDelegate.h.

This is a re-landing of PR project-chip#22609 but modified to preserve the old APIs.

* Rename the protocol (and header).
* Fix some comments.

* Address review comment.
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.

[Platform] Apply API review fixes for MTRPersistentStorageDelegate.h
2 participants