fusesocotb icon indicating copy to clipboard operation
fusesocotb copied to clipboard

Example has errors

Open psi-chuffine opened this issue 1 year ago • 5 comments

INFO: Preparing ::vlog_tb_utils:1.1-r1
INFO: Preparing ::axis_send_packet:0
iverilog -saxis_send_packet -svlog_tb_utils -c axis_send_packet_0.scr -o axis_send_packet_0
:0: error: parameter `DW` not found in `vlog_tb_utils`.
:0: error: parameter `MSG_LEN` not found in `vlog_tb_utils`.
:0: error: parameter `MSG_STR` not found in `vlog_tb_utils`.
4 error(s) during elaboration.
make: *** [Makefile:8: axis_send_packet_0] Error 4

psi-chuffine avatar Jan 18 '24 22:01 psi-chuffine

I see. With my version of icarus, these are only warnings. What version of Icarus are you using? Perhaps we can add some flag to downgrade this to a warning. That would be my preferred solution in the short term.

olofk avatar Jan 19 '24 15:01 olofk

I am using the toolchain packaged in OSS CAD Suite: https://github.com/YosysHQ/oss-cad-suite-build

iverilog -v returns Icarus Verilog version 13.0 (devel) (s20221226-397-gb1e602d27)

What version are you using?

psi-chuffine avatar Jan 19 '24 15:01 psi-chuffine

Looks like this commit changed the warning to an error and there doesn't seem to be a way to turn it off.

As a quick workaround you can change toplevel : [axis_send_packet, vlog_tb_utils] to toplevel : [axis_send_packet] in axis_send_packet.core but then you also lose the VCD generation. Let me look into a better way to fix this long-term.

olofk avatar Jan 19 '24 15:01 olofk

I can confirm that removing the vlog_tb_utils top level does allow the cocotb test to execute successfully.

psi-chuffine avatar Jan 19 '24 15:01 psi-chuffine

Thanks. That's good to know. I also filed a FuseSoC bug to fix this long-term https://github.com/olofk/fusesoc/issues/667

olofk avatar Jan 19 '24 16:01 olofk

I fixed this in another way and believe we can close this now.

olofk avatar Aug 22 '24 11:08 olofk