[ac_range_check] D1+D2 Review & Signoff
@andreaskurth is already working on this.
Since Apr 11, 2025.
- Dominant language
- SystemVerilog
- Stars
- 3.6k
- Forks
- 1.1k
- Avg merge
- 2d 22h
- Merged PRs (30d)
- 141
Description
## Design specification checks / updates
- [ ] https://github.com/lowRISC/opentitan/issues/27726
## RTL checks
- [ ] https://github.com/lowRISC/opentitan/issues/27727
- [ ] REVIEW_TODO (D2 item)
Grep the code base for any TODO comments.
Add link to an issue to any TODO missing it (create issue if necessary).
Triage all open TODO issues and justify why they are not needed for D2.
- [ ] LINT_PASS
Ensure that the IP gets linted as part of CI checks and that lint passes without errors or warnings.
Review and justify any lint waiver files.
## Various checks
The following items are also part of the D1 or D2 signoff checklists.
They are either explicitly included in tasks listed above or an implicit requirement for the parallel V signoff.
They are not tracked separately; their current status is visible in the [checklist document for this IP](https://github.com/lowRISC/opentitan/blob/master/hw/ip_templates/ac_range_check/doc/checklist.md).
- CSR_DEFINED (to be done as part of FEATURE_COMPLETE)
- CLKRST_CONNECTED (implicit requirement for parallel V signoff)
- IP_TOP (to be done as part of FEATURE_COMPLETE)
- IP_INSTANTIABLE (implicit requirement for parallel V signoff)
- PHYSICAL_MACROS_DEFINED_80 (to be done as part of BLOCK_DIAGRAM)
- FUNC_IMPLEMENTED (to be done as part of FEATURE_COMPLETE)
- ASSERT_KNOWN_ADDED (to be done as part of FEATURE_COMPLETE)
- NEW_FEATURES (to be done as part of SPEC_COMPLETE)
- MISSING_FUNC (to be done as part of FEATURE_COMPLETE)
- FEATURE_FROZEN (to be done as part of SPEC_COMPLETE)
- PORT_FROZEN (to be done as part of BLOCK_DIAGRAM)
- ARCHITECTURE_FROZEN (to be done as part of BLOCK_DIAGRAM)
- STYLE_X (to be done as part of FEATURE_COMPLETE)
- CDC_SYNCMACRO (to be done as part of FEATURE_COMPLETE)
## Security checks
Any non-standard security countermeasures are excluded from this signoff, hence SEC_CM_SCOPED + SEC_CM_DOCUMENTED are _waived_.
## CDC, RDC, area, and timing checks
These are currently not done at the block level, hence these items are _waived_ from this signoff.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.