[pattgen,dv] General code tidy-up #26318
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
rswarbrick
force-pushed
the
pattgen-dv-tidyup-0
branch
from
c88cd6f to
e3242c5
Compare
No functional change, but this adds some documentation comments and also sets list_of_alerts a bit earlier (in the constructor instead of initialize()). Signed-off-by: Rupert Swarbrick <[email protected]>
The variable is only used in the stress sequence, so it probably makes more sense to define it there, instead of the base vseq. Signed-off-by: Rupert Swarbrick <[email protected]>
Signed-off-by: Rupert Swarbrick <[email protected]>
This is just getting rid of a class variable. Signed-off-by: Rupert Swarbrick <[email protected]>
Signed-off-by: Rupert Swarbrick <[email protected]>
Signed-off-by: Rupert Swarbrick <[email protected]>
Signed-off-by: Rupert Swarbrick <[email protected]>
Rather than a function that operates in place on a reference, just use a task: there's only ever one thing being modified. Also, change the naming to "left rotate", which matches how things look in MSB-first SystemVerilog. Signed-off-by: Rupert Swarbrick <[email protected]>
We now have a setup_pattgen_channel task, which takes the channel index as an argument. Signed-off-by: Rupert Swarbrick <[email protected]>
It's dramatically easier to use this with the channel index instead of the enum that we were using. Write the simpler version and add a documentation comment. Signed-off-by: Rupert Swarbrick <[email protected]>
This should be a little less tied in with the two channel setup and avoid a bit of repetition. Signed-off-by: Rupert Swarbrick <[email protected]>
Signed-off-by: Rupert Swarbrick <[email protected]>
The call sites already had the "is an error injected?" logic. Use that. Signed-off-by: Rupert Swarbrick <[email protected]>
Signed-off-by: Rupert Swarbrick <[email protected]>
This was an enum representing the four possibilities for a 2-bit bitmask in a rather confusing way. Get rid of it. Signed-off-by: Rupert Swarbrick <[email protected]>
This is only ever set to 50% and is used to weight the random choice of a bit (and it seems unlikely we'll ever care more about one polarity). Get rid of the unused customizability. Signed-off-by: Rupert Swarbrick <[email protected]>
rswarbrick
force-pushed
the
pattgen-dv-tidyup-0
branch
from
e3242c5 to
ab00925
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There a detailed notes in the commits, but a big chunk of this work is in
pattgen_base_vseq(which was quite long, so had several possible cleanups).