[acc] Renaming and restructuring of repo for OTBN to ACC name change#199
Merged
[acc] Renaming and restructuring of repo for OTBN to ACC name change#199
Conversation
apinise
force-pushed
the
acc-name-change
branch
5 times, most recently
from
b317ef8 to
85436b2
Compare
apinise
force-pushed
the
acc-name-change
branch
2 times, most recently
from
efd7718 to
34ceda1
Compare
Contributor
Author
|
This PR is expected to fail the verible lint check for the same reason as #197. It has been tested offline however to make sure there are no linting issues after changing names. |
apinise
force-pushed
the
acc-name-change
branch
from
681ee17 to
f308c03
Compare
jadephilipoom
approved these changes
Feb 25, 2026
apinise
force-pushed
the
acc-name-change
branch
3 times, most recently
from
b8474b4 to
67d6d50
Compare
This commit renames any name instance of otbn to acc in files and directories, except for vendored files. It also replaces name instances in files from otbn to acc. All name changes were case sensitive. Signed-off-by: apinise <eapinis@zerorisc.com>
apinise
force-pushed
the
acc-name-change
branch
from
67d6d50 to
8dc47fb
Compare
Most issues were caused by either misaligned lines after OTBN was replaced with 3 letters in ACC. Others were due to alphabetical order being changed. The following files were resolved in this commit. C/C++ and clang errors. Rust file linting. Bazel BUILD file formatting. Python file formatting. Signed-off-by: Evan Apinis <eapinis@zerorisc.com>
Updated RSA expected values based on update "Hello ACC" message. Fixed clkmgr instantiation order for transactional clock IP blocks. The order of transactional clock IP blocks should be alphabetized. OTBN was originally last in the list but ACC should be first before AES. After renaming topgen was used to generate darjeeling and earlgrey. Signed-off-by: apinise <eapinis@zerorisc.com>
Replaced instances of OpenTitan Big Num Accelerator with Asymmetric Cryptographic Coprocessor. The first name change only replaced the abbreviated instances. This commit applies the full rename. Signed-off-by: Evan Apinis <eapinis@zerorisc.com>
apinise
force-pushed
the
acc-name-change
branch
from
8dc47fb to
0861fc8
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR changes the name of OTBN to ACC given that the IP has become more than just a bignum accelerator with the introduction of PQC support and a vector ISA extension. The Asymmetric Cryptographic Coprocessor is forked from OTBN.
PR #200 updates the documentation for ACC