Skip to content

[otp_ctrl] Third batch of commit cherry-picks from integrated #6534

[otp_ctrl] Third batch of commit cherry-picks from integrated

[otp_ctrl] Third batch of commit cherry-picks from integrated #6534

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

reviewdog [verible-verilog-lint] report

reported by reviewdog 🐶

Findings (0)
Filtered Findings (299)

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/dv/sv/spi_agent/spi_agent_cfg.sv|115 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/aes/dv/env/seq_lib/aes_wake_up_vseq.sv|14 col 101| Line length exceeds max: 100; is: 122 [Style: line-length] [line-length]
hw/ip/aes/dv/env/seq_lib/aes_wake_up_vseq.sv|33 col 101| Line length exceeds max: 100; is: 143 [Style: line-length] [line-length]
hw/ip/aes/dv/env/seq_lib/aes_wake_up_vseq.sv|56 col 101| Line length exceeds max: 100; is: 142 [Style: line-length] [line-length]
hw/ip/aes/dv/env/seq_lib/aes_wake_up_vseq.sv|72 col 101| Line length exceeds max: 100; is: 120 [Style: line-length] [line-length]
hw/ip/aes/dv/env/seq_lib/aes_wake_up_vseq.sv|80 col 101| Line length exceeds max: 100; is: 107 [Style: line-length] [line-length]
hw/vendor/pulp_riscv_dbg/src/dmi_test.sv|337 col 11| File must end with a newline. [Style: posix-file-endings] [posix-eof]
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/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/uart/dv/env/uart_scoreboard.sv|173 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/uart/dv/env/uart_scoreboard.sv|339 col 13| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/uart/dv/env/uart_scoreboard.sv|470 col 13| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/uart/dv/env/uart_scoreboard.sv|488 col 44| 489:49: The lines can't be continued with '', use concatenation operator with braces [Style: forbid-line-continuations] [forbid-line-continuations]
hw/ip/uart/dv/env/uart_scoreboard.sv|499 col 44| 500:49: The lines can't be continued with '', use concatenation operator with braces [Style: forbid-line-continuations] [forbid-line-continuations]
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/top_earlgrey/ip/ast/rtl/vcc_pgd.sv|34 col 1| Use 'always_comb' instead of 'always @'. [Style: combinational-logic] [always-comb]
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/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/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/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/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/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/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/vendor/pulp_riscv_dbg/tb/dp_ram.sv|33 col 16| Explicitly define a storage type for every parameter and localparam, (bytes). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/dp_ram.sv|58 col 21| Do not use $test$plusargs to access plusargs, use $value$plusargs instead. [Style: plusarg-value-assignment] [plusarg-assignment]
hw/vendor/pulp_riscv_dbg/tb/dp_ram.sv|74 col 18| Explicitly define static or automatic lifetime for non-class functions [Style: function-task-explicit-lifetime] [explicit-function-lifetime]
hw/vendor/pulp_riscv_dbg/tb/dp_ram.sv|78 col 10| Explicitly define static or automatic lifetime for non-class tasks [Style: function-task-explicit-lifetime] [explicit-task-lifetime]
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/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/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/vendor/pulp_riscv_dbg/src/dmi_intf.sv|9 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/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/ip/otbn/dv/uvm/env/otbn_env_pkg.sv|90 col 5| Struct names have to end with _t [Style: struct-union-conventions] [struct-union-name-style]
hw/ip/sram_ctrl/dv/env/sram_ctrl_scoreboard.sv|32 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/syn/rtl/prim_clock_gating.v|16 col 3| Use 'always_comb' instead of 'always @
'. [Style: combinational-logic] [always-comb]
hw/ip/kmac/dv/env/seq_lib/kmac_base_vseq.sv|870 col 101| Line length exceeds max: 100; is: 110 [Style: line-length] [line-length]
hw/vendor/lowrisc_ibex/dv/uvm/icache/dv/tb/tb.sv|225 col 101| Line length exceeds max: 100; is: 108 [Style: line-length] [line-length]
hw/ip/otbn/dv/uvm/env/otbn_loop_if.sv|151 col 25| Explicitly define static or automatic lifetime for non-class functions [Style: function-task-explicit-lifetime] [explicit-function-lifetime]
hw/ip/otbn/dv/uvm/env/otbn_loop_if.sv|160 col 25| Explicitly define static or automatic lifetime for non-class functions [Style: function-task-explicit-lifetime] [explicit-function-lifetime]
hw/top_earlgrey/dv/env/seq_lib/chip_sw_sram_ctrl_scrambled_access_vseq.sv|20 col 101| Line length exceeds max: 100; is: 110 [Style: line-length] [line-length]
hw/top_earlgrey/dv/env/seq_lib/chip_sw_sram_ctrl_scrambled_access_vseq.sv|21 col 101| Line length exceeds max: 100; is: 108 [Style: line-length] [line-length]
hw/dv/sv/pattgen_agent/pattgen_agent_cfg.sv|15 col 15| 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/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/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/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/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/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/ip/prim/fpv/tb/prim_keccak_tb.sv|69 col 101| Line length exceeds max: 100; is: 106 [Style: line-length] [line-length]
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/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/top_earlgrey/ip/ast/rtl/vcaon_pgd.sv|34 col 1| Use 'always_comb' instead of 'always @*'. [Style: combinational-logic] [always-comb]
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/ip/i2c/dv/env/i2c_scoreboard.sv|253 col 9| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/vendor/pulp_riscv_dbg/tb/SimJTAG.sv|15 col 28| Explicitly define a storage type for every parameter and localparam, (TICK_DELAY). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/SimJTAG.sv|16 col 28| Explicitly define a storage type for every parameter and localparam, (PORT). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/SimJTAG.sv|43 col 31| $random is a forbidden system function or task, please use $urandom instead [Style: forbidden-system-functions] [invalid-system-task-function]
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]
hw/ip/spi_device/pre_dv/tb/spid_common.sv|589 col 9| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/spi_device/pre_dv/tb/spid_common.sv|652 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/spi_device/pre_dv/tb/spid_common.sv|711 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
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/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/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/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/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/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/dv/sv/csrng_agent/csrng_sequencer.sv|12 col 101| Line length exceeds max: 100; is: 101 [Style: line-length] [line-length]
hw/ip/spi_device/pre_dv/tb/spid_passthrough_tb.sv|7 col 8| Declared module does not match the first dot-delimited component of file name: "spid_passthrough_tb" [Style: file-names] [module-filename]
hw/vendor/lowrisc_ibex/dv/uvm/core_ibex/env/core_ibex_scoreboard.sv|105 col 101| Line length exceeds max: 100; is: 111 [Style: line-length] [line-length]
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/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/kmac/dv/env/kmac_scoreboard.sv|1404 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
... (Too many findings. Dropped some findings)