Skip to content

Conversation

@nasahlpa
Copy link
Member

@nasahlpa nasahlpa commented Jun 14, 2024

This commit adds the following OTBN SCA routines to the code base:

  • init: Init the OTBN SCA test on the device
  • init_keymgr: Prepare the keymanager for the OTBN key sideloading test
  • key_sideload_fvsr: Start the test
  • read_response: Read response from device
  • ecdsa_p256_sign: Test available in single, batch, and FvsR mode

Please note that this commit only adds these tests to the communication
stack but not to the capture script.

@nasahlpa nasahlpa requested a review from vogelpi as a code owner June 14, 2024 09:59
@nasahlpa nasahlpa force-pushed the otbn_key_sideload branch from ada100f to 1525afb Compare June 14, 2024 10:01
@nasahlpa nasahlpa requested a review from johannheyszl June 14, 2024 10:01
@nasahlpa
Copy link
Member Author

Could I please get a review for this PR?

This commit adds the following OTBN SCA routines to the code base:
- init: Init the OTBN SCA test on the device
- init_keymgr: Prepare the keymanager for the OTBN key sideloading test
- key_sideload_fvsr: Start the test
- read_response: Read response from device
- ecdsa_p256_sign: Test available in single, batch, and FvsR mode

Please note that this commit only adds these tests to the communication
stack but not to the capture script.

Signed-off-by: Pascal Nasahl <[email protected]>
@nasahlpa nasahlpa force-pushed the otbn_key_sideload branch from 5997705 to 9d0458a Compare March 17, 2025 15:54
@nasahlpa nasahlpa changed the title [sca] Add KeySideloadFvsr OTBN test [sca] Add KeySideloadFvsr and EcdsaP256 OTBN tests Mar 17, 2025
Copy link
Collaborator

@vogelpi vogelpi left a comment

Choose a reason for hiding this comment

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

This looks good to me, thanks @nasahlpa !

@nasahlpa nasahlpa merged commit 0f27037 into lowRISC:master Mar 20, 2025
9 checks passed
@nasahlpa nasahlpa deleted the otbn_key_sideload branch March 20, 2025 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants