lowRISC / opentitan

OpenTitan: Open source silicon root of trust
https://www.opentitan.org
Apache License 2.0
2.51k stars 745 forks source link

[doc] Add D2/D3 review item for documentation of instantiation parameters #13858

Open msfschaffner opened 2 years ago

msfschaffner commented 2 years ago

This has come up as part of the OTP_CTRL D3 review #13438. In particular, we should make sure that the valid / tested configurations are documented in the spec before signing off. Such a checklist item could be placed into the D2 or D3 checklist (D2 would give DV a better handle on which configurations need to tested).

msfschaffner commented 2 years ago

As discussed on https://github.com/lowRISC/opentitan/pull/13860, one possibility would be to add the following checklist item to the D3 checklist:

### DOC_PARAM_IP

If not otherwise stated in the comportable IP specification, all instantiation parameter combinations are considered legal and do not impact D3/V3 status.
If an IP supports only a subset of parameterizations, it must explicitly document the supported parameterizations in a section called "Parameter Changes after D3/V3" in the design specification.
tjaychen commented 1 year ago

@tjaychen