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

[i2c,dv] Update testplan for new features introduced for D2(S) #22337

Merged

Conversation

hcallahan-lowrisc
Copy link
Contributor

This includes requirements for test coverage and functional coverage

  • Rename old 'perf' testpoints to 'maxperf', to make way for a new point 'host_mode_config_perf' which checks performance matches whatever perf is configured.
  • Addition of V2 test descriptions, currently unmapped

Closes #22168
Goes towards #21006

This includes requirements for test coverage and functional coverage
- Rename old 'perf' testpoints to 'maxperf', to make way for a new point
'host_mode_config_perf' which checks performance matches whatever perf is
configured.
- Addition of V2 test descriptions.

Signed-off-by: Harry Callahan <[email protected]>
Copy link
Contributor

@andreaskurth andreaskurth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thx @hcallahan-lowrisc, this LGTM 👍

@andreaskurth andreaskurth mentioned this pull request Mar 29, 2024
@andreaskurth
Copy link
Contributor

The failure of the CW310 Test ROM Tests CI check is unrelated to the changes in this PR; all other checks have passed. Merging

@andreaskurth andreaskurth merged commit 3b4bd98 into lowRISC:master Mar 29, 2024
32 of 34 checks passed
@hcallahan-lowrisc hcallahan-lowrisc deleted the i2c_testplan_updates_for_v2 branch March 29, 2024 15:23
@hcallahan-lowrisc hcallahan-lowrisc mentioned this pull request Jun 6, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component:DV DV issue: testbench, test case, etc. IP:i2c Milestone:V1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[i2c,dv] Extend testplan to cover changes to design since ES tapeout
2 participants