Skip to content

Cleanup (or waive) lint issues

Mike Thompson requested to merge github/fork/MikeOpenHWGroup/cv32e40p/dev into master

This PR contains several updates to address Verissimo lint issues:

cv32e40p/tb/core/mm_ram.sv:

  • created setup_transaction() function for common code between reads and writes.

cv32e40p/tb/uvmt/uvmt_cv32e40p_tb.sv:

  • remove dead code (unused variables)

lib/uvma_agents/uvma_obi_memory/src/comps/uvma_obi_memory_cov_model.sv:

  • add waiver pragmas

lib/uvma_agents/uvma_obi_memory/src/comps/uvma_obi_memory_drv.sv:

  • add waiver pragmas

lib/uvma_agents/uvma_obi_memory/src/comps/uvma_obi_memory_seq_item_log.sv:

  • add waiver pragmas @datum-dpoulin, please have a look at the updates to the obi_memory agent files. Verissimo has (correctly) identified redundant code. In at least one case (uvma_obi_memory_drv.sv) I think the redundancy is acceptable, so I have waived it. The other redunancies are (I believe) all related to the boiler-plate implementation of the agent. I have waived these as well, but we may want to update the templates.

Merge request reports

Loading