Hadir Khan
05/08/2022, 11:04 PMreg_spi_enable = 1;
commented out here: https://github.com/efabless/caravel_user_project/blob/main/verilog/dv/io_ports/io_ports.c#L49
3. Why do we need to disable debug mode in the la_test: https://github.com/efabless/caravel_user_project/blob/main/verilog/dv/la_test1/la_test1_tb.v#L104 whereas it is not required in the io_ports test.
4. Why are we enabling/disabling CSB
in the testbench: https://github.com/efabless/caravel_user_project/blob/main/verilog/dv/mprj_stimulus/mprj_stimulus_tb.v#L79-L86 when that is just a reg and is not being passed anywhere to the dut?
There are a lot of ambiguities with the current dv setup and I will appreciate some documentation (atleast comments behind the motive) or a reply to this thread so I can send a PR for documentation.Tim Edwards
05/09/2022, 1:34 PMHadir Khan
05/09/2022, 9:42 PM