Skip to content
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

Apio verify/lint commands now process also all the testbench files. #344

Merged
merged 1 commit into from
Feb 21, 2024
Merged

Conversation

zapta
Copy link
Contributor

@zapta zapta commented Feb 21, 2024

Before this PR, apio verify and apio lint commands processed only the verilog module files (.v files which are not testbenches). This PR adds also all the testbench files. I believe that this is more useful and it was also the case before the adding support for multiple testbenches.

NOTE: for the link command, I don't have a verilator installed so tested until the point it invokes the verilator.

--> DEBUG!. Function process_arguments(). END
     Returns: 
      * ['fpga_arch=ice40', 'fpga_size=5k', 'fpga_type=up', 'fpga_pack=sg48', 'top_module=main']
      * upduino31
      * ice40

verilator --lint-only -Wno-TIMESCALEMOD deserializer.v main.v queue.v queue_pusher.v sensor_timing.v deserializer_tb.v queue_tb.v sensor_timing_tb.v
sh: verilator: command not found
scons: *** [hardware] Error 127

…efore it processed only the module (non testbench) files.
@Obijuan
Copy link
Member

Obijuan commented Feb 21, 2024

Verilator should be installed with the apio oss-cad-suite (0.0.9 or later). Currently is not working ok. I have to fix that
Thanks for all your contribution. Are very very welcome! 😀️

@zapta
Copy link
Contributor Author

zapta commented Feb 21, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants