[nrfconnect] Update nRF Connect SDK version to 2.4.0 #27010
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
app
config
examples
github
inet
lib
nrf connect
platform
scripts
shell
system
workflows
zephyr
labels
|
PR #27010: Size comparison from 5d913c9 to d0dae75 Increases (11 builds for bl602, bl702, cc32xx, esp32, psoc6, qpg)
Decreases (2 builds for bl702)
Full report (22 builds for bl602, bl702, cc32xx, cyw30739, esp32, k32w, linux, psoc6, qpg)
|
kkasperczyk-no
force-pushed
the
ncs_2_4_0_update
branch
from
d0dae75 to
9c63da8
Compare
|
PR #27010: Size comparison from 28e0423 to 9c63da8 Increases (7 builds for bl602, bl702, cc32xx, psoc6)
Decreases (4 builds for bl602, bl702, cyw30739)
Full report (22 builds for bl602, bl702, cc32xx, cyw30739, esp32, k32w, linux, psoc6, qpg)
|
andy31415
reviewed
Jun 1, 2023
|
PR #27010: Size comparison from dc8d5eb to 20ce621 Increases (8 builds for bl602, bl702, cc32xx, psoc6, qpg)
Decreases (2 builds for bl702)
Full report (22 builds for bl602, bl702, cc32xx, cyw30739, esp32, k32w, linux, psoc6, qpg)
|
kkasperczyk-no
force-pushed
the
ncs_2_4_0_update
branch
from
20ce621 to
1569a0b
Compare
pullapprove
bot
requested review from
amitnj,
anush-apple,
arkq,
carol-apple,
chrisdecenzo,
chshu,
chulspro and
cliffamzn
|
@andy31415 could you point anybody familiar with chef app, as I can see the failures for this build? In the logs I can see |
|
@erwinpan1 maybe. @kkasperczyk-no How does nrf in the regular builds decide what compiler to use? I am wondering if this is a case of using the nrf tools vs pigweed defaults. |
|
@kkasperczyk-no I have a patch to use the Zephyr toolchain for chef (someone asked to help with that in the past but I forgot to contribute it). Ping me on Monday about that. Actually, I pushed changes to your branch. |
Bumped recommended nRF Connect SDK version to 2.4.0 and included several bug fixes and improvements: 1. Extend the connection request log with RSSI. This comes in handy especially during tests related to TX/RX power. Signed-off-by: Marcin Kajor <[email protected]> 2. Support disabling Wi-Fi LPM during the OTA DFU. Added subclass of OTAImageProcessorImpl whcih implements Wi-Fi specific optimization for LPM which should be disabled during OTA due to the high throughput requirements. This allows applications to use the OTAImageProcessorImplWiFi to do the job. Signed-off-by: Marcin Kajor <[email protected]> 3. Provide configs for Matter NUS feature Matter NUS feature allows to add of BLE Nordic UART Service and register commands which allow controlling the Matter device via BLE. This can be useful when the device loses connection with a Matter Controller but it should be controlled in another way for e.g. a Door Lock. 4. FactoryDataProvider: align the fprotect memory block Due to the smallest atomic SPU block (16k for nRF53) which can be protected we need to calculate proper offsets to include factory data partition. Signed-off-by: Marcin Kajor <[email protected]> 5. nrfconnect: migrate to new MCUmgr Kconfig options Migrate to new MCUmgr Kconfig options using: $ZEPHYR_BASE/scripts/utils/migrate_mcumgr_kconfigs.py Signed-off-by: Johann Fischer <[email protected]> 6. Added IMAGE_ACCESS_HOOKS config to nRF53 config set CONFIG_IMAGE_ACCESS_HOOKS is required for nRF53 to confirm image during DFU over BLE after the changes provided by the recent Zephyr's upmerge. Signed-off-by: Arkadiusz Balys <[email protected]> 7. Changed WPA SUPP logs level to error * Changed default log level to error * Changed WPA supplicant log levet to error. This is a workaround and it will be removed once default log levels fix to Zephyr will be merged. Signed-off-by: Kamil Kasperczyk <[email protected]> 8. Fixed dependencies for CHIP_DFU_OVER_BT_SMP kconfig Some Kconfigs selected by the CHIP_DFU_OVER_BT_SMP have unmet dependencies. Signed-off-by: Kamil Kasperczyk <[email protected]> 9. Set BLE connection to 1 for NUS We decided to work with single BLE connection when NUS is active. 10. Repair a path within error messages in FD ptyhon script Two paths were set wrongly in the factory data python script. 11. Fixed system workqueue stack overflow for Wi-Fi After failing to connect to the Wi-Fi network (e.g. due to bad password) the application falls into stack overflow. Increased the stack size to handle the connection failure properly. Signed-off-by: Kamil Kasperczyk <[email protected]> 12. Disable bonding for NUS Matter NUS should require providing a PIN code before each pairing, so we do not want to store anything in persistent storage and we should disable the Bluetooth LE bonding feature. 13. Disable unnecessary assert information Limit assert information for Matter applications to file name and file number. Signed-off-by: Lukasz Duda <[email protected]> 14. Introduce workaround to Wi-Fi connection recovery It was observed that sometimes WPA supplicant may stuck in the reconnection process. This change introduces a workaround and fixes for existing recovery mechanism: * Changed is-Recovery-Aborted approach to is-Recovery-started. * Distinguished disconnects requested by the application from the disconnects coming from the supplicant. * Added scheduling recovery procedure if supplicant was not able to re-connect to the network for given amount of time (60 s). Signed-off-by: Kamil Kasperczyk <[email protected]> 15. Adapt WiFiManager to the new wifi_ps_config API The wifi_ps_config has been extended in Zephyr so this patch is need to avoid build errors. Signed-off-by: Marcin Kajor <[email protected]> 16. Integrated new Wi-Fi disconnect completed event Integrated handling new Wi-Fi disconnect completed event, what should fix occassional failures observed during connection recovery. Signed-off-by: Kamil Kasperczyk <[email protected]> 17. Fixed nRF7002 EK shield kconfig name Due to introduction of new nRF7002 shields, the nRF7002 EK shield kconfig name was changed and we have to align our configuration. Signed-off-by: Kamil Kasperczyk <[email protected]> 18. [nrfconnect] Use default number of RTC channels The default number of RTC channels on nRF5340 was increased from 2 to 3 some time ago, but Matter applications are still built with 2 user channels. This causes that CSL transmitter does not work properly on Matter Thread Router devices. 19 Add a workaround to WiFi recovery During the recovery procedure, if the net_mgmt function returns the value "-16" the device cannot recover the WiFi network until the reboot is called. This workaround allows the device to try to rescan the WiFi network even if the net_mgmt function returned an error. This workaround works only, when the WiFi recovery mechanism is active. 20. moved bt_fixed_passkey config to the features set BT_FIXED_PASSKEY cannot be added to prj_release.conf file because it causes a CMAKE warning. The config has been moved to the Kconfig.feature file. Fix the build warning in door-lock-server This fixes the uninitialized variable warning appearing when compiling the door lock app. Signed-off-by: Marcin Kajor <[email protected]> 21. Fixed nrf7002 kconfig name again nRF7002 EK kconfig name changed once again, so we need to align our kconfigs to that. Signed-off-by: Kamil Kasperczyk <[email protected]>
The SYS_INIT() callback API changed in Zephyr and Matter code alignments were needed.
After recent changes in Zephyr, the Zephyr POSIX layer no longer includes missing POSIX definitions from the standard library. Given that and that the Zephyr POSIX layer is very incomplete, it is no longer possible to build Matter samples using that layer. Disable Zephyr socket layer, and use NET_SOCKETS_POSIX_NAMES which only wraps the socket functions with POSIX names. Make sure that proper headers are included when using this configuration.
Replaced CONFIG_MCUMGR_SMP_BT that is no longer supported with the CONFIG_MCUMGR_TRANSPORT_BT
* Added CHIP_NRF_PLATFORM symbol to conditionally modify generic Zephyr configuration * Updated docker versions for cloud builds * Updated sdk-zephyr toolchain version to 0.16.0 in build scripts for nrf platform.
* Fixed typo in wake event * Removed ifdef leading to compilation errors for Darwin and Linux and added ifdef for usleep usage that is a root cause of problems on Zephyr.
1. Use Zephyr SDK toolchain. 2. Use proper shell configuration.
kkasperczyk-no
force-pushed
the
ncs_2_4_0_update
branch
from
eac4e73 to
27422c6
Compare
|
PR #27010: Size comparison from 8ba86ec to 27422c6 Increases above 0.2%:
Increases (38 builds for cc32xx, cyw30739, nrfconnect, psoc6, telink)
Decreases (18 builds for bl702, esp32, nrfconnect, psoc6, qpg)
Full report (70 builds for bl602, bl702, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
LuDuda
approved these changes
Jun 5, 2023
bzbarsky-apple
approved these changes
Jun 6, 2023
andy31415
added a commit
to andy31415/connectedhomeip
that referenced
this pull request
Jun 7, 2023
…d is still located under zephyr
andy31415
added a commit
that referenced
this pull request
Jun 7, 2023
…ocated under zephyr (#27125)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Regular update of nRF Connect SDK version to 2.4.0 including:
Depends on: #27004