[csrng/dv] Adjust csrng_intr test to properly test sw/hw_cmd_ack_sts #22869
Assignees
Labels
Component:DV
DV issue: testbench, test case, etc.
IP:csrng
Subsystem:Entropy
entropy_src, csrng, or edn related issues
Milestone
Comments
vogelpi
added
Component:DV
vogelpi
added a commit
to vogelpi/opentitan
that referenced
this issue
Apr 28, 2024
lowRISC#21981 reworked the status signaling of CSRNG and changed the definition of the ack_sts signal from 1-bit to a multi-bit enum which broke the csrng_intr vseq trying to force the ack and ack_sts signals. This commit gets the sequence working again until it's reworked to properly test the error signaling without forcing signals. This is related to lowRISC#22869. Signed-off-by: Pirmin Vogel <[email protected]>
vogelpi
added a commit
that referenced
this issue
Apr 29, 2024
#21981 reworked the status signaling of CSRNG and changed the definition of the ack_sts signal from 1-bit to a multi-bit enum which broke the csrng_intr vseq trying to force the ack and ack_sts signals. This commit gets the sequence working again until it's reworked to properly test the error signaling without forcing signals. This is related to #22869. Signed-off-by: Pirmin Vogel <[email protected]>
Closed
Closed
|
@h-filali investigated this. The scoreboard is disabled in this test so it would be quite some work to do all the checking properly. It would be doable if the checking was done right in the sequence but we agreed that there are more important things to take care of first. I am thus moving this to M7 for the moment but raising the priority. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
With #21981, CSRNG gained support for properly triggering and signaling command errors back to EDN as well as the software instance. This is great. However, the
csrng_intrtest still tests the corresponding status register bits by forcing them. And there is TODO in the DV code referring to the original issue #16516Now that CSRNG can actually signal these errors, the
csrng_intrtest should be reworked to properly trigger those errors instead of just forcing the corresponding status bits.The text was updated successfully, but these errors were encountered: