[dv,sram] add axi_sram verification plan and tag architecture spec - #667
[dv,sram] add axi_sram verification plan and tag architecture spec#667tchilikov-semify wants to merge 4 commits into
Conversation
martin-velay
left a comment
There was a problem hiding this comment.
I have some comments, and also I'd like to know if you feel it was a bad experience to create a vPlan based on DVPlan? I think you are among th 1st to do the full exercise (except myself) and I need to get feedback 😃
59c77a5 to
f588fc4
Compare
|
Thanks for the feedback, @martin-velay 😃 Regarding DVPlan: I found that the flow and working with the tool was actually quite nice and polished (so far). One pain point I have though, is the spec annotation capabilities. A random generated tag means that each time the spec is updated and changed, all of the tags would be re-generated. This would make updating the vplans a bit of a nightmare, since all of the tags would have to be re-mapped. I think a solution to this could be a per-line / per-sentence hash. This way, the hash only changes if the contents of the line change. Of course, this sounds simple but it would introduce edge cases (white spaces, punctuation, etc). I have yet to re-generate or re-map anything, but I anticipate that this would be tedious as it currently stands. Other than this, it was quite straight-forward! I plan to map the actual SV testcases with a comment to the tags as well. |
Thanks @tchilikov-semify for the feedback. |
5ca16c0 to
d7e5e65
Compare
e4af936 to
3fba033
Compare
marnovandermaas
left a comment
There was a problem hiding this comment.
I've left some reviews on the initial commits (the vplan I'll leave to @martin-velay
e7efcf5 to
4f6c554
Compare
7df23d3 to
ccbd09d
Compare
f03a862 to
7b67315
Compare
7b67315 to
5ded936
Compare
5ded936 to
beee705
Compare
6477b85 to
fa76f28
Compare
fa76f28 to
46f6f12
Compare
This PR adds a verification plan for the axi_sram, and makes some minor updates to the specification. It also tags the specification and adds a markdown memory map table to it.