[prim] Add Trivium/Bivium stream cipher primitives #29409
reviewdog [verible-verilog-lint] report
reported by reviewdog 🐶
Findings (1)
hw/ip/prim/rtl/prim_trivium.sv|270 col 45| Do not use consecutive null statements like ';;'. [Style: redundant-semicolons] [forbid-consecutive-null-statements]
Filtered Findings (303)
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/top_earlgrey/ip/ast/rtl/vio_pgd.sv|34 col 1| Use 'always_comb' instead of 'always @'. [Style: combinational-logic] [always-comb]
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/rstmgr/dv/env/seq_lib/rstmgr_smoke_vseq.sv|15 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/src/dmi_test.sv|337 col 11| File must end with a newline. [Style: posix-file-endings] [posix-eof]
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/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/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/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/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/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/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/vendor/lowrisc_ibex/dv/uvm/core_ibex/tests/core_ibex_test_lib.sv|926 col 101| Line length exceeds max: 100; is: 105 [Style: line-length] [line-length]
hw/vendor/lowrisc_ibex/dv/uvm/core_ibex/tests/core_ibex_test_lib.sv|1087 col 23| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/vendor/lowrisc_ibex/dv/uvm/core_ibex/tests/core_ibex_test_lib.sv|1330 col 101| Line length exceeds max: 100; is: 104 [Style: line-length] [line-length]
hw/vendor/lowrisc_ibex/dv/uvm/core_ibex/tests/core_ibex_test_lib.sv|1390 col 101| Line length exceeds max: 100; is: 122 [Style: line-length] [line-length]
hw/vendor/lowrisc_ibex/dv/uvm/core_ibex/tests/core_ibex_test_lib.sv|1752 col 101| Line length exceeds max: 100; is: 105 [Style: line-length] [line-length]
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/lowrisc_ibex/dv/uvm/icache/dv/ibex_icache_core_agent/seq_lib/ibex_icache_core_base_seq.sv|65 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/seq_lib/ibex_icache_core_base_seq.sv|72 col 14| Constraint names must by styled with lower_snake_case and end with _c. [Style: constraints] [constraint-name-style]
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/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/clkmgr/dv/env/clkmgr_scoreboard.sv|196 col 101| Line length exceeds max: 100; is: 106 [Style: line-length] [line-length]
hw/ip/clkmgr/dv/env/clkmgr_scoreboard.sv|202 col 101| Line length exceeds max: 100; is: 116 [Style: line-length] [line-length]
hw/ip/clkmgr/dv/env/clkmgr_scoreboard.sv|208 col 101| Line length exceeds max: 100; is: 116 [Style: line-length] [line-length]
hw/ip/clkmgr/dv/env/clkmgr_scoreboard.sv|213 col 101| Line length exceeds max: 100; is: 110 [Style: line-length] [line-length]
hw/ip/clkmgr/dv/env/clkmgr_scoreboard.sv|218 col 101| Line length exceeds max: 100; is: 108 [Style: line-length] [line-length]
hw/top_earlgrey/ip/ast/rtl/vcmain_pgd.sv|34 col 1| Use 'always_comb' instead of 'always @'. [Style: combinational-logic] [always-comb]
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/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/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/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/tlul/generic_dv/env/seq_lib/xbar_stress_all_with_rand_reset_vseq.sv|29 col 61| The 'name' argument of type_id::create() must match the name of the variable to which it is assigned: xbar_vseq, got: "xbar_stress_all_vseq". [Style: uvm-naming] [create-object-name-match]
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/ip/otbn/dv/uvm/env/seq_lib/otbn_common_vseq.sv|146 col 21| 147:43: The lines can't be continued with '', use concatenation operator with braces [Style: forbid-line-continuations] [forbid-line-continuations]
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/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/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/ip/pattgen/dv/env/pattgen_scoreboard.sv|21 col 41| 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/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/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/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/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/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/vendor/lowrisc_ibex/dv/uvm/core_ibex/riscv_dv_extension/ibex_directed_instr_lib.sv|354 col 23| Binary literal 32'b10000111 has less digits than expected for 32 bits. [Style: number-literals] [undersized-binary-literal]
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/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/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/dv/sv/csrng_agent/csrng_sequencer.sv|12 col 101| Line length exceeds max: 100; is: 101 [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/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/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/top_earlgrey/ip/ast/rtl/vcc_pgd.sv|34 col 1| Use 'always_comb' instead of 'always @'. [Style: combinational-logic] [always-comb]
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/vendor/lowrisc_ibex/dv/uvm/core_ibex/tb/core_ibex_tb_top.sv|50 col 13| Macro names must contain only CAPITALS, underscores, and digits. Exception: UVM-like macros. [Style: defines] [macro-name-style]
hw/vendor/lowrisc_ibex/dv/uvm/core_ibex/tb/core_ibex_tb_top.sv|233 col 101| Line length exceeds max: 100; is: 109 [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/cip_lib/seq_lib/cip_base_vseq.sv|5 col 9| Macro names must contain only CAPITALS, underscores, and digits. Exception: UVM-like macros. [Style: defines] [macro-name-style]
hw/dv/sv/cip_lib/seq_lib/cip_base_vseq.sv|234 col 101| Line length exceeds max: 100; is: 109 [Style: line-length] [line-length]
hw/dv/sv/cip_lib/seq_lib/cip_base_vseq.sv|243 col 101| Line length exceeds max: 100; is: 110 [Style: line-length] [line-length]
hw/dv/sv/cip_lib/seq_lib/cip_base_vseq.sv|244 col 101| Line length exceeds max: 100; is: 101 [Style: line-length] [line-length]
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/vendor/lowrisc_ibex/syn/rtl/prim_clock_gating.v|16 col 3| Use 'always_comb' instead of 'always @*'. [Style: combinational-logic] [always-comb]
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/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/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/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/spi_device/dv/env/seq_lib/spi_device_base_vseq.sv|359 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/spi_device/dv/env/seq_lib/spi_device_base_vseq.sv|384 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_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/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/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/ip/otbn/dv/verilator/otbn_top_sim.sv|506 col 101| Line length exceeds max: 100; is: 103 [Style: line-length] [line-length]
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/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/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]
... (Too many findings. Dropped some findings)
Annotations
Check warning on line 270 in hw/ip/prim/rtl/prim_trivium.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] hw/ip/prim/rtl/prim_trivium.sv#L270
Do not use consecutive null statements like ';;'. [Style: redundant-semicolons] [forbid-consecutive-null-statements]
Raw output
message:"Do not use consecutive null statements like ';;'. [Style: redundant-semicolons] [forbid-consecutive-null-statements]" location:{path:"./hw/ip/prim/rtl/prim_trivium.sv" range:{start:{line:270 column:45}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}