Update lowrisc_ibex to lowRISC/ibex@123d46b4 #6229
reviewdog [verible-verilog-lint] report
reported by reviewdog 🐶
Findings (2)
hw/ip/rv_core_ibex/rtl/rv_core_ibex.sv|930 col 101| Line length exceeds max: 100; is: 107 [Style: line-length] [line-length]
hw/ip/rv_core_ibex/rtl/rv_core_ibex.sv|932 col 101| Line length exceeds max: 100; is: 107 [Style: line-length] [line-length]
Filtered Findings (302)
hw/ip/kmac/dv/env/kmac_scoreboard.sv|1404 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/kmac/dv/env/kmac_scoreboard.sv|1507 col 101| Line length exceeds max: 100; is: 101 [Style: line-length] [line-length]
hw/ip/kmac/dv/env/kmac_scoreboard.sv|1532 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/aes/dv/env/aes_env_cfg.sv|188 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/kmac/dv/env/kmac_env.sv|43 col 101| Line length exceeds max: 100; is: 105 [Style: line-length] [line-length]
hw/dv/sv/tl_agent/tl_agent_cfg.sv|210 col 101| Line length exceeds max: 100; is: 110 [Style: line-length] [line-length]
hw/ip/otbn/dv/uvm/env/seq_lib/otbn_base_vseq.sv|333 col 33| Explicitly define a storage type for every function parameter. [Style: function-task-argument-types] [explicit-function-task-parameter-type]
hw/vendor/lowrisc_ibex/dv/uvm/core_ibex/tests/core_ibex_vseq.sv|55 col 70| The 'name' argument of type_id::create() must match the name of the variable to which it is assigned: irq_raise_single_seq_h, got: "irq_single_seq_h". [Style: uvm-naming] [create-object-name-match]
hw/vendor/lowrisc_ibex/dv/uvm/icache/dv/fcov/ibex_icache_fcov_if.sv|242 col 31| All generate block statements must have a label [Style: generate-statements] [generate-label]
hw/vendor/lowrisc_ibex/dv/uvm/icache/dv/fcov/ibex_icache_fcov_if.sv|245 col 16| All generate block statements must have a label [Style: generate-statements] [generate-label]
hw/dv/sv/dv_base_reg/dv_base_reg.sv|133 col 41| 134:90: The lines can't be continued with '', use concatenation operator with braces [Style: forbid-line-continuations] [forbid-line-continuations]
hw/dv/sv/dv_base_reg/dv_base_reg.sv|142 col 41| 143:83: The lines can't be continued with '', use concatenation operator with braces [Style: forbid-line-continuations] [forbid-line-continuations]
hw/dv/sv/dv_base_reg/dv_base_reg.sv|152 col 41| 153:84: The lines can't be continued with '', use concatenation operator with braces [Style: forbid-line-continuations] [forbid-line-continuations]
hw/vendor/pulp_riscv_dbg/sva/dm_top_sva.sv|57 col 5| All generate block statements must have a label [Style: generate-statements] [generate-label]
hw/vendor/pulp_riscv_dbg/sva/dm_top_sva.sv|60 col 101| Line length exceeds max: 100; is: 133 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/sva/dm_top_sva.sv|61 col 101| Line length exceeds max: 100; is: 149 [Style: line-length] [line-length]
hw/vendor/lowrisc_ibex/dv/uvm/core_ibex/common/ibex_mem_intf_agent/ibex_mem_intf.sv|5 col 11| Interface names must use lower_snake_case naming convention and end with _if. [Style: interface-conventions] [interface-name-style]
hw/vendor/pulp_riscv_dbg/tb/tb_top.sv|53 col 13| Do not use $test$plusargs to access plusargs, use $value$plusargs instead. [Style: plusarg-value-assignment] [plusarg-assignment]
hw/vendor/pulp_riscv_dbg/tb/tb_top.sv|58 col 13| Do not use $test$plusargs to access plusargs, use $value$plusargs instead. [Style: plusarg-value-assignment] [plusarg-assignment]
hw/vendor/pulp_riscv_dbg/tb/tb_top.sv|71 col 16| Do not use $test$plusargs to access plusargs, use $value$plusargs instead. [Style: plusarg-value-assignment] [plusarg-assignment]
hw/vendor/pulp_riscv_dbg/tb/tb_top.sv|100 col 12| Do not use $test$plusargs to access plusargs, use $value$plusargs instead. [Style: plusarg-value-assignment] [plusarg-assignment]
hw/vendor/lowrisc_ibex/dv/uvm/core_ibex/fcov/core_ibex_fcov_if.sv|99 col 13| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/vendor/lowrisc_ibex/dv/uvm/core_ibex/fcov/core_ibex_fcov_if.sv|119 col 11| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/prim/pre_dv/prim_sync_reqack/rtl/prim_sync_reqack_tb.sv|30 col 101| Line length exceeds max: 100; is: 115 [Style: line-length] [line-length]
hw/ip/prim/pre_dv/prim_sync_reqack/rtl/prim_sync_reqack_tb.sv|111 col 13| $random is a forbidden system function or task, please use $urandom instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/dv/env/aes_seq_item.sv|237 col 101| Line length exceeds max: 100; is: 102 [Style: line-length] [line-length]
hw/ip/aes/dv/env/aes_seq_item.sv|242 col 101| Line length exceeds max: 100; is: 102 [Style: line-length] [line-length]
hw/ip/aes/dv/env/aes_seq_item.sv|247 col 101| Line length exceeds max: 100; is: 102 [Style: line-length] [line-length]
hw/ip/aes/dv/env/aes_seq_item.sv|252 col 101| Line length exceeds max: 100; is: 102 [Style: line-length] [line-length]
hw/ip/aes/dv/env/aes_seq_item.sv|331 col 18| $psprintf is a forbidden system function or task, please use $sformatf instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/dv/env/aes_seq_item.sv|333 col 18| $psprintf is a forbidden system function or task, please use $sformatf instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/dv/env/aes_seq_item.sv|335 col 18| $psprintf is a forbidden system function or task, please use $sformatf instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/dv/env/aes_seq_item.sv|337 col 18| $psprintf is a forbidden system function or task, please use $sformatf instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/dv/env/aes_seq_item.sv|339 col 18| $psprintf is a forbidden system function or task, please use $sformatf instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/dv/env/aes_seq_item.sv|339 col 101| Line length exceeds max: 100; is: 102 [Style: line-length] [line-length]
hw/ip/aes/dv/env/aes_seq_item.sv|340 col 101| Line length exceeds max: 100; is: 104 [Style: line-length] [line-length]
hw/ip/aes/dv/env/aes_seq_item.sv|341 col 18| $psprintf is a forbidden system function or task, please use $sformatf instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/dv/env/aes_seq_item.sv|343 col 19| $psprintf is a forbidden system function or task, please use $sformatf instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/dv/env/aes_seq_item.sv|345 col 18| $psprintf is a forbidden system function or task, please use $sformatf instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/dv/env/aes_seq_item.sv|347 col 19| $psprintf is a forbidden system function or task, please use $sformatf instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/dv/env/aes_seq_item.sv|353 col 18| $psprintf is a forbidden system function or task, please use $sformatf instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/dv/env/aes_seq_item.sv|354 col 18| $psprintf is a forbidden system function or task, please use $sformatf instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/dv/env/aes_seq_item.sv|355 col 18| $psprintf is a forbidden system function or task, please use $sformatf instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/dv/env/aes_seq_item.sv|356 col 18| $psprintf is a forbidden system function or task, please use $sformatf instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/dv/env/aes_seq_item.sv|357 col 18| $psprintf is a forbidden system function or task, please use $sformatf instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/dv/env/aes_seq_item.sv|365 col 19| $psprintf is a forbidden system function or task, please use $sformatf instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/gpio/dv/env/gpio_scoreboard.sv|287 col 17| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|27 col 14| Explicitly define a storage type for every parameter and localparam, (EsFifoDepth). [Style: constants] [explicit-parameter-storage-type]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|28 col 14| Explicitly define a storage type for every parameter and localparam, (MAX_INTRP_CNT). [Style: constants] [explicit-parameter-storage-type]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|29 col 14| Explicitly define a storage type for every parameter and localparam, (WD_DELAY). [Style: constants] [explicit-parameter-storage-type]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|107 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|132 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|162 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|177 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|204 col 17| randomize() is forbidden within void casts [Style: void-casts] [void-cast]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|237 col 101| Line length exceeds max: 100; is: 122 [Style: line-length] [line-length]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|240 col 101| Line length exceeds max: 100; is: 107 [Style: line-length] [line-length]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|242 col 101| Line length exceeds max: 100; is: 122 [Style: line-length] [line-length]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|245 col 101| Line length exceeds max: 100; is: 107 [Style: line-length] [line-length]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|247 col 101| Line length exceeds max: 100; is: 122 [Style: line-length] [line-length]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|257 col 101| Line length exceeds max: 100; is: 119 [Style: line-length] [line-length]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|261 col 101| Line length exceeds max: 100; is: 115 [Style: line-length] [line-length]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|262 col 101| Line length exceeds max: 100; is: 113 [Style: line-length] [line-length]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|266 col 101| Line length exceeds max: 100; is: 108 [Style: line-length] [line-length]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|267 col 101| Line length exceeds max: 100; is: 122 [Style: line-length] [line-length]
hw/ip/entropy_src/pre_dv/entropy_src_tb.sv|268 col 101| Line length exceeds max: 100; is: 120 [Style: line-length] [line-length]
hw/vendor/lowrisc_ibex/dv/uvm/core_ibex/common/ibex_mem_intf_agent/ibex_mem_intf_response_seq_lib.sv|107 col 101| Line length exceeds max: 100; is: 114 [Style: line-length] [line-length]
hw/vendor/lowrisc_ibex/dv/riscv_compliance/rtl/riscv_testutil.sv|47 col 14| Explicitly define a storage type for every parameter and localparam, (ADDR_HALT). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/lowrisc_ibex/dv/riscv_compliance/rtl/riscv_testutil.sv|48 col 14| Explicitly define a storage type for every parameter and localparam, (ADDR_SET_BEGIN_SIGNATURE). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/lowrisc_ibex/dv/riscv_compliance/rtl/riscv_testutil.sv|49 col 14| Explicitly define a storage type for every parameter and localparam, (ADDR_SET_END_SIGNATURE). [Style: constants] [explicit-parameter-storage-type]
hw/dv/sv/dv_lib/dv_base_env.sv|37 col 101| Line length exceeds max: 100; is: 102 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/jtag_dmi/jtag_test.sv|13 col 15| Explicitly define a storage type for every parameter and localparam, (IDCODE). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/jtag_dmi/jtag_test.sv|129 col 15| Explicitly define a storage type for every parameter and localparam, (IDCODE). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/jtag_dmi/jtag_test.sv|130 col 15| Explicitly define a storage type for every parameter and localparam, (DTMCSR). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/jtag_dmi/jtag_test.sv|131 col 15| Explicitly define a storage type for every parameter and localparam, (DMIACCESS). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/jtag_dmi/jtag_test.sv|136 col 101| Line length exceeds max: 100; is: 106 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/jtag_dmi/jtag_test.sv|139 col 16| Explicitly define a storage type for every parameter and localparam, (DMIWidth). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/jtag_dmi/jtag_test.sv|223 col 58| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|199 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|253 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|275 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|313 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|345 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|357 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|367 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|372 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|383 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|413 col 9| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|436 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|437 col 28| Explicitly define a storage type for every function parameter. [Style: function-task-argument-types] [explicit-function-task-parameter-type]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|450 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|451 col 27| Explicitly define a storage type for every function parameter. [Style: function-task-argument-types] [explicit-function-task-parameter-type]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|464 col 8| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
hw/ip/spi_device/pre_dv/tb/spi_tpm_tb.sv|466 col 23| Explicitly define a storage type for every function parameter. [Style: function-task-argument-types] [explicit-function-task-parameter-type]
hw/ip/flash_ctrl/dv/env/flash_ctrl_env_cfg.sv|564 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/flash_ctrl/dv/env/seq_lib/flash_ctrl_otp_reset_vseq.sv|108 col 20| Binary literal 2'b1 has less digits than expected for 2 bits. [Style: number-literals] [undersized-binary-literal]
hw/vendor/lowrisc_ibex/vendor/google_riscv-dv/src/riscv_load_store_instr_lib.sv|606 col 101| Line length exceeds max: 100; is: 101 [Style: line-length] [line-length]
hw/vendor/lowrisc_ibex/vendor/google_riscv-dv/src/riscv_load_store_instr_lib.sv|620 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/vendor/lowrisc_ibex/vendor/google_riscv-dv/src/riscv_load_store_instr_lib.sv|639 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/pwm/dv/tb.sv|18 col 14| Explicitly define a storage type for every parameter and localparam, (PWM_NUM_CHANNELS). [Style: constants] [explicit-parameter-storage-type]
hw/dv/sv/csrng_agent/csrng_item.sv|21 col 14| Constraint names must by styled with lower_snake_case and end with _c. [Style: constraints] [constraint-name-style]
hw/dv/sv/csrng_agent/csrng_item.sv|25 col 14| Constraint names must by styled with lower_snake_case and end with _c. [Style: constraints] [constraint-name-style]
hw/dv/sv/csrng_agent/csrng_item.sv|30 col 14| Constraint names must by styled with lower_snake_case and end with _c. [Style: constraints] [constraint-name-style]
hw/dv/sv/csrng_agent/csrng_item.sv|34 col 14| Constraint names must by styled with lower_snake_case and end with _c. [Style: constraints] [constraint-name-style]
hw/vendor/lowrisc_ibex/vendor/google_riscv-dv/src/riscv_asm_program_gen.sv|561 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/vendor/lowrisc_ibex/vendor/google_riscv-dv/src/riscv_asm_program_gen.sv|586 col 101| Line length exceeds max: 100; is: 105 [Style: line-length] [line-length]
hw/vendor/lowrisc_ibex/vendor/google_riscv-dv/src/riscv_asm_program_gen.sv|589 col 101| Line length exceeds max: 100; is: 105 [Style: line-length] [line-length]
hw/ip/aes/dv/env/aes_message_item.sv|50 col 101| Line length exceeds max: 100; is: 122 [Style: line-length] [line-length]
hw/ip/aes/dv/env/aes_message_item.sv|237 col 101| Line length exceeds max: 100; is: 108 [Style: line-length] [line-length]
hw/ip/aes/dv/env/aes_message_item.sv|281 col 101| Line length exceeds max: 100; is: 113 [Style: line-length] [line-length]
hw/vendor/lowrisc_ibex/dv/uvm/icache/dv/ibex_icache_core_agent/ibex_icache_core_protocol_checker.sv|13 col 11| Interface names must use lower_snake_case naming convention and end with _if. [Style: interface-conventions] [interface-name-style]
hw/ip/flash_ctrl/dv/env/seq_lib/flash_ctrl_rand_ops_base_vseq.sv|395 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/pattgen/dv/env/seq_lib/pattgen_base_vseq.sv|18 col 51| Unpacked dimension range must be declared in big-endian ([0:N-1]) order. Declare zero-based big-endian unpacked dimensions sized as [N]. [Style: unpacked-ordering] [unpacked-dimensions-range-ordering]
hw/ip/pattgen/dv/env/seq_lib/pattgen_base_vseq.sv|216 col 9| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/pattgen/dv/env/seq_lib/pattgen_base_vseq.sv|257 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/pattgen/dv/env/seq_lib/pattgen_base_vseq.sv|274 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/uart/dv/env/seq_lib/uart_intr_vseq.sv|54 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/top_earlgrey/dv/env/seq_lib/chip_sw_lc_ctrl_scrap_vseq.sv|34 col 9| Do not use $test$plusargs to access plusargs, use $value$plusargs instead. [Style: plusarg-value-assignment] [plusarg-assignment]
hw/dv/sv/dv_base_reg/csr_excl_item.sv|15 col 5| Struct names have to end with _t [Style: struct-union-conventions] [struct-union-name-style]
hw/vendor/pulp_riscv_dbg/tb/boot_rom.sv|21 col 29| Non-type localparam names must be styled with CamelCase or ALL_CAPS [Style: constants] [parameter-name-style]
hw/vendor/pulp_riscv_dbg/tb/tb_top_verilator.sv|31 col 16| Explicitly define a storage type for every parameter and localparam, (CLUSTER_ID). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/tb_top_verilator.sv|32 col 16| Explicitly define a storage type for every parameter and localparam, (CORE_ID). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/tb_top_verilator.sv|34 col 16| Explicitly define a storage type for every parameter and localparam, (CORE_MHARTID). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/tb_top_verilator.sv|35 col 16| Explicitly define a storage type for every parameter and localparam, (NrHarts). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/tb_top_verilator.sv|37 col 16| Explicitly define a storage type for every parameter and localparam, (HARTINFO). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/tb_top_verilator.sv|96 col 12| Declare packed dimension range in little-endian (decreasing) order, e.g. [N-1:0]. [Style: packed-ordering] [packed-dimensions-range-ordering]
hw/vendor/pulp_riscv_dbg/tb/tb_top_verilator.sv|98 col 12| Declare packed dimension range in little-endian (decreasing) order, e.g. [N-1:0]. [Style: packed-ordering] [packed-dimensions-range-ordering]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|73 col 5| enum types always should be named using typedef. [Style: typedef-enums] [typedef-enums]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|75 col 5| enum types always should be named using typedef. [Style: typedef-enums] [typedef-enums]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|195 col 101| Line length exceeds max: 100; is: 106 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|230 col 101| Line length exceeds max: 100; is: 106 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|265 col 101| Line length exceeds max: 100; is: 101 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|271 col 101| Line length exceeds max: 100; is: 110 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|272 col 101| Line length exceeds max: 100; is: 101 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|273 col 101| Line length exceeds max: 100; is: 101 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|275 col 101| Line length exceeds max: 100; is: 104 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|280 col 101| Line length exceeds max: 100; is: 101 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|313 col 101| Line length exceeds max: 100; is: 101 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|315 col 101| Line length exceeds max: 100; is: 110 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|317 col 101| Line length exceeds max: 100; is: 101 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|319 col 101| Line length exceeds max: 100; is: 101 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|323 col 101| Line length exceeds max: 100; is: 104 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|325 col 101| Line length exceeds max: 100; is: 101 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|362 col 101| Line length exceeds max: 100; is: 108 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|370 col 101| Line length exceeds max: 100; is: 102 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|444 col 17| Do not use $test$plusargs to access plusargs, use $value$plusargs instead. [Style: plusarg-value-assignment] [plusarg-assignment]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|498 col 13| Do not use $test$plusargs to access plusargs, use $value$plusargs instead. [Style: plusarg-value-assignment] [plusarg-assignment]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|521 col 59| Use spaces, not tabs. [Style: tabs] [no-tabs]
hw/vendor/pulp_riscv_dbg/tb/mm_ram.sv|524 col 59| Use spaces, not tabs. [Style: tabs] [no-tabs]
hw/dv/sv/alert_esc_agent/esc_monitor.sv|144 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
... (Too many findings. Dropped some findings)
Annotations
Check warning on line 930 in hw/ip/rv_core_ibex/rtl/rv_core_ibex.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] hw/ip/rv_core_ibex/rtl/rv_core_ibex.sv#L930
Line length exceeds max: 100; is: 107 [Style: line-length] [line-length]
Raw output
message:"Line length exceeds max: 100; is: 107 [Style: line-length] [line-length]" location:{path:"./hw/ip/rv_core_ibex/rtl/rv_core_ibex.sv" range:{start:{line:930 column:101}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 932 in hw/ip/rv_core_ibex/rtl/rv_core_ibex.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] hw/ip/rv_core_ibex/rtl/rv_core_ibex.sv#L932
Line length exceeds max: 100; is: 107 [Style: line-length] [line-length]
Raw output
message:"Line length exceeds max: 100; is: 107 [Style: line-length] [line-length]" location:{path:"./hw/ip/rv_core_ibex/rtl/rv_core_ibex.sv" range:{start:{line:932 column:101}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}