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

manifest: Remove cryptocell usable configs and use HAS_HW instead #10863

Merged
merged 1 commit into from
Nov 1, 2023

Conversation

joerchan
Copy link
Contributor

Remove the configurations for cryptocell usable and use the standard HAS_HW configurations instead.

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

NordicBuilder commented Apr 17, 2023

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

Name Old Revision New Revision Diff
nrfxlib nrfconnect/sdk-nrfxlib@89d19c9 nrfconnect/sdk-nrfxlib@ccf385b nrfconnect/[email protected]

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Apr 17, 2023

Test specification

CI/Jenkins/NRF

  • Integration Platforms

CI/Jenkins/integration

Test Module File based changes Manually selected West overwrite
test-fw-nrfconnect-chip X
test-fw-nrfconnect-nrf-iot_cloud X
test-fw-nrfconnect-nrf_crypto X
test-fw-nrfconnect-tfm X
test-fw-nrfconnect-thread X
test-sdk-find-my X
test-sdk-sidewalk X
test-sdk-wifi X

Detailed information of selected test modules

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

@NordicBuilder
Copy link
Contributor

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.

@joerchan joerchan marked this pull request as draft April 18, 2023 07:39
@joerchan joerchan force-pushed the remove-cryptcell-usable-config branch from 8e4c642 to 5613144 Compare July 24, 2023 14:02
@joerchan joerchan marked this pull request as ready for review July 24, 2023 14:03
@joerchan joerchan force-pushed the remove-cryptcell-usable-config branch from 5613144 to 0c65e49 Compare July 25, 2023 08:31
Copy link
Contributor

@frkv frkv left a comment

Choose a reason for hiding this comment

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

lgtm

@joerchan joerchan force-pushed the remove-cryptcell-usable-config branch 2 times, most recently from ba6ffb2 to fcf3d3e Compare July 26, 2023 07:45
@joerchan joerchan force-pushed the remove-cryptcell-usable-config branch from fcf3d3e to 9e70283 Compare September 25, 2023 14:31
@joerchan
Copy link
Contributor Author

Split this PR into 2 parts: follow up for zephyr change here: #12428

@joerchan joerchan force-pushed the remove-cryptcell-usable-config branch 3 times, most recently from 2d38799 to 53dec08 Compare October 31, 2023 10:54
Remove the configurations for cryptocell usable and use the standard
HAS_HW configurations instead.

Signed-off-by: Joakim Andersson <[email protected]>
@joerchan joerchan force-pushed the remove-cryptcell-usable-config branch from 53dec08 to dfca3ab Compare November 1, 2023 08:31
@NordicBuilder NordicBuilder removed the DNM label Nov 1, 2023
@nordicjm nordicjm merged commit 6e35840 into nrfconnect:main Nov 1, 2023
21 of 22 checks passed
@joerchan joerchan deleted the remove-cryptcell-usable-config branch November 1, 2023 12:38
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-nrfxlib manifest-zephyr
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants