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

Upmerge branch #478

Merged
merged 6 commits into from
Oct 4, 2024
Merged

Conversation

bjarki-andreasen
Copy link

Upmerge branch

See nrfconnect/sdk-nrf#16860

krish2718 and others added 2 commits September 24, 2024 11:31
nRF70 support moved upstream, update the names.

Signed-off-by: Chaitanya Tata <[email protected]>
-Previously there was a library called mbedcrypto_common which was
 used to pass build-related configurations to matter. This is changed
 to instead reference to the following:
 -psa_crypto_config (for PSA and Mbed TLS configurations)
 -psa_interface (for PSA crypto include files)
-The commit also removes passing the PSA crypto driver config file
 as this is no longer needed for an interface build (call from
 matter)

Signed-off-by: Frank Audun Kvamtrø <[email protected]>
@ArekBalysNordic
Copy link
Contributor

Once this PR: bjarki-andreasen#1 is merged we need to remove this one: c429269

If the CHIP_CRYPTO_PSA_SPAKE2P is enabled the build system should
not compile Legacy Spake2p implementation, because the PSA one
is available in the PSASpake2p.cpp file.
The updated way of handling mbedtls config file is done
correctly only for TF-M builds, but without it some of the files
are missing

Signed-off-by: Kamil Kasperczyk <[email protected]>
We should use psa_crypto_config only if PSA SSF crypto client
is in use.
SHELL_WILDCARD is not supported for Matter devices in the newest
Zephyr version because it selects POSIX_C_LIB_EXT, so we need to
disable it.
@bjarki-andreasen bjarki-andreasen marked this pull request as ready for review October 4, 2024 17:37
@carlescufi carlescufi merged commit a5bd017 into nrfconnect:master Oct 4, 2024
8 checks passed
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.

6 participants