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

Change type from INT16U to VendorId into the Operational Credentials Cluster definition #18210

Conversation

vivien-apple
Copy link
Contributor

Problem

src/app/zap-templates/zcl/data-model/chip/operational-credentials-cluster.xml uses INT16U instead of VendorId.

Change overview

  • Use VendorId where spec`ed.
  • Update generated code

@vivien-apple
Copy link
Contributor Author

/rebase

@vivien-apple vivien-apple force-pushed the SpecDef_UpdateOpCredsFromINT16UtoVendorId branch from aeaa11b to 927d3ba Compare May 9, 2022 16:46
@github-actions
Copy link

github-actions bot commented May 9, 2022

PR #18210: Size comparison from d85700e to 927d3ba

Increases (14 builds for cc13x2_26x2, cyw30739, esp32, k32w, linux, p6, telink)
platform target config section d85700e 927d3ba change % change
cc13x2_26x2 pump-app LP_CC2652R7 (read only) 663035 663043 8 0.0
.text 581580 581588 8 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 627586 627602 16 0.0
.app_xip_area 530176 530192 16 0.0
lock cyw930739m2evb_01 (read/write) 626474 626490 16 0.0
.app_xip_area 530520 530536 16 0.0
esp32 all-clusters-app m5stack (read only) 1056031 1056047 16 0.0
.flash.text 1050647 1050663 16 0.0
k32w lock k32w061+release (read/write) 730468 730484 16 0.0
.text 645104 645120 16 0.0
linux bridge-app debug+rpc (read only) 1894545 1895009 464 0.0
.text 1610098 1610562 464 0.0
lighting-app debug+rpc (read only) 2328625 2328817 192 0.0
.text 1976194 1976386 192 0.0
lock-app debug (read only) 2235921 2236113 192 0.0
.text 1880098 1880290 192 0.0
ota-provider-app debug (read only) 2064249 2064441 192 0.0
.text 1729890 1730082 192 0.0
thermostat-no-ble arm64 (read only) 2368620 2368764 144 0.0
.text 1992032 1992176 144 0.0
tv-app debug (read only) 2852865 2853057 192 0.0
.text 2451778 2451970 192 0.0
p6 light-app default (read/write) 2421592 2421608 16 0.0
.text 1379856 1379872 16 0.0
lock-app default (read/write) 2431096 2431112 16 0.0
.text 1389360 1389376 16 0.0
telink lighting-app tlsr9518adk80d text 572426 572430 4 0.0
Decreases (1 build for cc13x2_26x2)
platform target config section d85700e 927d3ba change % change
cc13x2_26x2 pump-app LP_CC2652R7 (read/write) 181708 181700 -8 -0.0
Full report (34 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section d85700e 927d3ba change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 690935 690935 0 0.0
(read/write) 161416 161416 0 0.0
.bss 75332 75332 0 0.0
.data 3412 3412 0 0.0
.rodata 102983 102983 0 0.0
.text 587468 587468 0 0.0
lock-ftd LP_CC2652R7 (read only) 678507 678507 0 0.0
(read/write) 164948 164948 0 0.0
.bss 73492 73492 0 0.0
.data 3236 3236 0 0.0
.rodata 94771 94771 0 0.0
.text 583256 583256 0 0.0
lock-mtd LP_CC2652R7 (read only) 627259 627259 0 0.0
(read/write) 146308 146308 0 0.0
.bss 69212 69212 0 0.0
.data 3236 3236 0 0.0
.rodata 94651 94651 0 0.0
.text 532120 532120 0 0.0
pump-app LP_CC2652R7 (read only) 663035 663043 8 0.0
(read/write) 181708 181700 -8 -0.0
.bss 73756 73756 0 0.0
.data 3268 3268 0 0.0
.rodata 80971 80971 0 0.0
.text 581580 581588 8 0.0
pump-controller-app LP_CC2652R7 (read only) 655943 655943 0 0.0
(read/write) 188600 188600 0 0.0
.bss 73812 73812 0 0.0
.data 3232 3232 0 0.0
.rodata 83911 83911 0 0.0
.text 571548 571548 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 627586 627602 16 0.0
.app_xip_area 530176 530192 16 0.0
.bss 80052 80052 0 0.0
.data 708 708 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 626474 626490 16 0.0
.app_xip_area 530520 530536 16 0.0
.bss 78628 78628 0 0.0
.data 672 672 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 575234 575234 0 0.0
.app_xip_area 469564 469564 0 0.0
.bss 88048 88048 0 0.0
.data 584 584 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 910096 910096 0 0.0
(read/write) 134520 134520 0 0.0
.bss 132456 132456 0 0.0
.data 2064 2064 0 0.0
.text 910088 910088 0 0.0
BRD4161A+rpc (read only) 944440 944440 0 0.0
(read/write) 151208 151208 0 0.0
.bss 148936 148936 0 0.0
.data 2268 2268 0 0.0
.text 944432 944432 0 0.0
BRD4161A+rs911x (read only) 787676 787676 0 0.0
(read/write) 129776 129776 0 0.0
.bss 127708 127708 0 0.0
.data 2068 2068 0 0.0
.text 787668 787668 0 0.0
lock-app BRD4161A+wf200 (read only) 946768 946768 0 0.0
(read/write) 124268 124268 0 0.0
.bss 122244 122244 0 0.0
.data 2024 2024 0 0.0
.text 946760 946760 0 0.0
window-app BRD4161A (read only) 890408 890408 0 0.0
(read/write) 134472 134472 0 0.0
.bss 132416 132416 0 0.0
.data 2052 2052 0 0.0
.text 890400 890400 0 0.0
esp32 all-clusters-app c3devkit (read only) 1001090 1001090 0 0.0
(read/write) 1475658 1475658 0 0.0
.dram0.bss 68464 68464 0 0.0
.dram0.data 14444 14444 0 0.0
.flash.rodata 208304 208304 0 0.0
.flash.text 1001090 1001090 0 0.0
.iram0.text 62020 62020 0 0.0
m5stack (read only) 1056031 1056047 16 0.0
(read/write) 478088 478088 0 0.0
.dram0.bss 73984 73984 0 0.0
.dram0.data 34184 34184 0 0.0
.flash.rodata 238084 238084 0 0.0
.flash.text 1050647 1050663 16 0.0
.iram0.text 123107 123107 0 0.0
k32w light k32w061+release (read/write) 685136 685136 0 0.0
.bss 81248 81248 0 0.0
.data 2020 2020 0 0.0
.text 600164 600164 0 0.0
lock k32w061+release (read/write) 730468 730484 16 0.0
.bss 81680 81680 0 0.0
.data 1980 1980 0 0.0
.text 645104 645120 16 0.0
linux all-clusters-app debug (read only) 2740441 2740441 0 0.0
(read/write) 174488 174488 0 0.0
.bss 83904 83904 0 0.0
.data 2064 2064 0 0.0
.data.rel.ro 82392 82392 0 0.0
.dynamic 608 608 0 0.0
.got 4464 4464 0 0.0
.init 27 27 0 0.0
.init_array 1008 1008 0 0.0
.rodata 236357 236357 0 0.0
.text 2328882 2328882 0 0.0
bridge-app debug+rpc (read only) 1894545 1895009 464 0.0
(read/write) 120984 120984 0 0.0
.bss 71520 71520 0 0.0
.data 3488 3488 0 0.0
.data.rel.ro 40648 40648 0 0.0
.dynamic 592 592 0 0.0
.got 4032 4032 0 0.0
.init 27 27 0 0.0
.init_array 688 688 0 0.0
.rodata 161593 161593 0 0.0
.text 1610098 1610562 464 0.0
chip-tool debug (read only) 9093941 9093941 0 0.0
(read/write) 576944 576944 0 0.0
.bss 22816 22816 0 0.0
.data 1136 1136 0 0.0
.data.rel.ro 546728 546728 0 0.0
.dynamic 624 624 0 0.0
.got 4952 4952 0 0.0
.init 27 27 0 0.0
.init_array 648 648 0 0.0
.rodata 468693 468693 0 0.0
.text 7301429 7301429 0 0.0
chip-tool-no-interactive-ipv6only arm64 (read only) 8914548 8914548 0 0.0
(read/write) 643121 643121 0 0.0
.bss 41105 41105 0 0.0
.data 1192 1192 0 0.0
.data.rel.ro 582024 582024 0 0.0
.dynamic 560 560 0 0.0
.got 14976 14976 0 0.0
.init 24 24 0 0.0
.init_array 184 184 0 0.0
.rodata 436164 436164 0 0.0
.text 7019108 7019108 0 0.0
lighting-app debug+rpc (read only) 2328625 2328817 192 0.0
(read/write) 151936 151936 0 0.0
.bss 73568 73568 0 0.0
.data 2048 2048 0 0.0
.data.rel.ro 70568 70568 0 0.0
.dynamic 608 608 0 0.0
.got 4320 4320 0 0.0
.init 27 27 0 0.0
.init_array 792 792 0 0.0
.rodata 185721 185721 0 0.0
.text 1976194 1976386 192 0.0
lock-app debug (read only) 2235921 2236113 192 0.0
(read/write) 146520 146520 0 0.0
.bss 72192 72192 0 0.0
.data 1568 1568 0 0.0
.data.rel.ro 67080 67080 0 0.0
.dynamic 592 592 0 0.0
.got 4312 4312 0 0.0
.init 27 27 0 0.0
.init_array 752 752 0 0.0
.rodata 195465 195465 0 0.0
.text 1880098 1880290 192 0.0
ota-provider-app debug (read only) 2064249 2064441 192 0.0
(read/write) 139632 139632 0 0.0
.bss 71680 71680 0 0.0
.data 1736 1736 0 0.0
.data.rel.ro 60440 60440 0 0.0
.dynamic 608 608 0 0.0
.got 4480 4480 0 0.0
.init 27 27 0 0.0
.init_array 648 648 0 0.0
.rodata 176467 176467 0 0.0
.text 1729890 1730082 192 0.0
ota-requestor-app debug (read only) 2095425 2095425 0 0.0
(read/write) 142440 142440 0 0.0
.bss 72320 72320 0 0.0
.data 1992 1992 0 0.0
.data.rel.ro 62504 62504 0 0.0
.dynamic 592 592 0 0.0
.got 4320 4320 0 0.0
.init 27 27 0 0.0
.init_array 672 672 0 0.0
.rodata 172796 172796 0 0.0
.text 1763250 1763250 0 0.0
shell debug (read only) 2565417 2565417 0 0.0
(read/write) 198160 198160 0 0.0
.bss 114408 114408 0 0.0
.data 1376 1376 0 0.0
.data.rel.ro 76656 76656 0 0.0
.dynamic 592 592 0 0.0
.got 4184 4184 0 0.0
.init 27 27 0 0.0
.init_array 928 928 0 0.0
.rodata 217810 217810 0 0.0
.text 2185506 2185506 0 0.0
thermostat-no-ble arm64 (read only) 2368620 2368764 144 0.0
(read/write) 175121 175121 0 0.0
.bss 86417 86417 0 0.0
.data 1520 1520 0 0.0
.data.rel.ro 79376 79376 0 0.0
.dynamic 560 560 0 0.0
.got 4768 4768 0 0.0
.init 24 24 0 0.0
.init_array 376 376 0 0.0
.rodata 146844 146844 0 0.0
.text 1992032 1992176 144 0.0
tv-app debug (read only) 2852865 2853057 192 0.0
(read/write) 277248 277248 0 0.0
.bss 189464 189464 0 0.0
.data 4672 4672 0 0.0
.data.rel.ro 76872 76872 0 0.0
.dynamic 592 592 0 0.0
.got 4696 4696 0 0.0
.init 27 27 0 0.0
.init_array 928 928 0 0.0
.rodata 218411 218411 0 0.0
.text 2451778 2451970 192 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2420084 2420084 0 0.0
.bss 205820 205820 0 0.0
.data 5872 5872 0 0.0
.text 1382684 1382684 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1179787 1179787 0 0.0
bss 139680 139680 0 0.0
rodata 151532 151532 0 0.0
text 809856 809856 0 0.0
p6 all-clusters-app default (read/write) 2531536 2531536 0 0.0
.bss 139328 139328 0 0.0
.data 2808 2808 0 0.0
.text 1489800 1489800 0 0.0
light-app default (read/write) 2421592 2421608 16 0.0
.bss 132656 132656 0 0.0
.data 2608 2608 0 0.0
.text 1379856 1379872 16 0.0
lock-app default (read/write) 2431096 2431112 16 0.0
.bss 132472 132472 0 0.0
.data 2568 2568 0 0.0
.text 1389360 1389376 16 0.0
telink lighting-app tlsr9518adk80d (read/write) 806024 806024 0 0.0
bss 72176 72176 0 0.0
noinit 40416 40416 0 0.0
text 572426 572430 4 0.0

@vivien-apple vivien-apple force-pushed the SpecDef_UpdateOpCredsFromINT16UtoVendorId branch from 927d3ba to ede10ba Compare May 11, 2022 09:12
@github-actions
Copy link

github-actions bot commented May 11, 2022

PR #18210: Size comparison from 603efe6 to ede10ba

Increases (8 builds for cc13x2_26x2, cyw30739, esp32, linux, p6, telink)
platform target config section 603efe6 ede10ba change % change
cc13x2_26x2 pump-app LP_CC2652R7 (read only) 663039 663047 8 0.0
.text 581564 581572 8 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 627606 627614 8 0.0
.app_xip_area 530196 530204 8 0.0
lock cyw930739m2evb_01 (read/write) 626526 626534 8 0.0
.app_xip_area 530572 530580 8 0.0
esp32 all-clusters-app m5stack (read only) 1055947 1055963 16 0.0
.flash.text 1050563 1050579 16 0.0
linux thermostat-no-ble arm64 (read only) 2369556 2369700 144 0.0
.text 1992800 1992944 144 0.0
p6 light-app default (read/write) 2421496 2421512 16 0.0
.text 1379760 1379776 16 0.0
lock-app default (read/write) 2431040 2431056 16 0.0
.text 1389304 1389320 16 0.0
telink lighting-app tlsr9518adk80d (read/write) 806128 806136 8 0.0
text 572504 572508 4 0.0
Decreases (1 build for cc13x2_26x2)
platform target config section 603efe6 ede10ba change % change
cc13x2_26x2 pump-app LP_CC2652R7 (read/write) 181704 181696 -8 -0.0
Full report (25 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section 603efe6 ede10ba change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 691019 691019 0 0.0
(read/write) 161332 161332 0 0.0
.bss 75332 75332 0 0.0
.data 3412 3412 0 0.0
.rodata 103083 103083 0 0.0
.text 587452 587452 0 0.0
lock-ftd LP_CC2652R7 (read only) 678543 678543 0 0.0
(read/write) 164912 164912 0 0.0
.bss 73492 73492 0 0.0
.data 3236 3236 0 0.0
.rodata 94823 94823 0 0.0
.text 583240 583240 0 0.0
lock-mtd LP_CC2652R7 (read only) 627303 627303 0 0.0
(read/write) 146308 146308 0 0.0
.bss 69212 69212 0 0.0
.data 3236 3236 0 0.0
.rodata 94711 94711 0 0.0
.text 532104 532104 0 0.0
pump-app LP_CC2652R7 (read only) 663039 663047 8 0.0
(read/write) 181704 181696 -8 -0.0
.bss 73756 73756 0 0.0
.data 3268 3268 0 0.0
.rodata 80991 80991 0 0.0
.text 581564 581572 8 0.0
pump-controller-app LP_CC2652R7 (read only) 655963 655963 0 0.0
(read/write) 188580 188580 0 0.0
.bss 73812 73812 0 0.0
.data 3232 3232 0 0.0
.rodata 83939 83939 0 0.0
.text 571540 571540 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 627606 627614 8 0.0
.app_xip_area 530196 530204 8 0.0
.bss 80052 80052 0 0.0
.data 708 708 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 626526 626534 8 0.0
.app_xip_area 530572 530580 8 0.0
.bss 78628 78628 0 0.0
.data 672 672 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 575278 575278 0 0.0
.app_xip_area 469608 469608 0 0.0
.bss 88048 88048 0 0.0
.data 584 584 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 910000 910000 0 0.0
(read/write) 134520 134520 0 0.0
.bss 132456 132456 0 0.0
.data 2064 2064 0 0.0
.text 909992 909992 0 0.0
BRD4161A+rpc (read only) 944344 944344 0 0.0
(read/write) 151208 151208 0 0.0
.bss 148936 148936 0 0.0
.data 2268 2268 0 0.0
.text 944336 944336 0 0.0
BRD4161A+rs911x (read only) 787596 787596 0 0.0
(read/write) 129776 129776 0 0.0
.bss 127708 127708 0 0.0
.data 2068 2068 0 0.0
.text 787588 787588 0 0.0
lock-app BRD4161A+wf200 (read only) 946804 946804 0 0.0
(read/write) 124268 124268 0 0.0
.bss 122244 122244 0 0.0
.data 2024 2024 0 0.0
.text 946796 946796 0 0.0
window-app BRD4161A (read only) 890304 890304 0 0.0
(read/write) 134472 134472 0 0.0
.bss 132416 132416 0 0.0
.data 2052 2052 0 0.0
.text 890296 890296 0 0.0
esp32 all-clusters-app c3devkit (read only) 1001094 1001094 0 0.0
(read/write) 1475770 1475770 0 0.0
.dram0.bss 68464 68464 0 0.0
.dram0.data 14444 14444 0 0.0
.flash.rodata 208416 208416 0 0.0
.flash.text 1001094 1001094 0 0.0
.iram0.text 62020 62020 0 0.0
m5stack (read only) 1055947 1055963 16 0.0
(read/write) 478184 478184 0 0.0
.dram0.bss 73984 73984 0 0.0
.dram0.data 34184 34184 0 0.0
.flash.rodata 238180 238180 0 0.0
.flash.text 1050563 1050579 16 0.0
.iram0.text 123107 123107 0 0.0
k32w light k32w061+release (read/write) 685148 685148 0 0.0
.bss 81248 81248 0 0.0
.data 2020 2020 0 0.0
.text 600176 600176 0 0.0
lock k32w061+release (read/write) 730528 730528 0 0.0
.bss 81680 81680 0 0.0
.data 1980 1980 0 0.0
.text 645164 645164 0 0.0
linux chip-tool-no-interactive-ipv6only arm64 (read only) 8936460 8936460 0 0.0
(read/write) 643121 643121 0 0.0
.bss 41105 41105 0 0.0
.data 1192 1192 0 0.0
.data.rel.ro 582008 582008 0 0.0
.dynamic 560 560 0 0.0
.got 14976 14976 0 0.0
.init 24 24 0 0.0
.init_array 184 184 0 0.0
.rodata 438924 438924 0 0.0
.text 7038004 7038004 0 0.0
thermostat-no-ble arm64 (read only) 2369556 2369700 144 0.0
(read/write) 175153 175153 0 0.0
.bss 86417 86417 0 0.0
.data 1520 1520 0 0.0
.data.rel.ro 79408 79408 0 0.0
.dynamic 560 560 0 0.0
.got 4768 4768 0 0.0
.init 24 24 0 0.0
.init_array 376 376 0 0.0
.rodata 146868 146868 0 0.0
.text 1992800 1992944 144 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2419956 2419956 0 0.0
.bss 205820 205820 0 0.0
.data 5872 5872 0 0.0
.text 1382556 1382556 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1179879 1179879 0 0.0
bss 139680 139680 0 0.0
rodata 151640 151640 0 0.0
text 809840 809840 0 0.0
p6 all-clusters-app default (read/write) 2531536 2531536 0 0.0
.bss 139328 139328 0 0.0
.data 2808 2808 0 0.0
.text 1489800 1489800 0 0.0
light-app default (read/write) 2421496 2421512 16 0.0
.bss 132656 132656 0 0.0
.data 2608 2608 0 0.0
.text 1379760 1379776 16 0.0
lock-app default (read/write) 2431040 2431056 16 0.0
.bss 132472 132472 0 0.0
.data 2568 2568 0 0.0
.text 1389304 1389320 16 0.0
telink lighting-app tlsr9518adk80d (read/write) 806128 806136 8 0.0
bss 72176 72176 0 0.0
noinit 40416 40416 0 0.0
text 572504 572508 4 0.0

@vivien-apple vivien-apple merged commit 401c393 into project-chip:master May 11, 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.

4 participants