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

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

Open
msfschaffner opened this issue Jul 26, 2022 · 2 comments
Open
Assignees
Labels
Component:Doc Documentation issue Earlgrey-PROD Triaged Temporary label to triage issues into Earlgrey-PROD Milestones Type:Task Tasks, to-do list.
Milestone

Comments

@msfschaffner
Copy link
Contributor

msfschaffner commented Jul 26, 2022

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 msfschaffner added Component:Doc Documentation issue Priority:P2 Priority: medium Type:Task Tasks, to-do list. labels Jul 26, 2022
@msfschaffner msfschaffner added this to the Project: M3 milestone Jul 26, 2022
@msfschaffner msfschaffner changed the title [doc] Update D2 or D3 review process to include documentation of instantiation parameters [doc] Add D2/D3 review item for documentation of instantiation parameters Jul 26, 2022
@msfschaffner msfschaffner mentioned this issue Jul 26, 2022
9 tasks
@msfschaffner
Copy link
Contributor Author

msfschaffner commented Jul 29, 2022

As discussed on #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.

msfschaffner added a commit to msfschaffner/opentitan that referenced this issue Jul 29, 2022
Addresses lowRISC#13858

Signed-off-by: Michael Schaffner <[email protected]>
msfschaffner added a commit to msfschaffner/opentitan that referenced this issue Aug 4, 2022
Addresses lowRISC#13858

Signed-off-by: Michael Schaffner <[email protected]>
msfschaffner added a commit to msfschaffner/opentitan that referenced this issue Aug 4, 2022
Addresses lowRISC#13858

Signed-off-by: Michael Schaffner <[email protected]>
msfschaffner added a commit to msfschaffner/opentitan that referenced this issue Aug 4, 2022
Addresses lowRISC#13858

Signed-off-by: Michael Schaffner <[email protected]>
msfschaffner added a commit to msfschaffner/opentitan that referenced this issue Aug 4, 2022
Addresses lowRISC#13858

Signed-off-by: Michael Schaffner <[email protected]>
msfschaffner added a commit to msfschaffner/opentitan that referenced this issue Aug 4, 2022
Addresses lowRISC#13858

Signed-off-by: Michael Schaffner <[email protected]>
msfschaffner added a commit to msfschaffner/opentitan that referenced this issue Aug 4, 2022
Addresses lowRISC#13858

Signed-off-by: Michael Schaffner <[email protected]>
msfschaffner added a commit to msfschaffner/opentitan that referenced this issue Aug 4, 2022
Addresses lowRISC#13858

Signed-off-by: Michael Schaffner <[email protected]>
msfschaffner added a commit to msfschaffner/opentitan that referenced this issue Aug 4, 2022
Addresses lowRISC#13858

Signed-off-by: Michael Schaffner <[email protected]>
msfschaffner added a commit to msfschaffner/opentitan that referenced this issue Aug 4, 2022
Addresses lowRISC#13858

Signed-off-by: Michael Schaffner <[email protected]>
@tjaychen tjaychen removed their assignment Dec 13, 2022
@tjaychen
Copy link

@tjaychen

@moidx moidx modified the milestones: Discrete: M3, Integrated: M0 Mar 17, 2023
@moidx moidx removed the Priority:P2 Priority: medium label Mar 17, 2023
@msfschaffner msfschaffner added the Earlgrey-PROD Candidate Temporary label to triage issues into Earlgrey-PROD Milestones label Oct 6, 2023
@msfschaffner msfschaffner added Earlgrey-PROD Triaged Temporary label to triage issues into Earlgrey-PROD Milestones and removed Earlgrey-PROD Candidate Temporary label to triage issues into Earlgrey-PROD Milestones labels Jan 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component:Doc Documentation issue Earlgrey-PROD Triaged Temporary label to triage issues into Earlgrey-PROD Milestones Type:Task Tasks, to-do list.
Projects
None yet
Development

No branches or pull requests

4 participants