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

No sicrypto ecdsa #20136

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

degjorva
Copy link
Contributor

Prototype of the removal cracenpsa implementation of ecdsa

@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 Jan 31, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jan 31, 2025

CI Information

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

Inputs:

Sources:

more details

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 (0)

Outputs:

Toolchain

Version:
Build docker image:

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

  • ❌ Toolchain
  • ❌ Build twister
  • ❌ Integration tests

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.

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

@degjorva degjorva force-pushed the no-sicrypto-ECDSA branch 16 times, most recently from 64ba00c to 42fdcf2 Compare February 4, 2025 13:22
@degjorva degjorva force-pushed the no-sicrypto-ECDSA branch 3 times, most recently from ccbf6ae to 87a4734 Compare February 12, 2025 12:39
@degjorva degjorva force-pushed the no-sicrypto-ECDSA branch 7 times, most recently from 9bb70c5 to 0f10dbf Compare February 17, 2025 17:04
@degjorva degjorva force-pushed the no-sicrypto-ECDSA branch 7 times, most recently from 705567c to 9225cbf Compare February 20, 2025 08:39
Copy link

github-actions bot commented Feb 20, 2025

After documentation is built, you will find the preview for this PR here.

Preview links for modified nRF Connect SDK documents:

https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/app_dev/device_guides/fem/fem_nrf21540_gpio.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/app_dev/device_guides/fem/fem_nrf21540_gpio_spi.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/app_dev/device_guides/fem/fem_nrf2220.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/app_dev/device_guides/fem/fem_simple_gpio.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/app_dev/device_guides/fem/index.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/app_dev/device_guides/nrf54h/index.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/app_dev/device_guides/nrf54h/ug_nrf54h20_architecture_lifecycle.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/app_dev/device_guides/nrf54h/ug_nrf54h20_custom_pcb.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/app_dev/device_guides/nrf54h/ug_nrf54h20_gs.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/app_dev/device_guides/wifi_coex.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/dev_model_and_contributions/adding_code.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/drivers/wifi.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/libraries/modem/location.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/libraries/networking/downloader.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/protocols/bt/bt_mesh/dfu_over_bt_mesh.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/protocols/lte/index.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/protocols/matter/index.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/releases_and_maturity/abi_compatibility.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/releases_and_maturity/known_issues.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/releases_and_maturity/migration/migration_guide_2.8.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/releases_and_maturity/migration/migration_guide_2.9.0-nrf54h20-rc1.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/releases_and_maturity/release_notes.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/releases_and_maturity/releases/release-notes-2.9.0-nrf54h20-rc1.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/releases_and_maturity/releases/release-notes-changelog.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/samples/matter.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/security/tfm.html
doc/nrf/samples/matter.rst
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/samples/bluetooth/fast_pair/locator_tag/README.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/samples/cellular/nrf_cloud_multi_service/README.html
https://ncsdoc.z6.web.core.windows.net/PR-20136/nrf/samples/suit/recovery/README.html

@degjorva degjorva force-pushed the no-sicrypto-ECDSA branch 4 times, most recently from 7b4b926 to 9225cbf Compare February 20, 2025 16:37
@degjorva degjorva force-pushed the no-sicrypto-ECDSA branch 2 times, most recently from a2e75f4 to fd25f26 Compare February 28, 2025 12:13
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
6.5% Duplication on New Code (required ≤ 3%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@degjorva degjorva force-pushed the no-sicrypto-ECDSA branch 13 times, most recently from d14a3e4 to 534ecef Compare March 26, 2025 15:42
Add support for ECDSA in cracenpsa
Directly using silexpk/sxsymcrypt
This bypasses sicrypto, which saves on flash usage
Remove sicrypto implementation of ECDSA being
accessible from cracenpsa.

Signed-off-by: Dag Erik Gjørvad <dag.erik.gjorvad@nordicsemi.no>
@degjorva degjorva force-pushed the no-sicrypto-ECDSA branch from 534ecef to dd4c71f Compare March 26, 2025 17:11
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants