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

quarantine: update #5

Open
wants to merge 124 commits into
base: psa_crypto_init_generic
Choose a base branch
from
Open

Conversation

PerMac
Copy link

@PerMac PerMac commented Feb 10, 2025

As in the topic

anangl and others added 30 commits February 7, 2025 15:59
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 <andrzej.glabek@nordicsemi.no>
Pull in upmerged revisions of zephyr and mcuboot.

Update other modules with related adjustments.

Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
Adapt to the latest changes applied to Zephyr.

Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
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 <gerard.marull@nordicsemi.no>
- Upgrade to latest Sphinx
- Fix some references
- Fix Zephyr docset required env vars

Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
Align with list of inputs in upstream Zephyr.

Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
According to the new extension parameters.

Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
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 <gerard.marull@nordicsemi.no>
Instead of the removed CONFIG_NRF_REGTOOL_GENERATE_BICR.

Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
`bt-hci-bus` property values are now specified in lowercase and without
the `BT_HCI_BUS_` prefix.

Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
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 <andrzej.glabek@nordicsemi.no>
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 <andrzej.glabek@nordicsemi.no>
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 <andrzej.glabek@nordicsemi.no>
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 <andrzej.glabek@nordicsemi.no>
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 <andrzej.glabek@nordicsemi.no>
- 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 <andrzej.glabek@nordicsemi.no>
Bluetooth LL is now selected via devicetree so remove the obsolete
Kconfig stuff. See Zephyr PR 82157.

Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
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 <andrzej.glabek@nordicsemi.no>
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 <andrzej.glabek@nordicsemi.no>
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 <andrzej.glabek@nordicsemi.no>
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 <andrzej.glabek@nordicsemi.no>
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 <andrzej.glabek@nordicsemi.no>
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 <sean.madigan@nordicsemi.no>
Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
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 <andrzej.glabek@nordicsemi.no>
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 <ravi.dondaputi@nordicsemi.no>
Add default file system configuration depending on flash driver.

Signed-off-by: Maciej Baczmanski <maciej.baczmanski@nordicsemi.no>
This region region is no longer enabled by default.
It must be enabled explicitly when actually used.

Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
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 <andrzej.glabek@nordicsemi.no>
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 <tomasz.chyrowicz@nordicsemi.no>
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 <aleksandr.khromykh@nordicsemi.no>
maxd-nordic and others added 23 commits February 7, 2025 16:00
Change posix names to zsock in nrf_cloud lib.

Signed-off-by: Maximilian Deubel <maximilian.deubel@nordicsemi.no>
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 <katarzyna.giadla@nordicsemi.no>
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 <katarzyna.giadla@nordicsemi.no>
PSA requires large stacks, especially on 54.

Signed-off-by: Sean Madigan <sean.madigan@nordicsemi.no>
Signed-off-by: Frank Audun Kvamtrø <frank.kvamtro@nordicsemi.no>
Test configurations with secure_storage should be
move to quarantine due to issues on native_sim
platform.

Signed-off-by: Katarzyna Giądła <katarzyna.giadla@nordicsemi.no>
Clang is not available in our toolchain, therefore
test kernel.common.toolchain will fail.

Signed-off-by: Katarzyna Giądła <katarzyna.giadla@nordicsemi.no>
`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 <andrzej.glabek@nordicsemi.no>
Use zsock_addrinfo to avoid dependency on the POSIX subsystem.

Remove CONFIG_POSIX_API from provisioning tests.

Signed-off-by: Juha Ylinen <juha.ylinen@nordicsemi.no>
... because this causes conflicting types compilation errors in library
files it uses.

Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
Test scenarios `shell.device_filter` and
`init.check_init_priorities` should be fixed in
Zephyr first.

Signed-off-by: Katarzyna Giądła <katarzyna.giadla@nordicsemi.no>
-Ensuring psa_crypto_init is called before PSA crypto APIs are used
 in case the subsystem is not calling psa_crypto_init()
-Also adds a kernel panic if there is a failure to initialize

Signed-off-by: Frank Audun Kvamtrø <frank.kvamtro@nordicsemi.no>
-Changing default y for nrf_security to use
 DT_HAS_ZEPHYR_PSA_CRYPTO_RNG_ENABLED instead of depending
 on ENTROPY_PSA_CRYPTO_RNG. Doing this avoids a dependency
 loop on ENTROPY_GENERATOR

Signed-off-by: Frank Audun Kvamtrø <frank.kvamtro@nordicsemi.no>
-This commit is only used to enact testing. The dependency for
 PR nrfconnect#2464 to the previous commit is inherent, but we would like
 to have a commit that is pickable in the upmerge

Signed-off-by: Frank Audun Kvamtrø <frank.kvamtro@nordicsemi.no>
This is required by PSA in NCS, so make sure it is selected
NRF_SECURITY_ENABLER.

Signed-off-by: Sean Madigan <sean.madigan@nordicsemi.no>
-Enabling ENTROPY_GENERATOR for BT_HOST_CRYPTO_PRNG and BT_ECC
 as a workaround of having ENTROPY_GENERATOR in sample/tests

Also the heap size required for bsim is larger than on
actual target, reason unknown.

Signed-off-by: Sean Madigan <sean.madigan@nordicsemi.no>
Signed-off-by: Frank Audun Kvamtrø <frank.kvamtro@nordicsemi.no>
The NRF_RNG_ENTROPY_DRIVER provides entropy though the PSA APIs.

This is used to seed/reseed the Oberon PSA random drivers
(CTR_DRBG/HMAC_DRBG). Before this was only used to allow enabling
this PSA driver with the devices which use the DT label 'rng'.

We extend that now to allow this driver to be used with the fake
entropy node which uses the DT label 'prng'.

This is needed to unblock development until real RNG is enabled
for all the needed applications/samples.

In practice this means that when 'prng' is enabled a device can
generate random numbers with the psa_generate_random call.

Signed-off-by: Georgios Vasilakis <georgios.vasilakis@nordicsemi.no>
-Adding CONFIG_MBEDTLS_FORCE_LEGACY_MD for cc3xx/oberon legacy
 psa_tls to avoid that MBEDTLS_MD_SOME_PSA is getting set

Signed-off-by: Frank Audun Kvamtrø <frank.kvamtro@nordicsemi.no>
-Fixes issues with nRF54H20 and nRF5340dk in upmerge. This can be
 seen as a workaround
-Adjusting board-files for central_uart and peripheral_uart
 for nRF5340 and nRFH20 cpuapp

Signed-off-by: Frank Audun Kvamtrø <frank.kvamtro@nordicsemi.no>
-HKDF reports errors when MBEDTLS_MD_SOME_PSA is set. Setting this
 configuration to avoid trying to resolve with PSA APIs.
 This configurations and these tests to be removed with more
 standardization on PSA crypto

Signed-off-by: Frank Audun Kvamtrø <frank.kvamtro@nordicsemi.no>
Increased stack sizes to prevent the stack overflows.

Jira: NCSDK-31374

Signed-off-by: Mateusz Kapala <mateusz.kapala@nordicsemi.no>
-Adjusting CONFIG_HEAP_MEM_POOL_SIZE down from 2048 to 1536 to try
 to get the smallest devices to build

Signed-off-by: Frank Audun Kvamtrø <frank.kvamtro@nordicsemi.no>
-Setting main stack size to 3584 for nRF54LX series that enable rng
 to avoid stack overflow

Signed-off-by: Frank Audun Kvamtrø <frank.kvamtro@nordicsemi.no>
As in the topic

Signed-off-by: Maciej Perkowski <maciej.perkowski@nordicsemi.no>
@frkv frkv force-pushed the psa_crypto_init_generic branch from 072e7b3 to 3d833ec Compare February 10, 2025 16:28
@frkv frkv requested a review from magnev as a code owner February 10, 2025 16:28
@frkv frkv force-pushed the psa_crypto_init_generic branch 4 times, most recently from 9a1b371 to 3e6b87a Compare February 11, 2025 14:31
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.