Skip to content

manifest: MCUboot KMU support for ED25519 signature verification #17616

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

manifest: MCUboot KMU support for ED25519 signature verification #17616

wants to merge 2 commits into from

Conversation

@de-nordic
Copy link
Contributor

@de-nordic de-nordic commented Oct 2, 2024

Update MCUboot to support KMU with ED25519

@de-nordic de-nordic requested a review from a team as a code owner October 2, 2024 16:01
@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 Oct 2, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 2, 2024 •
edited
Loading

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
mcuboot nrfconnect/sdk-mcuboot@7e01086 (main) nrfconnect/sdk-mcuboot#335 nrfconnect/sdk-mcuboot#335/files

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 2, 2024 •
edited
Loading

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: b4bfcf6d1386c4cb5ef55fee55746ecd23d0770c
mcuboot: PR head: afb04eae258f342b9a9f1771930d00cee5040ce6

more details

sdk-nrf:

PR head: b4bfcf6d1386c4cb5ef55fee55746ecd23d0770c
merge base: cbf1d39c3fa2c33708480011d115ac442a9a7cf2
target head (main): cbf1d39c3fa2c33708480011d115ac442a9a7cf2
Diff

mcuboot:

PR head: afb04eae258f342b9a9f1771930d00cee5040ce6
merge base: 7e010864696446fc42e185161d1f73bff563ee29
target head (main): 7e010864696446fc42e185161d1f73bff563ee29
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 (7) 
bootloader
│  ├── mcuboot
│  │  ├── boot
│  │  │  ├── bootutil
│  │  │  │  ├── src
│  │  │  │  │  ├── ed25519_psa.c
│  │  │  │  │  ├── image_ed25519.c
│  │  │  │  │  │ image_validate.c
│  │  │  ├── zephyr
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  │ Kconfig
subsys
│  ├── nrf_security
│  │  │ Kconfig
west.yml

Outputs:

Toolchain

Version: 3dd8985b56
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:3dd8985b56_912848a074

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 1495
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-boot
    • ✅ test-fw-nrfconnect-chip
    • ✅ test-fw-nrfconnect-nrf-iot_cloud
    • ✅ test-fw-nrfconnect-nrf_crypto
    • ✅ test-fw-nrfconnect-tfm
    • ✅ test-sdk-find-my
    • ✅ test-sdk-sidewalk
    • ✅ test-sdk-mcuboot
    • ✅ test-sdk-dfu
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_nrf_provisioning
    • 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_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-low-level
    • test-sdk-audio
    • test-sdk-pmic-samples
    • test-sdk-wifi

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 2, 2024

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

@de-nordic de-nordic force-pushed the x25519_kmu branch 2 times, most recently from 7a926a4 to ccbbb3c Compare October 8, 2024 13:02
@nvlsianpu nvlsianpu added this to the 2.8.0 milestone Oct 14, 2024
@de-nordic de-nordic requested a review from a team as a code owner October 23, 2024 20:10
de-nordic and others added 2 commits October 23, 2024 20:13
@de-nordic
manifest: MCUboot KMU support for ED25519 signature verification
4738cd0 
Update MCUboot to support KMU with ED25519

Signed-off-by: Dominik Ermel <[email protected]>
@nordicjm @de-nordic
nrf_security: Fix MBEDTLS_CFG_FILE Kconfig symbol
b4bfcf6 
Removes a prompt, type and duplicated help text from this symbol
as this is an override symbol only

Signed-off-by: Jamie McCrae <[email protected]>
@nordicjm
Copy link
Contributor

nordicjm commented Oct 24, 2024

sdk-mcuboot PR has been merged, have brought in manifest update and commit here into #17567 to save on CI

@nordicjm nordicjm closed this Oct 24, 2024
@nordicjm nordicjm mentioned this pull request Oct 24, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nvlsianpu nvlsianpu nvlsianpu approved these changes

@nordicjm nordicjm nordicjm approved these changes

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

2.8.0

Development

Successfully merging this pull request may close these issues.

4 participants

@de-nordic @NordicBuilder @nordicjm @nvlsianpu