-
Notifications
You must be signed in to change notification settings - Fork 762
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
[otp_ctrl] V2S Signoff #21018
Comments
Commits since Earlgrey-ES tapeout
Issues closed since the Earlgrey-ES tapeout
Currently open issues
Coverage Results from 03/09/2024DVFormal (CMs)
SummaryThe OTP_CTRL has undergone quite some refactoring in order to make generation of RTL and DV collateral possible, given an Hjson memory map description. This refactoring was carried out on the While overall the functionality is largely unchanged, the series of patches do change a few things:
The OTP_CTRL IP has seen quite some RTL changes, but the associated DV tests have been updated alongside. The coverage is still above 90% across the board and the formal countermeasure FPV proofs all pass. Note that this is a "focus block" and hence the PROD.M2 requirement would be a V1 signoff. However, since no countermeasures have been changed / removed in the series of patches, and since the coverage is still fulfilling V2S requiremetns, the recommendation is to sign this block off at V2S instead. |
There is a D2S issue that still needs to be fixed #21953. From a DV perspective, this is however tested implicitly and impact to coverage numbers should be minimal (if at all, since we have pretty high toggle coverage). We can wait for the Tuesday regression to make sure everything is still good. |
Fix lowRISC#21018 Signed-off-by: Michael Schaffner <[email protected]>
Fix lowRISC#21018 Signed-off-by: Michael Schaffner <[email protected]>
@rswarbrick @matutem PTAL |
Fix #21018 Signed-off-by: Michael Schaffner <[email protected]>
Note: OTP_CTRL has effectively be signed off at V2S. I've now corrected the issue title and first comment accordingly. |
Description
Ensure V2S signoff criteria are fulfilled after focus area changes have landed.
The text was updated successfully, but these errors were encountered: