Skip to content

Add ratchet_seed_lock_en test#685

Merged
ekarabu merged 2 commits intomainfrom
ocp_lock_enable_ratchet_seed
Sep 23, 2025
Merged

Add ratchet_seed_lock_en test#685
ekarabu merged 2 commits intomainfrom
ocp_lock_enable_ratchet_seed

Conversation

@antmarzam
Copy link
Collaborator

@antmarzam antmarzam commented Sep 4, 2025

Opening PR in behalf of @engdoreis

This PR takes care of TLT#3 from issue #603.

Testplan:

  • Configure OCP_LOCK_EN pin using plusargs argument in yml file;
  • From MCU
    • Programs racthet seed and reads it;
    • Populates Caliptra-core's HEK_SEED registers with the read value;
    • Releases the caliptra core by setting CPTRA_FUSE_WR_DONE;
    • Wait for OCP_LOCK_PROGRESS;
  • From Caliptra core
    • Read HEK_SEED from FC and check correctness;
    • Set OCP_LOCK_PROGRESS

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Sep 4, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

@antmarzam antmarzam force-pushed the ocp_lock_enable_ratchet_seed branch from f401ddd to 48af894 Compare September 5, 2025 14:52
@antmarzam antmarzam force-pushed the ocp_lock_enable_ratchet_seed branch from 48af894 to cf560d7 Compare September 9, 2025 16:17
@moidx moidx requested review from moidx and removed request for ekarabu September 9, 2025 16:35
martin-velay
martin-velay previously approved these changes Sep 12, 2025
Copy link
Collaborator

@martin-velay martin-velay left a comment

Choose a reason for hiding this comment

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

I am not a SW specialist, so I cannot really on that but I ran the test locally and it seems to work correctly

Testplan:
 - Configure OCP_LOCK_EN pin using plusargs argument in yml file;
 - From MCU
    - Programs racthet seed and reads it;
    - Populates Caliptra-core's HEK_SEED registers with the read value;
    - Releases the caliptra core by setting CPTRA_FUSE_WR_DONE;
    - Wait for OCP_LOCK_PROGRESS;
 - From Caliptra core
    - Read HEK_SEED from FC and check correctness;
    - Set OCP_LOCK_PROGRESS

Signed-off-by: Douglas Reis <doreis@lowrisc.org>
@andreaskurth
Copy link
Contributor

@engdoreis, I think you need to rebase this PR before stamping, because it now shows conflicts in pr_hash and pr_timestamp

@andreaskurth andreaskurth force-pushed the ocp_lock_enable_ratchet_seed branch from 9bc98b1 to ab4f27a Compare September 23, 2025 13:36
…th updated timestamp and hash after successful run
@ekarabu ekarabu merged commit 4097877 into main Sep 23, 2025
8 checks passed
@ekarabu ekarabu deleted the ocp_lock_enable_ratchet_seed branch September 23, 2025 18:46
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.

6 participants