forked from nrfconnect/sdk-nrf
-
Notifications
You must be signed in to change notification settings - Fork 0
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
net: nrf_provisioning: change addrinfo to zsock_addrinfo #57
Closed
juhaylinen
wants to merge
110
commits into
anangl:upmerge_20241220
from
juhaylinen:fix_provisioning_tests
Closed
net: nrf_provisioning: change addrinfo to zsock_addrinfo #57
juhaylinen
wants to merge
110
commits into
anangl:upmerge_20241220
from
juhaylinen:fix_provisioning_tests
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
Add corrections for a bunch of sdk-zephyr revert commits that were cherry-picked after rebase without updating their references to reverted commits. Remove a no longer needed correction for one sdk-trusted-firmware-m commit. The added corrections can be removed after the next sdk-zephyr rebase. Signed-off-by: Andrzej Głąbek <[email protected]>
Pull in upmerged revisions of zephyr and mcuboot. Update other modules with related adjustments. Signed-off-by: Andrzej Głąbek <[email protected]>
Adapt to the latest changes applied to Zephyr. Signed-off-by: Andrzej Głąbek <[email protected]>
The bluetooth shell has been refactored (see bf897cf941a514e7ea7a5f837d2287e360281f24), so the sourced file no longer exists. Note that it was actually a bad use of sourcing, as the upstream options are already part of the Kconfig tree (unless I missed something). Signed-off-by: Gerard Marull-Paretas <[email protected]>
- Upgrade to latest Sphinx - Fix some references - Fix Zephyr docset required env vars Signed-off-by: Gerard Marull-Paretas <[email protected]>
Align with list of inputs in upstream Zephyr. Signed-off-by: Gerard Marull-Paretas <[email protected]>
According to the new extension parameters. Signed-off-by: Gerard Marull-Paretas <[email protected]>
Now the output will be $name/html|xml, so that we can generate other outputs other than HTML only, e.g. XML. Signed-off-by: Gerard Marull-Paretas <[email protected]>
Instead of the removed CONFIG_NRF_REGTOOL_GENERATE_BICR. Signed-off-by: Gerard Marull-Paretas <[email protected]>
`bt-hci-bus` property values are now specified in lowercase and without the `BT_HCI_BUS_` prefix. Signed-off-by: Andrzej Głąbek <[email protected]>
The ESB_DYNAMIC_INTERRUPTS option enables code that uses direct dynamic interrupts mechanism, so it should select the DYNAMIC_DIRECT_INTERRUPTS option together with its dependency DIRECT_INTERRUPTS, not depend on those, otherwise some unobvious dependency loops may appear (like one with MBOX; through NRF_RPC, MPSL_CX_SOFTWARE_RPC, NRF_FEM_ONLY, and MBOX_ANDES_PLIC_SW). Signed-off-by: Andrzej Głąbek <[email protected]>
This dependency introduced as a fix for cmake warnings doesn't seem to be needed anymore but it leads to a dependency loop with BT_CTLR_ADV_EXT_SUPPORT. This commit basically reverts 43c4519. Signed-off-by: Andrzej Głąbek <[email protected]>
After the latest update of the "nordic,npm1300-charger" binding, 1000 mA is the maximum value for the dischg-limit-microamp property. Signed-off-by: Andrzej Głąbek <[email protected]>
Remove entries for the following files that were recently deleted from the hostap module: - crypto_mbedtls-bignum.c - crypto_mbedtls-ec.c - crypto_mbedtls.c - tls_mbedtls.c Signed-off-by: Andrzej Głąbek <[email protected]>
As the BT_LL_CHOICE choice was removed upstream, BT_LL_SOFTDEVICE needs to be a regular option now (and not depending on BT_CTLR anymore). Also it should select HAS_BT_CTLR as it is supposed for all LL implemenations. See Zephyr PR 82157. Also adjust dragoon revision accordingly. Signed-off-by: Andrzej Głąbek <[email protected]>
- the BT_CTLR option was replaced with HAS_BT_CTLR which is selected by the LL implementations - the BT_LL_CHOICE choice option was removed See Zephyr PR 82157. Signed-off-by: Andrzej Głąbek <[email protected]>
Bluetooth LL is now selected via devicetree so remove the obsolete Kconfig stuff. See Zephyr PR 82157. Signed-off-by: Andrzej Głąbek <[email protected]>
net_pkt_set_txtime() was deprecated for two Zephyr releases and has been finally removed. net_pkt_set_timestamp_ns() is to be used instead. Signed-off-by: Andrzej Głąbek <[email protected]>
Such configuration would lead to a recursive logging loop and a special check catching it was introduced, so now it generates a build warning. See Zephyr PR 78449. Signed-off-by: Andrzej Głąbek <[email protected]>
Follow the renaming applied in upstream Zephyr when common CDC ACM UART configuration was introduced. See Zephyr PR 81308. Signed-off-by: Andrzej Głąbek <[email protected]>
The BT_LL_SW_SPLIT option is now automatically enabled based on the corresponding node ("zephyr,bt-hci-ll-sw-split" compatible) being enabled in devicetree (and this is done by the bt-ll-sw-split snippet). The BT_CTLR option is deprecated in favor of HAS_BT_CTLR which is selected accordingly by LL implementations. Signed-off-by: Andrzej Głąbek <[email protected]>
Add an option that will make NRF_SECURITY enabled when this is required for builds involving BT. This is a temporary solution, just to unblock building of many samples and tests. It is supposed to be eventually replaced with something more suitable. Signed-off-by: Andrzej Głąbek <[email protected]>
zephyr commit 8cfad44852845cd30336d40f61dade69ab4357db removed the auto adv resume and deprecated the options that were using this. So this commit updates all uses of the deprecated options in nrf. Also update accordingly find-my and sidewalk modules. Signed-off-by: Sean Madigan <[email protected]> Signed-off-by: Andrzej Głąbek <[email protected]>
These macros are now provided by Zephyr (in sys_clock.h). As these macros for mesh were defined with `unsigned long long` type and now they are `unsigned long`, casting was added where needed to avoid overflows. Signed-off-by: Andrzej Głąbek <[email protected]>
Wi-Fi credentials lib has been moved to upstream. And Wi-Fi management extension library is merged with Wi-Fi management in upstream. Signed-off-by: Ravi Dondaputi <[email protected]>
Add default file system configuration depending on flash driver. Signed-off-by: Maciej Baczmanski <[email protected]>
This region region is no longer enabled by default. It must be enabled explicitly when actually used. Signed-off-by: Andrzej Głąbek <[email protected]>
LE legacy pairing is no longer enabled by default. Samples that use it, need to disable the BT_SMP_SC_PAIR_ONLY Kconfig option. See Zephyr PR 73217. Signed-off-by: Andrzej Głąbek <[email protected]>
SUIT unit tests rely on the automatically assigned volatile key IDs. Since the key IDs changes with the amount of allowed keys, we should not change it without regenerating all test payloads. The new Zephyr brings a Kconfig to control this variable and changes the default amount from 32 to 16, so the SUIT test platform needs to override it back to the previous default value. Ref: NCSDK-31359 Signed-off-by: Tomasz Chyrowicz <[email protected]>
Commit adapts configuration ble mesh, trusted storage and mbedtls psa parameters to be able to run mesh samples and tests. Signed-off-by: Aleksandr Khromykh <[email protected]>
- `native_sim/native` needs to be used now instead of `native_sim` - add entries for failing samples and tests that are to be handled after the upmerge: * sample.mcumgr.smp_svr.ram_load.* * nrf_modem_dect_phy_pcc_event * secure_storage.psa.its.secure_storage.* * sample.net.prometheus - remove `native_posix` Signed-off-by: Maciej Perkowski <[email protected]> Signed-off-by: Andrzej Głąbek <[email protected]>
So far when building for nRF54L15 static DPPI channels were assigned. After adding nRF54L15 support in the DPPI driver this code is unused for nRF54L15 and the common DPPI allocator used hardcoded DPPIC instance. This commit parameterizes the DDPIC instance number depending on the SoC and enables the coresponding instances in the ESB samples' configurations. ESB uses hardcoded instance number per SoC. TODO: Allow selection of the DPPIC instance in one place e.g. Kconfig based on the DTS. Signed-off-by: Michał Grochala <[email protected]>
- Change posix names to zsock. - Remove CONFIG_x_NRF_HW_ASYNC usage. - Correct size for the external flash. Signed-off-by: Markus Lassila <[email protected]>
Function has been deprecated in upstream zephyr, so remove from here. Signed-off-by: Sean Madigan <[email protected]>
Let HOSTAP_CRYPTO_ALT_LEGACY_PSA select MBEDTLS_MD5_C so that md5 related calls are available when crypto_alt is used. Signed-off-by: Ravi Dondaputi <[email protected]>
In scan_only build we need to override the sysbuild config option as well. By default it gets set to SB_CONFIG_WIFI_NRF70_SYSTEM_MODE which enables supplicant. Overriding this with SB_CONFIG_WIFI_NRF70_SCAN_ONLY fixes this. Signed-off-by: Ravi Dondaputi <[email protected]>
The ZMS implementation of the ITS store module is not supported when partition manager is used because the DT chosen `secure_storage_its_partition` cannot be used. This would require using a different, partition manager-specific configuration. As partition manager will be replaced this combination is just not supported. Signed-off-by: Tomi Fontanilles <[email protected]>
We cannot render/link Doxygen groups that belong to another project like Zephyr, see nrfconnect#19341 for progress on that. Signed-off-by: Gerard Marull-Paretas <[email protected]>
Add Zephyr tests due to overflow issues. Refers: NCSDK-31589 NCSDK-31590 NCSDK-31591 Signed-off-by: Katarzyna Giądła <[email protected]>
BLE logging size requirements have increased a lot. We need to reduce logging to fit the application. Signed-off-by: Maximilian Deubel <[email protected]>
Switched from speed to size optimization for nRF54H20 DK target in debug configurations to reduce memory footprint. Jira: NCSDK-31374 Signed-off-by: Mateusz Kapala <[email protected]>
Switched from speed to size optimizations for nRF54L15 DK (nRF54L05 emulated SoC) and enabled LTO in debug and release configurations to reduce memory footprint. Jira: NCSDK-31374 Signed-off-by: Mateusz Kapala <[email protected]>
Remove reference of z_impl_sys_rand_get(). Signed-off-by: Eivind Jølsgard <[email protected]>
Updates required by changes in zephyr upstream for the BT HCI driver. Signed-off-by: Tommi Rantanen <[email protected]>
Update pm policy event usage after API was changed from using relative time in cycles to absolute time in ticks. Signed-off-by: Knut Eldhuset <[email protected]>
This test is not really useful in NCS context (checking if the nRF Wi-Fi driver can be built successfully is done by other tests and samples), and it requires the hal_atmel module to be pulled in (because the WINC1500 driver requires two header files that are located in that module). And the module weights ~87 MB, so it's not reasonable to pull it in just for this one test. Signed-off-by: Andrzej Głąbek <[email protected]>
Fixes an issue with stream writer by providing the size of the partition Signed-off-by: Jamie McCrae <[email protected]>
Change posix names to zsock in nrf_cloud lib. Signed-off-by: Maximilian Deubel <[email protected]>
Some sdk-zephyr tests and samples reports issues due to RAM overflow issues. Refers: NCSDK-31667 NCSDK-31671 Signed-off-by: Katarzyna Giądła <[email protected]>
Applications `applications.nrf_desktop.zrelease` and `applications.nrf_desktop.zdebug` report RAM overflow. Therefore, applications on listed platforms are moved to quaratine. Refers: NCSDK-31675 Signed-off-by: Katarzyna Giądła <[email protected]>
PSA requires large stacks, especially on 54. Signed-off-by: Sean Madigan <[email protected]> Signed-off-by: Frank Audun Kvamtrø <[email protected]>
Test configurations with secure_storage should be move to quarantine due to issues on native_sim platform. Signed-off-by: Katarzyna Giądła <[email protected]>
Clang is not available in our toolchain, therefore test kernel.common.toolchain will fail. Signed-off-by: Katarzyna Giądła <[email protected]>
`nanopb_BUILD_RUNTIME` is not an option to be passed to nanopb, but rather a cmake one, so set it to off in the proper way. The previous invalid form was not working as intended and only by chance not causing any problems. With a correction that was applied in the nanopb module (removal of leading space before the first -I directive passed to nanopb), this behavior changed and because of that the generation could not be performed correctly. Signed-off-by: Andrzej Głąbek <[email protected]>
Use zsock_addrinfo to avoid dependency on the POSIX subsystem. Remove CONFIG_POSIX_API from provisioning tests. Signed-off-by: Juha Ylinen <[email protected]>
c701a28
to
0c2235e
Compare
Cherry-picked into the branch. |
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.
Use zsock_addrinfo to avoid dependency on the POSIX subsystem.
Remove CONFIG_POSIX_API from provisioning tests.