lowRISC / lowRISC/opentitan

Change Documentation covergroups to cover points

Open
#10,954 6 comments 0 reactions 6 assignees View on GitHub

@matutem is already working on this.

Since Mar 28, 2022.

Component:DV Earlgrey-PROD Triaged Priority:P3 Type:Enhancement Type:Question
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

In the current DV documentation, there is a list of covergroups.
unfortunately, this method is not practical in capturing the coverpoints.

as an example, in AES I have a cover point (currently documented as a covergroup)
![image](https://user-images.githubusercontent.com/53917183/154672579-6c2a436a-90f8-4632-b89a-96a4ed9b4e13.png)

I think this is a crucial cover point in terms of security - but I don't want to do a specific covergroup for this alone.
I already cover the 3 different registers in other covergroups, so I could easily add a coverpoint to each of those covergroups

which means this "covergroup" is now split into 3 coverpoints in three different groups.

I suggest we update the list to match the format for test points
```
covergroups: [
{
name: My_cover_point_a
desc: '''
Make sure that we cover this point '''
covergroup: ["My_cover_group"]
}
{
name: My_cover_point_B
desc: '''
Make sure that we cover this point also'''
covergroup: ["My_cover_group",]
}
{
name: My_cover_point_C
desc: '''
Make sure that we cover this point also'''
covergroup: ["my_cover_group","My_Other_cg"]
}
]

```

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.