[rstmgr,dv] Reset Manager DV ported to mocha#506
[rstmgr,dv] Reset Manager DV ported to mocha#506KolosKoblasz-Semify wants to merge 4 commits intolowRISC:mainfrom
Conversation
martin-velay
changed the title
| - files_rtl | ||
| - files_dv | ||
| default_tool: vcs | ||
| default_tool: xcelium |
There was a problem hiding this comment.
This is the source of your issue with the src file being not generated. The .src file is your file list generated from FuseSoc. I know that confusing, I went into this trap earlier too.
The VCS backend generates a .scr file, which is an identical format to a more standard .f format. We are only running FuseSoc up to the point it generates the filelist file (.scr in this case).
KolosKoblasz-Semify
force-pushed
the
kk_rst_mgr_dv
branch
2 times, most recently
from
d3730e3 to
a74803a
Compare
* open titan specific paths changed to mocha specific ones in the template files * 0001_Fix_Paths_And_Tools.patch file created with the changes Signed-off-by: Kolos Koblasz <kolos.koblasz@semify-eda.com>
KolosKoblasz-Semify
force-pushed
the
kk_rst_mgr_dv
branch
3 times, most recently
from
e105ee4 to
d7f29db
Compare
* 0002 patches fix the auto generated reset manager's dv files to be run Signed-off-by: Kolos Koblasz <kolos.koblasz@semify-eda.com>
* Reset Manager DV paths fixed * Template files fixed to enable running block level simulations Signed-off-by: Kolos Koblasz <kolos.koblasz@semify-eda.com>
* Since templates files were chnaged to enable running block level simulations the rstmgr files had to be regenerated and commited. Signed-off-by: Kolos Koblasz <kolos.koblasz@semify-eda.com>
KolosKoblasz-Semify
force-pushed
the
kk_rst_mgr_dv
branch
from
d7f29db to
7d337b2
Compare
| // Wait till rst_lc_n is inactive for non-aon. | ||
| `DV_WAIT(cfg.rstmgr_vif.resets_o.rst_lc_n[1]) | ||
| // Wait till rst_por_aon_n is inactive for non-aon. | ||
| `DV_WAIT(cfg.rstmgr_vif.resets_o.rst_por_aon_n[1]) |
There was a problem hiding this comment.
I think this should be the same reset as the one fed into the reset manager block in the top which is rst_por_io_n
There was a problem hiding this comment.
In rstmgr.sv line 351 constant zero is assigned to it:
There was a problem hiding this comment.
Probably makes sense to switch rst_*lc* with rst_por_io_n everywhere (and Marno agrees as well).
There was a problem hiding this comment.
Okay, but then it is DomainAonSel right?
There was a problem hiding this comment.
rst_lc_n[1] -> rst_por_io_n[1] so this is going to be Domain0Sel?
There was a problem hiding this comment.
Yes, DomainAonSel is 0, Domain0Sel is 1.
4 participants
TO DO
This PR will address: #433