-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[nrfconnect] Refactored configuration for the nrfconnect examples #16991
[nrfconnect] Refactored configuration for the nrfconnect examples #16991
Conversation
c5a7afa
to
cb2698c
Compare
PR #16991: Size comparison from a208c3d to cb2698c Full report (30 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, p6, telink)
|
cb2698c
to
619f684
Compare
PR #16991: Size comparison from 3666138 to 619f684 Increases above 0.2%:
Increases (3 builds for nrfconnect)
Decreases (1 build for nrfconnect)
Full report (41 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fast tracking platform changes
619f684
to
7f7f921
Compare
PR #16991: Size comparison from 2ff1d72 to 7f7f921 Full report (25 builds for cc13x2_26x2, efr32, k32w, linux, mbed, p6, telink)
|
7f7f921
to
cc7acde
Compare
PR #16991: Size comparison from e92157a to cc7acde Full report (27 builds for cc13x2_26x2, efr32, esp32, k32w, linux, mbed, p6, telink)
|
nrfconnect platform uses Kconfig overlays and defaults in a way that is not consistent with the nRF Connect SDK recommendations * Created Kconfig.defaults that set default configs common for all examples (replaced sample-defaults.conf) * Created Kconfig.features that allows to configure set of options within single feature using single option * Replaced approach of adding overlays in example cmake file with the pattern of placing prj.conf files in a configuration/build-target/ location that allows adding config files to the build automatically by the toolchain. * Created debug and release configuration variants for several examples.
cc7acde
to
8dee62f
Compare
PR #16991: Size comparison from 04f1acd to 8dee62f Increases above 0.2%:
Increases (1 build for nrfconnect)
Decreases (1 build for nrfconnect)
Full report (38 builds for cc13x2_26x2, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
|
…oject-chip#16991) nrfconnect platform uses Kconfig overlays and defaults in a way that is not consistent with the nRF Connect SDK recommendations * Created Kconfig.defaults that set default configs common for all examples (replaced sample-defaults.conf) * Created Kconfig.features that allows to configure set of options within single feature using single option * Replaced approach of adding overlays in example cmake file with the pattern of placing prj.conf files in a configuration/build-target/ location that allows adding config files to the build automatically by the toolchain. * Created debug and release configuration variants for several examples.
…oject-chip#16991) nrfconnect platform uses Kconfig overlays and defaults in a way that is not consistent with the nRF Connect SDK recommendations * Created Kconfig.defaults that set default configs common for all examples (replaced sample-defaults.conf) * Created Kconfig.features that allows to configure set of options within single feature using single option * Replaced approach of adding overlays in example cmake file with the pattern of placing prj.conf files in a configuration/build-target/ location that allows adding config files to the build automatically by the toolchain. * Created debug and release configuration variants for several examples.
…oject-chip#16991) nrfconnect platform uses Kconfig overlays and defaults in a way that is not consistent with the nRF Connect SDK recommendations * Created Kconfig.defaults that set default configs common for all examples (replaced sample-defaults.conf) * Created Kconfig.features that allows to configure set of options within single feature using single option * Replaced approach of adding overlays in example cmake file with the pattern of placing prj.conf files in a configuration/build-target/ location that allows adding config files to the build automatically by the toolchain. * Created debug and release configuration variants for several examples.
Problem
nrfconnect platform uses Kconfig overlays and defaults in a way that is not consistent with the nRF Connect SDK recommendations
Change overview
Testing
Tested manually builds and basic functionalities, however change should not introduce functional changes, as it's only structural refactor.