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

[NXP] Remove circular dependency on application code in platform file ConfigurationManagerImpl.cpp #30891

Merged
merged 1 commit into from
Dec 11, 2023

Conversation

dinabenamar
Copy link
Contributor

The aim of this PR is to fix #30599 layering inversion issue in 'src/platform/nxp/common/ConfigurationManagerImpl.cpp' , by removing the dependency on application code in the platform file.

Copy link

github-actions bot commented Dec 8, 2023

PR #30891: Size comparison from 1583a24 to f8fb824

Decreases (1 build for efr32)
platform target config section 1583a24 f8fb824 change % change
efr32 lock-app BRD4161A+wf200 (read/write) 1114016 1114008 -8 -0.0
.text 922660 922652 -8 -0.0
Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
platform target config section 1583a24 f8fb824 change % change
bl602 lighting-app bl602 (read/write) 1413742 1413742 0 0.0
.bss 85160 85160 0 0.0
.data 9416 9416 0 0.0
.rodata 156892 156892 0 0.0
.text 1081626 1081626 0 0.0
bl602+mfd (read/write) 1428206 1428206 0 0.0
.bss 85328 85328 0 0.0
.data 9384 9384 0 0.0
.rodata 155852 155852 0 0.0
.text 1096990 1096990 0 0.0
bl602+rpc (read/write) 1461526 1461526 0 0.0
.bss 93192 93192 0 0.0
.data 9784 9784 0 0.0
.rodata 164516 164516 0 0.0
.text 1113370 1113370 0 0.0
bl702 lighting-app bl702 (read only) 3478 3478 0 0.0
(read/write) 1178419 1178419 0 0.0
.bss 11197 11197 0 0.0
.data 3584 3584 0 0.0
.rodata 106268 106268 0 0.0
.text 950994 950994 0 0.0
bl702+mfd (read only) 3478 3478 0 0.0
(read/write) 1189295 1189295 0 0.0
.bss 11373 11373 0 0.0
.data 3568 3568 0 0.0
.rodata 105208 105208 0 0.0
.text 962802 962802 0 0.0
bl702+rpc (read only) 3478 3478 0 0.0
(read/write) 1270431 1270431 0 0.0
.bss 19677 19677 0 0.0
.data 4144 4144 0 0.0
.rodata 121688 121688 0 0.0
.text 1026056 1026056 0 0.0
bl706-eth (read/write) 998373 998373 0 0.0
.bss 23772 23772 0 0.0
.data 3160 3160 0 0.0
.rodata 99600 99600 0 0.0
.text 744218 744218 0 0.0
bl706-wifi (read/write) 1235010 1235010 0 0.0
.bss 10641 10641 0 0.0
.data 3600 3600 0 0.0
.rodata 120580 120580 0 0.0
.text 970016 970016 0 0.0
bl702l lighting-app bl702l (read/write) 1148636 1148636 0 0.0
.bss 16392 16392 0 0.0
.data 4952 4952 0 0.0
.rodata 100272 100272 0 0.0
.text 944120 944120 0 0.0
bl702l+mfd (read only) 512 512 0 0.0
(read/write) 1159576 1159576 0 0.0
.bss 16568 16568 0 0.0
.data 4928 4928 0 0.0
.rodata 99212 99212 0 0.0
.text 956496 956496 0 0.0
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 (read only) 742212 742212 0 0.0
(read/write) 167116 167116 0 0.0
.bss 89532 89532 0 0.0
.data 3420 3420 0 0.0
.rodata 84648 84648 0 0.0
.text 657176 657176 0 0.0
lock-ftd LP_EM_CC1354P10_6 (read only) 759420 759420 0 0.0
(read/write) 177344 177344 0 0.0
.bss 99708 99708 0 0.0
.data 3448 3448 0 0.0
.rodata 79728 79728 0 0.0
.text 679304 679304 0 0.0
lock-mtd LP_EM_CC1354P10_6 (read only) 746404 746404 0 0.0
(read/write) 171800 171800 0 0.0
.bss 94164 94164 0 0.0
.data 3448 3448 0 0.0
.rodata 105504 105504 0 0.0
.text 640508 640508 0 0.0
pump-app LP_EM_CC1354P10_6 (read only) 700004 700004 0 0.0
(read/write) 166120 166120 0 0.0
.bss 88312 88312 0 0.0
.data 3408 3408 0 0.0
.rodata 80440 80440 0 0.0
.text 619176 619176 0 0.0
pump-controller-app LP_EM_CC1354P10_6 (read only) 685524 685524 0 0.0
(read/write) 166292 166292 0 0.0
.bss 88488 88488 0 0.0
.data 3404 3404 0 0.0
.rodata 76280 76280 0 0.0
.text 608856 608856 0 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL (read only) 566578 566578 0 0.0
(read/write) 207620 207620 0 0.0
.bss 201084 201084 0 0.0
.data 1412 1412 0 0.0
.rodata 83922 83922 0 0.0
.text 480536 480536 0 0.0
lock CC3235SF_LAUNCHXL (read only) 619818 619818 0 0.0
(read/write) 208016 208016 0 0.0
.bss 201448 201448 0 0.0
.data 1448 1448 0 0.0
.rodata 105786 105786 0 0.0
.text 511912 511912 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 560975 560975 0 0.0
.app_xip_area 451265 451265 0 0.0
.bss 64624 64624 0 0.0
.data 660 660 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
lock cyw930739m2evb_01 (read/write) 543079 543079 0 0.0
.app_xip_area 428633 428633 0 0.0
.bss 69320 69320 0 0.0
.data 700 700 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
ota-requestor cyw930739m2evb_01 (read/write) 561951 561951 0 0.0
.app_xip_area 453697 453697 0 0.0
.bss 63240 63240 0 0.0
.data 588 588 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
switch cyw930739m2evb_01 (read/write) 557263 557263 0 0.0
.app_xip_area 445785 445785 0 0.0
.bss 66360 66360 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) 1016428 1016428 0 0.0
.bss 172568 172568 0 0.0
.data 2688 2688 0 0.0
.text 841152 841152 0 0.0
BRD4187C (read/write) 1100620 1100620 0 0.0
.bss 196172 196172 0 0.0
.data 3296 3296 0 0.0
.text 901132 901132 0 0.0
lock-app BRD4161A+wf200 (read/write) 1114016 1114008 -8 -0.0
.bss 188624 188624 0 0.0
.data 2712 2712 0 0.0
.text 922660 922652 -8 -0.0
window-app BRD4187C (read/write) 1135128 1135128 0 0.0
.bss 168204 168204 0 0.0
.data 3168 3168 0 0.0
.text 963736 963736 0 0.0
esp32 all-clusters-app c3devkit (read only) 1136024 1136024 0 0.0
(read/write) 1670548 1670548 0 0.0
.dram0.bss 72536 72536 0 0.0
.dram0.data 12988 12988 0 0.0
.flash.rodata 241152 241152 0 0.0
.flash.text 1136024 1136024 0 0.0
.iram0.text 75694 75694 0 0.0
m5stack (read only) 1181199 1181199 0 0.0
(read/write) 519744 519744 0 0.0
.dram0.bss 79192 79192 0 0.0
.dram0.data 34004 34004 0 0.0
.flash.rodata 272184 272184 0 0.0
.flash.text 1175815 1175815 0 0.0
.iram0.text 123907 123907 0 0.0
k32w contact k32w0+release (read only) 594816 594816 0 0.0
(read/write) 88128 88128 0 0.0
.bss 71376 71376 0 0.0
.data 2128 2128 0 0.0
.text 594280 594280 0 0.0
k32w1+release (read only) 1024 1024 0 0.0
(read/write) 690448 690448 0 0.0
.bss 67584 67584 0 0.0
.data 2716 2716 0 0.0
.text 576672 576672 0 0.0
light k32w0+release (read only) 594512 594512 0 0.0
(read/write) 87804 87804 0 0.0
.bss 71064 71064 0 0.0
.data 2116 2116 0 0.0
.text 593976 593976 0 0.0
k32w1+release (read only) 1024 1024 0 0.0
(read/write) 782252 782252 0 0.0
.bss 77252 77252 0 0.0
.data 1928 1928 0 0.0
.text 659616 659616 0 0.0
lock k32w0+release (read only) 565904 565904 0 0.0
(read/write) 85600 85600 0 0.0
.bss 68920 68920 0 0.0
.data 2056 2056 0 0.0
.text 565368 565368 0 0.0
linux air-purifier-app debug (read only) 2524413 2524413 0 0.0
(read/write) 127176 127176 0 0.0
.bss 46944 46944 0 0.0
.data 1472 1472 0 0.0
.data.rel.ro 72776 72776 0 0.0
.dynamic 608 608 0 0.0
.got 4568 4568 0 0.0
.init 27 27 0 0.0
.init_array 800 800 0 0.0
.rodata 179272 179272 0 0.0
.text 2169781 2169781 0 0.0
all-clusters-app debug (read only) 5365065 5365065 0 0.0
(read/write) 447712 447712 0 0.0
.bss 130064 130064 0 0.0
.data 3840 3840 0 0.0
.data.rel.ro 306584 306584 0 0.0
.dynamic 624 624 0 0.0
.got 5352 5352 0 0.0
.init 27 27 0 0.0
.init_array 1224 1224 0 0.0
.rodata 308028 308028 0 0.0
.text 4653603 4653603 0 0.0
all-clusters-minimal-app debug (read only) 4950121 4950121 0 0.0
(read/write) 234656 234656 0 0.0
.bss 127856 127856 0 0.0
.data 3776 3776 0 0.0
.data.rel.ro 95912 95912 0 0.0
.dynamic 624 624 0 0.0
.got 5312 5312 0 0.0
.init 27 27 0 0.0
.init_array 1160 1160 0 0.0
.rodata 272813 272813 0 0.0
.text 4444883 4444883 0 0.0
bridge-app debug (read only) 4415305 4415305 0 0.0
(read/write) 215896 215896 0 0.0
.bss 119224 119224 0 0.0
.data 4704 4704 0 0.0
.data.rel.ro 85232 85232 0 0.0
.dynamic 624 624 0 0.0
.got 5280 5280 0 0.0
.init 27 27 0 0.0
.init_array 824 824 0 0.0
.rodata 219125 219125 0 0.0
.text 3977651 3977651 0 0.0
chip-tool debug (read only) 11424193 11424193 0 0.0
(read/write) 496072 496072 0 0.0
.bss 94072 94072 0 0.0
.data 4994 4994 0 0.0
.data.rel.ro 389760 389760 0 0.0
.dynamic 624 624 0 0.0
.got 5784 5784 0 0.0
.init 27 27 0 0.0
.init_array 776 776 0 0.0
.rodata 418313 418313 0 0.0
.text 10328691 10328691 0 0.0
chip-tool-ipv6only arm64 (read only) 11145680 11145680 0 0.0
(read/write) 562336 562336 0 0.0
.bss 103352 103352 0 0.0
.data 4280 4280 0 0.0
.data.rel.ro 428472 428472 0 0.0
.dynamic 512 512 0 0.0
.got 19584 19584 0 0.0
.init 24 24 0 0.0
.init_array 248 248 0 0.0
.rodata 341096 341096 0 0.0
.text 9950136 9950136 0 0.0
lighting-app debug+rpc+ui (read only) 5300865 5300865 0 0.0
(read/write) 224968 224968 0 0.0
.bss 119360 119360 0 0.0
.data 4416 4416 0 0.0
.data.rel.ro 93632 93632 0 0.0
.dynamic 672 672 0 0.0
.got 5896 5896 0 0.0
.init 27 27 0 0.0
.init_array 960 960 0 0.0
.rodata 340244 340244 0 0.0
.text 4715875 4715875 0 0.0
lock-app debug (read only) 4499809 4499809 0 0.0
(read/write) 203616 203616 0 0.0
.bss 113760 113760 0 0.0
.data 3648 3648 0 0.0
.data.rel.ro 79464 79464 0 0.0
.dynamic 624 624 0 0.0
.got 5232 5232 0 0.0
.init 27 27 0 0.0
.init_array 856 856 0 0.0
.rodata 247029 247029 0 0.0
.text 4042787 4042787 0 0.0
ota-provider-app debug (read only) 4164561 4164561 0 0.0
(read/write) 192696 192696 0 0.0
.bss 113632 113632 0 0.0
.data 3904 3904 0 0.0
.data.rel.ro 69248 69248 0 0.0
.dynamic 624 624 0 0.0
.got 4544 4544 0 0.0
.init 27 27 0 0.0
.init_array 720 720 0 0.0
.rodata 201301 201301 0 0.0
.text 3771427 3771427 0 0.0
ota-requestor-app debug (read only) 4253985 4253985 0 0.0
(read/write) 196976 196976 0 0.0
.bss 114656 114656 0 0.0
.data 4176 4176 0 0.0
.data.rel.ro 72240 72240 0 0.0
.dynamic 624 624 0 0.0
.got 4480 4480 0 0.0
.init 27 27 0 0.0
.init_array 776 776 0 0.0
.rodata 205109 205109 0 0.0
.text 3852643 3852643 0 0.0
shell debug (read only) 2777793 2777793 0 0.0
(read/write) 149792 149792 0 0.0
.bss 60616 60616 0 0.0
.data 768 768 0 0.0
.data.rel.ro 82608 82608 0 0.0
.dynamic 592 592 0 0.0
.got 4104 4104 0 0.0
.init 27 27 0 0.0
.init_array 1072 1072 0 0.0
.rodata 181024 181024 0 0.0
.text 2426082 2426082 0 0.0
thermostat-no-ble arm64 (read only) 4402816 4402816 0 0.0
(read/write) 247656 247656 0 0.0
.bss 122232 122232 0 0.0
.data 2784 2784 0 0.0
.data.rel.ro 102312 102312 0 0.0
.dynamic 512 512 0 0.0
.got 12008 12008 0 0.0
.init 24 24 0 0.0
.init_array 424 424 0 0.0
.rodata 162316 162316 0 0.0
.text 3900184 3900184 0 0.0
tv-app debug (read only) 5382729 5382729 0 0.0
(read/write) 348896 348896 0 0.0
.bss 244384 244384 0 0.0
.data 4992 4992 0 0.0
.data.rel.ro 92216 92216 0 0.0
.dynamic 624 624 0 0.0
.got 5520 5520 0 0.0
.init 27 27 0 0.0
.init_array 1144 1144 0 0.0
.rodata 272853 272853 0 0.0
.text 4872067 4872067 0 0.0
tv-casting-app debug (read only) 9337033 9337033 0 0.0
(read/write) 334776 334776 0 0.0
.bss 156192 156192 0 0.0
.data 2464 2464 0 0.0
.data.rel.ro 169272 169272 0 0.0
.dynamic 624 624 0 0.0
.got 5064 5064 0 0.0
.init 27 27 0 0.0
.init_array 1136 1136 0 0.0
.rodata 359976 359976 0 0.0
.text 8521747 8521747 0 0.0
mbed lock-app-release cy8cproto_062_4343w (read only) 6224 6224 0 0.0
(read/write) 2525656 2525656 0 0.0
.bss 220184 220184 0 0.0
.data 5144 5144 0 0.0
.text 1488340 1488340 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read only) 4 4 0 0.0
(read/write) 1060780 1060780 0 0.0
bss 129029 129029 0 0.0
rodata 104172 104172 0 0.0
text 779912 779912 0 0.0
nrf7002dk_nrf5340_cpuapp (read only) 4 4 0 0.0
(read/write) 1187336 1187336 0 0.0
bss 121239 121239 0 0.0
rodata 138300 138300 0 0.0
text 777480 777480 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 (read only) 4 4 0 0.0
(read/write) 1046408 1046408 0 0.0
bss 129316 129316 0 0.0
rodata 99432 99432 0 0.0
text 769996 769996 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 (read only) 836224 836224 0 0.0
(read/write) 1741060 1741060 0 0.0
.bss 194644 194644 0 0.0
.data 2472 2472 0 0.0
.text 1535556 1535556 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 (read only) 837944 837944 0 0.0
(read/write) 1705364 1705364 0 0.0
.bss 192924 192924 0 0.0
.data 2472 2472 0 0.0
.text 1501580 1501580 0 0.0
light cy8ckit_062s2_43012 (read only) 844632 844632 0 0.0
(read/write) 1630420 1630420 0 0.0
.bss 186348 186348 0 0.0
.data 2360 2360 0 0.0
.text 1433324 1433324 0 0.0
lock cy8ckit_062s2_43012 (read only) 817512 817512 0 0.0
(read/write) 1657268 1657268 0 0.0
.bss 213444 213444 0 0.0
.data 2384 2384 0 0.0
.text 1433052 1433052 0 0.0
qpg lighting-app qpg6105+debug (read/write) 1116364 1116364 0 0.0
.bss 101200 101200 0 0.0
.data 740 740 0 0.0
.text 628740 628740 0 0.0
lock-app qpg6105+debug (read/write) 1080004 1080004 0 0.0
.bss 97336 97336 0 0.0
.data 784 784 0 0.0
.text 592384 592384 0 0.0
telink air-quality-sensor-app tlsr9528a_retention (read only) 51774 51774 0 0.0
(read/write) 819998 819998 0 0.0
bss 54904 54904 0 0.0
text 612678 612678 0 0.0
all-clusters-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 1077828 1077828 0 0.0
bss 100072 100072 0 0.0
text 760552 760552 0 0.0
all-clusters-minimal-app tlsr9528a (read only) 4 4 0 0.0
(read/write) 1083064 1083064 0 0.0
bss 109764 109764 0 0.0
text 752562 752562 0 0.0
bridge-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 933864 933864 0 0.0
bss 94272 94272 0 0.0
text 646142 646142 0 0.0
contact-sensor-app tlsr9528a_retention (read only) 51774 51774 0 0.0
(read/write) 820210 820210 0 0.0
bss 54948 54948 0 0.0
text 613024 613024 0 0.0
light-switch-app-ota-shell-factory-data tlsr9528a (read only) 4 4 0 0.0
(read/write) 977240 977240 0 0.0
bss 78932 78932 0 0.0
text 693056 693056 0 0.0
lighting-app-ota-rpc-factory-data-4mb tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 1092140 1092140 0 0.0
bss 99836 99836 0 0.0
text 768962 768962 0 0.0
lock-app-dfu tlsr9528a (read only) 4 4 0 0.0
(read/write) 963476 963476 0 0.0
bss 97196 97196 0 0.0
text 646710 646710 0 0.0
ota-requestor-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 947516 947516 0 0.0
bss 92380 92380 0 0.0
text 662232 662232 0 0.0
pump-app tlsr9528a (read only) 4 4 0 0.0
(read/write) 878992 878992 0 0.0
bss 68924 68924 0 0.0
text 621014 621014 0 0.0
pump-controller-app tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 819596 819596 0 0.0
bss 57160 57160 0 0.0
text 587890 587890 0 0.0
shell tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 705416 705416 0 0.0
bss 73588 73588 0 0.0
text 461916 461916 0 0.0
smoke_co_alarm-app tlsr9528a_retention (read only) 51774 51774 0 0.0
(read/write) 821866 821866 0 0.0
bss 55084 55084 0 0.0
text 615696 615696 0 0.0
temperature-measurement-app-mars-ota tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 876697 876697 0 0.0
bss 60636 60636 0 0.0
text 629638 629638 0 0.0
thermostat tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 843840 843840 0 0.0
bss 57456 57456 0 0.0
text 606268 606268 0 0.0
window-covering tlsr9518adk80d (read only) 4 4 0 0.0
(read/write) 838260 838260 0 0.0
bss 57808 57808 0 0.0
text 600402 600402 0 0.0

@mergify mergify bot merged commit 86b53e1 into project-chip:master Dec 11, 2023
54 checks passed
@dinabenamar dinabenamar deleted the fix-layering-inversion branch November 14, 2024 12:34
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.

[NXP] Layering inversion in ConfigurationmanagerImpl.cpp
2 participants