Skip to content

[bazel] Convert sigverify_key_validity to the new rules. #29564

[bazel] Convert sigverify_key_validity to the new rules.

[bazel] Convert sigverify_key_validity to the new rules. #29564

GitHub Actions / verible-verilog-lint succeeded Jan 8, 2024 in 0s

reviewdog [verible-verilog-lint] report

reported by reviewdog 🐶

Findings (0)
Filtered Findings (302)

hw/ip/spi_device/dv/env/seq_lib/spi_device_tpm_base_vseq.sv|50 col 65| $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_scoreboard.sv|363 col 9| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/aes/dv/env/aes_scoreboard.sv|477 col 13| 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/dv/sv/csrng_agent/csrng_agent.sv|40 col 101| Line length exceeds max: 100; is: 101 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/src/dmi_bscane_tap.sv|14 col 8| Declared module does not match the first dot-delimited component of file name: "dmi_bscane_tap" [Style: file-names] [module-filename]
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/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/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/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/vendor/lowrisc_ibex/dv/uvm/icache/dv/ibex_icache_mem_agent/ibex_icache_mem_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/vendor/lowrisc_ibex/dv/uvm/icache/dv/ibex_icache_mem_agent/ibex_icache_mem_resp_item.sv|29 col 14| Constraint names must by styled with lower_snake_case and end with _c. [Style: constraints] [constraint-name-style]
hw/vendor/lowrisc_ibex/dv/uvm/icache/dv/ibex_icache_mem_agent/ibex_icache_mem_resp_item.sv|39 col 14| Constraint names must by styled with lower_snake_case and end with _c. [Style: constraints] [constraint-name-style]
hw/ip/flash_ctrl/dv/env/seq_lib/flash_ctrl_hw_sec_otp_vseq.sv|309 col 101| Line length exceeds max: 100; is: 104 [Style: line-length] [line-length]
hw/vendor/lowrisc_ibex/vendor/google_riscv-dv/test/riscv_instr_cov_test.sv|67 col 101| Line length exceeds max: 100; is: 103 [Style: line-length] [line-length]
hw/vendor/lowrisc_ibex/vendor/google_riscv-dv/test/riscv_instr_cov_test.sv|119 col 9| Do not use $test$plusargs to access plusargs, use $value$plusargs instead. [Style: plusarg-value-assignment] [plusarg-assignment]
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/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/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/tl_agent/tl_seq_item.sv|11 col 9| Macro names must contain only CAPITALS, underscores, and digits. Exception: UVM-like macros. [Style: defines] [macro-name-style]
hw/dv/sv/tl_agent/tl_seq_item.sv|15 col 9| Macro names must contain only CAPITALS, underscores, and digits. Exception: UVM-like macros. [Style: defines] [macro-name-style]
hw/dv/sv/tl_agent/tl_seq_item.sv|22 col 9| Macro names must contain only CAPITALS, underscores, and digits. Exception: UVM-like macros. [Style: defines] [macro-name-style]
hw/dv/sv/tl_agent/tl_seq_item.sv|27 col 9| Macro names must contain only CAPITALS, underscores, and digits. Exception: UVM-like macros. [Style: defines] [macro-name-style]
hw/dv/sv/tl_agent/tl_seq_item.sv|31 col 9| Macro names must contain only CAPITALS, underscores, and digits. Exception: UVM-like macros. [Style: defines] [macro-name-style]
hw/dv/sv/tl_agent/tl_seq_item.sv|35 col 9| Macro names must contain only CAPITALS, underscores, and digits. Exception: UVM-like macros. [Style: defines] [macro-name-style]
hw/dv/sv/tl_agent/tl_seq_item.sv|119 col 14| Constraint names must by styled with lower_snake_case and end with _c. [Style: constraints] [constraint-name-style]
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/aes/dv/env/seq_lib/aes_base_vseq.sv|412 col 9| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/vendor/pulp_riscv_dbg/tb/jtag_dmi/jtag_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/top_earlgrey/dv/env/seq_lib/chip_sw_power_virus_vseq.sv|65 col 101| Line length exceeds max: 100; is: 116 [Style: line-length] [line-length]
hw/ip/prim/rtl/prim_assert.sv|124 col 101| Line length exceeds max: 100; is: 103 [Style: line-length] [line-length]
hw/ip/prim/rtl/prim_assert.sv|129 col 101| Line length exceeds max: 100; is: 108 [Style: line-length] [line-length]
hw/ip/prim/rtl/prim_assert.sv|167 col 101| Line length exceeds max: 100; is: 114 [Style: line-length] [line-length]
hw/vendor/lowrisc_ibex/vendor/google_riscv-dv/src/riscv_pmp_cfg.sv|607 col 101| Line length exceeds max: 100; is: 154 [Style: line-length] [line-length]
hw/ip/aes/dv/env/seq_lib/aes_nist_vectors_vseq.sv|78 col 101| Line length exceeds max: 100; is: 119 [Style: line-length] [line-length]
hw/ip/aes/dv/env/seq_lib/aes_nist_vectors_vseq.sv|108 col 101| Line length exceeds max: 100; is: 129 [Style: line-length] [line-length]
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/sram_ctrl/dv/env/sram_ctrl_scoreboard.sv|31 col 5| Enum names must use lower_snake_case naming convention and end with _t or _e. [Style: enumerations] [enum-name-style]
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/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/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/vendor/lowrisc_ibex/vendor/google_riscv-dv/src/riscv_instr_gen_config.sv|173 col 101| Line length exceeds max: 100; is: 168 [Style: line-length] [line-length]
hw/vendor/lowrisc_ibex/vendor/google_riscv-dv/src/riscv_instr_gen_config.sv|176 col 101| Line length exceeds max: 100; is: 102 [Style: line-length] [line-length]
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/tb_test_env.sv|34 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_test_env.sv|35 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_test_env.sv|37 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_test_env.sv|38 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_test_env.sv|40 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_test_env.sv|99 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_test_env.sv|101 col 12| Declare packed dimension range in little-endian (decreasing) order, e.g. [N-1:0]. [Style: packed-ordering] [packed-dimensions-range-ordering]
hw/top_earlgrey/dv/env/seq_lib/chip_sw_uart_rand_baudrate_vseq.sv|14 col 14| Explicitly define a storage type for every parameter and localparam, (NCO_WIDTH). [Style: constants] [explicit-parameter-storage-type]
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/vendor/lowrisc_ibex/dv/uvm/icache/dv/env/ibex_icache_scoreboard.sv|20 col 14| Explicitly define a storage type for every parameter and localparam, (BusWidth). [Style: constants] [explicit-parameter-storage-type]
hw/ip/aes/pre_dv/aes_sbox_tb/rtl/aes_sbox_tb.sv|67 col 15| $random is a forbidden system function or task, please use $urandom instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/ip/aes/pre_dv/aes_sbox_tb/rtl/aes_sbox_tb.sv|87 col 15| $random is a forbidden system function or task, please use $urandom instead [Style: forbidden-system-functions] [invalid-system-task-function]
hw/vendor/lowrisc_ibex/dv/uvm/core_ibex/tests/core_ibex_base_test.sv|73 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/vendor/lowrisc_ibex/dv/uvm/icache/dv/ibex_icache_core_agent/ibex_icache_core_req_item.sv|27 col 14| Constraint names must by styled with lower_snake_case and end with _c. [Style: constraints] [constraint-name-style]
hw/vendor/lowrisc_ibex/dv/uvm/icache/dv/ibex_icache_core_agent/ibex_icache_core_req_item.sv|38 col 14| Constraint names must by styled with lower_snake_case and end with _c. [Style: constraints] [constraint-name-style]
hw/vendor/lowrisc_ibex/dv/uvm/icache/dv/ibex_icache_core_agent/ibex_icache_core_req_item.sv|45 col 14| Constraint names must by styled with lower_snake_case and end with _c. [Style: constraints] [constraint-name-style]
hw/vendor/lowrisc_ibex/dv/uvm/icache/dv/ibex_icache_core_agent/ibex_icache_core_req_item.sv|56 col 14| Constraint names must by styled with lower_snake_case and end with _c. [Style: constraints] [constraint-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/top_earlgrey/ip/ast/rtl/vcc_pgd.sv|34 col 1| Use 'always_comb' instead of 'always @*'. [Style: combinational-logic] [always-comb]
hw/ip/otbn/pre_sca/alma/rtl/otbn_top_coco.v|7 col 13| Explicitly define a storage type for every parameter and localparam, (ImemDataWidth). [Style: constants] [explicit-parameter-storage-type]
hw/ip/otbn/pre_sca/alma/rtl/otbn_top_coco.v|9 col 13| Explicitly define a storage type for every parameter and localparam, (WLEN). [Style: constants] [explicit-parameter-storage-type]
hw/ip/otbn/pre_sca/alma/rtl/otbn_top_coco.v|11 col 13| Explicitly define a storage type for every parameter and localparam, (SideloadKeyWidth). [Style: constants] [explicit-parameter-storage-type]
hw/ip/otbn/pre_sca/alma/rtl/otbn_top_coco.v|25 col 14| Explicitly define a storage type for every parameter and localparam, (ImemSizeByte). [Style: constants] [explicit-parameter-storage-type]
hw/ip/otbn/pre_sca/alma/rtl/otbn_top_coco.v|27 col 14| Explicitly define a storage type for every parameter and localparam, (DmemSizeByte). [Style: constants] [explicit-parameter-storage-type]
hw/ip/otbn/pre_sca/alma/rtl/otbn_top_coco.v|29 col 14| Explicitly define a storage type for every parameter and localparam, (ExtWLEN). [Style: constants] [explicit-parameter-storage-type]
hw/ip/otbn/pre_sca/alma/rtl/otbn_top_coco.v|55 col 14| Explicitly define a storage type for every parameter and localparam, (ImemAddrWidth). [Style: constants] [explicit-parameter-storage-type]
hw/ip/otbn/pre_sca/alma/rtl/otbn_top_coco.v|57 col 14| Explicitly define a storage type for every parameter and localparam, (DmemAddrWidth). [Style: constants] [explicit-parameter-storage-type]
hw/ip/otbn/pre_sca/alma/rtl/otbn_top_coco.v|91 col 20| Explicitly define a storage type for every parameter and localparam, (MuBi4False). [Style: constants] [explicit-parameter-storage-type]
hw/ip/otbn/pre_sca/alma/rtl/otbn_top_coco.v|195 col 14| Explicitly define a storage type for every parameter and localparam, (DmemSizeWords). [Style: constants] [explicit-parameter-storage-type]
hw/ip/otbn/pre_sca/alma/rtl/otbn_top_coco.v|196 col 14| Explicitly define a storage type for every parameter and localparam, (DmemIndexWidth). [Style: constants] [explicit-parameter-storage-type]
hw/ip/otbn/pre_sca/alma/rtl/otbn_top_coco.v|222 col 14| Explicitly define a storage type for every parameter and localparam, (ImemSizeWords). [Style: constants] [explicit-parameter-storage-type]
hw/ip/otbn/pre_sca/alma/rtl/otbn_top_coco.v|223 col 14| Explicitly define a storage type for every parameter and localparam, (ImemIndexWidth). [Style: constants] [explicit-parameter-storage-type]
hw/ip/prim/rtl/prim_flop_macros.sv|24 col 101| Line length exceeds max: 100; is: 111 [Style: line-length] [line-length]
hw/ip/prim/rtl/prim_flop_macros.sv|44 col 101| Line length exceeds max: 100; is: 164 [Style: line-length] [line-length]
hw/ip/rom_ctrl/dv/env/rom_ctrl_scoreboard.sv|145 col 101| Line length exceeds max: 100; is: 112 [Style: line-length] [line-length]
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/lowrisc_ibex/dv/uvm/core_ibex/env/core_ibex_instr_monitor_if.sv|10 col 13| Explicitly define a storage type for every parameter and localparam, (DATA_WIDTH). [Style: constants] [explicit-parameter-storage-type]
hw/dv/sv/tl_agent/tl_reg_adapter.sv|21 col 66| The 'name' argument of type_id::create() must match the name of the variable to which it is assigned: parent_sequence, got: "m_tl_host_base_seq". [Style: uvm-naming] [create-object-name-match]
hw/vendor/pulp_riscv_dbg/tb/jtag_dmi/tb_jtag_dmi.sv|17 col 14| 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/tb_jtag_dmi.sv|17 col 42| Binary literal 32'b1 has less digits than expected for 32 bits. [Style: number-literals] [undersized-binary-literal]
hw/vendor/pulp_riscv_dbg/tb/jtag_dmi/tb_jtag_dmi.sv|142 col 14| Explicitly define a storage type for every parameter and localparam, (IRLength). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/jtag_dmi/tb_jtag_dmi.sv|188 col 14| Explicitly define a storage type for every parameter and localparam, (NrRandomTransactions). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/jtag_dmi/tb_jtag_dmi.sv|250 col 11| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/vendor/pulp_riscv_dbg/tb/jtag_dmi/tb_jtag_dmi.sv|291 col 87| Do not use consecutive null statements like ';;'. [Style: redundant-semicolons] [forbid-consecutive-null-statements]
hw/dv/sv/csr_utils/csr_seq_lib.sv|371 col 13| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
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/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/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]
... (Too many findings. Dropped some findings)