Skip to content

All required crypto changes #16210

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 4 commits into from
Closed

All required crypto changes #16210

wants to merge 4 commits into from

Conversation

@osaether
Copy link
Contributor

@osaether osaether commented Jun 26, 2024

All required crypto changes

@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jun 26, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jun 26, 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.

@osaether osaether force-pushed the all_required_crypto_changes branch 3 times, most recently from 0626af2 to 2912902 Compare June 27, 2024 08:05
@Vge0rge Vge0rge force-pushed the all_required_crypto_changes branch from 2912902 to 1c36d5d Compare July 12, 2024 08:48
@Vge0rge Vge0rge marked this pull request as ready for review July 12, 2024 08:48
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jul 12, 2024 •
edited
Loading

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
desktop52_verification X
test-fw-nrfconnect-ble_samples X
test-fw-nrfconnect-chip X
test-fw-nrfconnect-fem X
test-fw-nrfconnect-nfc X
test-fw-nrfconnect-nrf-iot_cloud X
test-fw-nrfconnect-nrf-iot_thingy91 X
test-fw-nrfconnect-nrf_crypto X
test-fw-nrfconnect-rs X
test-fw-nrfconnect-tfm X
test-fw-nrfconnect-thread X
test-low-level X
test-sdk-audio X
test-sdk-dfu X
test-sdk-find-my X
test-sdk-sidewalk X
test_ble_commit X

Detailed information of selected test modules

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

@Vge0rge Vge0rge force-pushed the all_required_crypto_changes branch 2 times, most recently from 607e4f2 to 8da7e77 Compare July 15, 2024 08:15
@Vge0rge Vge0rge requested a review from carlescufi as a code owner July 15, 2024 08:15
@Vge0rge Vge0rge force-pushed the all_required_crypto_changes branch 5 times, most recently from 858c9f3 to 4731889 Compare July 16, 2024 11:32
@Vge0rge Vge0rge requested a review from tejlmand as a code owner July 16, 2024 12:04
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jul 16, 2024 •
edited
Loading

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

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@631b9a2 nrfconnect/sdk-zephyr#1817 nrfconnect/sdk-zephyr#1817/files

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

@Vge0rge Vge0rge force-pushed the all_required_crypto_changes branch 2 times, most recently from d6e1591 to 11cf347 Compare July 18, 2024 13:32
@Vge0rge Vge0rge force-pushed the all_required_crypto_changes branch from 11cf347 to 0c05d1e Compare July 22, 2024 08:41
@Vge0rge Vge0rge requested review from a team and anangl as code owners July 22, 2024 08:41
@Vge0rge Vge0rge force-pushed the all_required_crypto_changes branch from 0c05d1e to 10609d7 Compare July 22, 2024 11:23
nordic-piks
nordic-piks requested changes Jul 22, 2024
View reviewed changes
Copy link
Contributor

@nordic-piks nordic-piks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please note that at samples/cyrpto at sample.yamls there was added new tag: ci_samples_crypto
Please add it also into new configurations.

@jonathannilsen jonathannilsen force-pushed the all_required_crypto_changes branch from 10609d7 to 0e36c68 Compare July 25, 2024 12:51
@osaether osaether force-pushed the all_required_crypto_changes branch from 0e36c68 to 8aa56ce Compare July 30, 2024 10:43
@karstenkoenig karstenkoenig force-pushed the all_required_crypto_changes branch from 8aa56ce to bbc832c Compare August 5, 2024 06:02
@osaether osaether force-pushed the all_required_crypto_changes branch from bbc832c to 1b0c20a Compare August 6, 2024 13:51
osaether and others added 4 commits August 7, 2024 14:12
@osaether
nrf_security: Added cache handling in PSA Crypto service
d4ab2c1 
Added cache handling in PSA Crypto service

Ref: NCSDK-NONE

Signed-off-by: Ole Sæther <[email protected]>
@osaether
samples: crypto: Added snippet for crypto_service for target nrf54h20
75bf20d 
Added snippet for crypto_service for targets nrf54h20dk_nrf54h20_cpuapp
and nrf54h20dk_nrf54h20_cpurad and enabled the snippet for supported
samples.

Ref: NCSDK-NONE

Signed-off-by: Ole Sæther <[email protected]>
@Vge0rge @osaether
manifest: Bring updated sdk-zephyr
f70a5b0 
Bring updated Zephyr with required commits for
the PSA crypto support for nRF54H20.

Signed-off-by: Georgios Vasilakis <[email protected]>
@nika-nordic @osaether
samples: matter: dts: nrf54h20: align to cpuapp_data size change
bef5e3d 
RAM space available for App core is now smaller
because of miscellaneous area needed for dmm buffers.

Signed-off-by: Nikodem Kastelik <[email protected]>
@osaether osaether force-pushed the all_required_crypto_changes branch from 1b0c20a to bef5e3d Compare August 7, 2024 12:12
@github-actions
Copy link

github-actions bot commented Oct 9, 2024

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Oct 9, 2024
@osaether osaether closed this Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nordic-piks nordic-piks nordic-piks requested changes

@jonathannilsen jonathannilsen jonathannilsen approved these changes

@frkv frkv Awaiting requested review from frkv

@Vge0rge Vge0rge Awaiting requested review from Vge0rge

@vlilleboe vlilleboe Awaiting requested review from vlilleboe

@anhmolt anhmolt Awaiting requested review from anhmolt

@hakonfam hakonfam Awaiting requested review from hakonfam

@carlescufi carlescufi Awaiting requested review from carlescufi

@tejlmand tejlmand Awaiting requested review from tejlmand

@anangl anangl Awaiting requested review from anangl

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-zephyr Stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@osaether @NordicBuilder @jonathannilsen @nordic-piks @Vge0rge @nika-nordic