-
Notifications
You must be signed in to change notification settings - Fork 896
[dv,ottf,spi_console] Add objects for interacting with a DUT's OTTF SPI Console #28194
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
[dv,ottf,spi_console] Add objects for interacting with a DUT's OTTF SPI Console #28194
Conversation
f0c2417
to
e26aab4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I really enjoyed reading this: it's very clear and enjoyable to go through. I've left quite a few small comments (crikey! more than I thought!), but they are all small things: it's really nice code!
…ctions See header comments in ottf_spi_console_pkg.sv for more details. Signed-off-by: Harry Callahan <[email protected]>
Signed-off-by: Harry Callahan <[email protected]>
…and EOP detection Signed-off-by: Harry Callahan <[email protected]>
e26aab4
to
a68bd6d
Compare
In moving the class to it's own file, the diff from the last push is mostly useless for reviewing the feedback changes. Sorry! |
aae256c
into
lowRISC:earlgrey_1.0.0
Successfully created backport PR for |
Instantiate this new piece of testbench machinery in the top_earlgrey testbench, leaving it connected but as of yet, unused.
This is in preparation for full simulations of manuf / provisioning stages using binaries built for the 'silicon_creator' exec_env, which do not use the backdoor software logger and make heavy use of the spi_console for data transfer during test execution.
Review commit-by-commit, but the first commit is the bulk of this changeset.