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

[bazel] Convert sigverify_key_validity to the new rules. #20391

Merged
merged 3 commits into from
Jan 10, 2024

Conversation

cfrantz
Copy link
Contributor

@cfrantz cfrantz commented Nov 17, 2023

Also cleans up some comment references to opentitan_functest().

Fixes: #20098

@cfrantz cfrantz requested review from pamaury and moidx November 17, 2023 22:25
@cfrantz cfrantz requested a review from a team as a code owner November 17, 2023 22:25
@moidx moidx removed the request for review from a team November 18, 2023 00:47
@cfrantz cfrantz force-pushed the bazel-e2e branch 3 times, most recently from b42384c to a53bc8d Compare January 8, 2024 18:51
1. Eliminate references to opentitan_functest().
2. Eliminate explicit splices in favor of the implied universal splice.

Signed-off-by: Chris Frantz <[email protected]>
1. Prefer an OTP configuration to an explicit bitstream splice.
2. Do not run as a test_rom test.  This test is not compatible with the
   test_rom because the test_rom does not respect the exec_disabled OTP
   config word.

Signed-off-by: Chris Frantz <[email protected]>
@cfrantz cfrantz merged commit cf38c1d into lowRISC:master Jan 10, 2024
32 checks passed
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.

[bazel] New rules: //sw/device/silicon_creator/rom/e2e/sigverify_key_validity/BUILD
3 participants