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.

Sign up for GitHub

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

Jump to bottom

[sram_ctrl/dv] Regwen test for READBACK feature #24358

Closed
nasahlpa opened this issue Aug 19, 2024 · 0 comments · Fixed by #24596
Closed

[sram_ctrl/dv] Regwen test for READBACK feature #24358

nasahlpa opened this issue Aug 19, 2024 · 0 comments · Fixed by #24596
Assignees
@nasahlpa
Labels
Component:DV DV issue: testbench, test case, etc. IP:sram_ctrl
Milestone
Earlgrey-PROD.M7

Comments

@nasahlpa
Copy link
Member

Description

Currently, no regwen test for the SRAM CTRL Readback feature is provided. sram_ctrl_regwen_vseq could be used to implement this test.
@nasahlpa nasahlpa added Component:DV DV issue: testbench, test case, etc. IP:sram_ctrl labels Aug 19, 2024
@nasahlpa nasahlpa added this to the Earlgrey-PROD.M7 milestone Aug 19, 2024
@nasahlpa nasahlpa self-assigned this Aug 19, 2024
nasahlpa added a commit to nasahlpa/opentitan that referenced this issue Sep 18, 2024
@nasahlpa
[sram_ctrl,dv] Add readback_regwen test
89e66a1 
To close a coverage gap, this commit enhances the regwen_vseq to also
cover the readback_regwen register.

Closes lowRISC#24358.

Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
@nasahlpa nasahlpa mentioned this issue Sep 18, 2024
Merged
nasahlpa added a commit to nasahlpa/opentitan that referenced this issue Sep 18, 2024
@nasahlpa
[sram_ctrl,dv] Add readback_regwen test
efeba07 
To close a coverage gap, this commit enhances the regwen_vseq to also
cover the readback_regwen register. To reflect this change, the
`sec_cm_readback_config_regwen` label is added. No functional
RTL change.

Closes lowRISC#24358.

Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
nasahlpa added a commit to nasahlpa/opentitan that referenced this issue Sep 19, 2024
@nasahlpa
[sram_ctrl,dv] Add readback_regwen test
b6c0326 
To close a coverage gap, this commit enhances the regwen_vseq to also
cover the readback_regwen register. To reflect this change, the
`sec_cm_readback_config_regwen` label is added. No functional
RTL change.

Now, EXPRESSION (readback_we & readback_regwen_qs) gets full cond.
coverage.

Closes lowRISC#24358.

Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
nasahlpa added a commit to nasahlpa/opentitan that referenced this issue Sep 20, 2024
@nasahlpa
[sram_ctrl,dv] Add readback_regwen test
c680d6e 
To close a coverage gap, this commit enhances the regwen_vseq to also
cover the readback_regwen register. To reflect this change, the
`sec_cm_readback_config_regwen` label is added. No functional
RTL change.

Now, EXPRESSION (readback_we & readback_regwen_qs) gets full cond.
coverage.

Closes lowRISC#24358.

Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
nasahlpa added a commit to nasahlpa/opentitan that referenced this issue Sep 20, 2024
@nasahlpa
[sram_ctrl,dv] Add readback_regwen test
b2058de 
To close a coverage gap, this commit enhances the regwen_vseq to also
cover the readback_regwen register. To reflect this change, the
`sec_cm_readback_config_regwen` label is added. No functional
RTL change.

Now, EXPRESSION (readback_we & readback_regwen_qs) gets full cond.
coverage.

Closes lowRISC#24358.

Signed-off-by: Pascal Nasahl <nasahlpa@lowrisc.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nasahlpa nasahlpa

Labels
Component:DV DV issue: testbench, test case, etc. IP:sram_ctrl
Projects
None yet
Milestone
Earlgrey-PROD.M7
Development

Successfully merging a pull request may close this issue.

[sram_ctrl,dv] Add readback_regwen test nasahlpa/opentitan
1 participant
@nasahlpa