Skip to content

Conversation

@de-nordic
Copy link
Contributor

Replaces PSA ed25519 sdk-nrf specific implementation with provided from upstream.

@github-actions github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Mar 4, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 4, 2025

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 28

Inputs:

Sources:

sdk-nrf: PR head: 0ca70e40898a8fee9865708ecbfc7f1107517b99
mcuboot: PR head: 91c282e7afc83c666ad478fbe70ff3e48ac56a05
zephyr: PR head: 8b5f3489c3d3e64723c7538e3346a7e0c1c9f3cf

more details

sdk-nrf:

PR head: 0ca70e40898a8fee9865708ecbfc7f1107517b99
merge base: 8798f86aa24582bd18256cbaca76cb25e25096cb
target head (main): 8798f86aa24582bd18256cbaca76cb25e25096cb
Diff

mcuboot:

PR head: 91c282e7afc83c666ad478fbe70ff3e48ac56a05
merge base: 148712e7b4618aadbedd04e8d3ce5c3847d3be4f
Diff

zephyr:

PR head: 8b5f3489c3d3e64723c7538e3346a7e0c1c9f3cf
merge base: 3fef3bd6ec5cc3eb18776de621326ff2eb0ea0bb
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (18)
bootloader
│  ├── mcuboot
│  │  ├── boot
│  │  │  ├── bootutil
│  │  │  │  ├── include
│  │  │  │  │  ├── bootutil
│  │  │  │  │  │  ├── crypto
│  │  │  │  │  │  │  ├── aes_ctr.h
│  │  │  │  │  │  │  │ sha.h
│  │  │  │  ├── pkg.yml
│  │  │  │  ├── src
│  │  │  │  │  ├── bootutil_priv.h
│  │  │  │  │  ├── ed25519_psa.c
│  │  │  │  │  ├── encrypted.c
│  │  │  │  │  ├── encrypted_psa.c
│  │  │  │  │  ├── image_ed25519.c
│  │  │  │  │  │ image_validate.c
│  │  │  │  ├── zephyr
│  │  │  │  │  │ CMakeLists.txt
│  │  │  ├── zephyr
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig
│  │  │  │  ├── include
│  │  │  │  │  ├── mcuboot-mbedtls-cfg.h
│  │  │  │  │  ├── mcuboot_config
│  │  │  │  │  │  │ mcuboot_config.h
│  │  ├── docs
│  │  │  │ design.md
west.yml
zephyr
│  ├── modules
│  │  ├── mbedtls
│  │  │  ├── Kconfig.tls-generic
│  │  │  ├── configs
│  │  │  │  │ config-tls-generic.h

Outputs:

Toolchain

Version: 4ffa2202d5
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:4ffa2202d5_8bf7ca4353

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 988
    • sdk-zephyr test count: 553
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-boot
    • ✅ test-fw-nrfconnect-nrf-iot_thingy91
    • ✅ test-fw-nrfconnect-nrf_crypto
    • ✅ test-fw-nrfconnect-rs
    • ✅ test-fw-nrfconnect-fem
    • ✅ test-fw-nrfconnect-thread
    • ✅ test-sdk-mcuboot
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-tfm
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 4, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
mcuboot nrfconnect/sdk-mcuboot@148712e nrfconnect/sdk-mcuboot@91c282e (main) nrfconnect/[email protected]
zephyr nrfconnect/sdk-zephyr@3fef3bd nrfconnect/sdk-zephyr@8b5f348 (main) nrfconnect/[email protected]

All manifest checks OK

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions
Copy link

github-actions bot commented Mar 4, 2025

You can find the documentation preview for this PR here.

@de-nordic de-nordic force-pushed the get_psa_from_upstream branch from f86aed5 to b329261 Compare March 14, 2025 09:55
@de-nordic de-nordic force-pushed the get_psa_from_upstream branch from b329261 to 337d122 Compare March 14, 2025 09:57
@de-nordic de-nordic marked this pull request as ready for review March 14, 2025 09:57
@de-nordic de-nordic requested review from a team as code owners March 14, 2025 09:57
@de-nordic de-nordic force-pushed the get_psa_from_upstream branch 12 times, most recently from 3e04173 to d7533da Compare March 21, 2025 11:53
@sonarqubecloud
Copy link

@de-nordic de-nordic force-pushed the get_psa_from_upstream branch 3 times, most recently from a245bf4 to 63ffb39 Compare March 25, 2025 12:11
@de-nordic de-nordic force-pushed the get_psa_from_upstream branch from 63ffb39 to bee2b1d Compare March 25, 2025 14:20
The commit brings in mbedTLS module changes that are required
for updating sdk-mcuboot with PSA ed25519 implementation
from upstream.
The commit replaces PSA ed25519 sdk-nrf specific implementation with
provided from upstream.

Signed-off-by: Dominik Ermel <[email protected]>
@de-nordic de-nordic force-pushed the get_psa_from_upstream branch from bee2b1d to 0ca70e4 Compare March 25, 2025 14:21
@NordicBuilder NordicBuilder removed the DNM label Mar 25, 2025
@rlubos rlubos merged commit c35c15e into nrfconnect:main Mar 25, 2025
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. manifest manifest-mcuboot manifest-zephyr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants