Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[crypto] Pass buffers by value. #20891

Merged
merged 4 commits into from
Jan 25, 2024

[crypto] Change the way RSA encryption returns plaintext len.

bd00104
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

[crypto] Pass buffers by value. #20891

[crypto] Change the way RSA encryption returns plaintext len.
bd00104
Select commit
Loading
Failed to load commit list.
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/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/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/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/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/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/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/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/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/ip/otbn/dv/verilator/otbn_top_sim.sv|506 col 101| Line length exceeds max: 100; is: 103 [Style: line-length] [line-length]
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/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/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/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/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/ip/kmac/fpv/tb/keccak_round_fpv.sv|202 col 101| Line length exceeds max: 100; is: 106 [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/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/dv/sv/csrng_agent/csrng_agent.sv|40 col 101| Line length exceeds max: 100; is: 101 [Style: line-length] [line-length]
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/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/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/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/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/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/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/vendor/google_riscv-dv/src/isa/riscv_csr_instr.sv|49 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/isa/riscv_csr_instr.sv|55 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/isa/riscv_csr_instr.sv|61 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/isa/riscv_csr_instr.sv|67 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/isa/riscv_csr_instr.sv|112 col 101| Line length exceeds max: 100; is: 136 [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/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/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/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/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/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/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/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/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/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/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/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/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/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/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_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/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/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/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/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]
... (Too many findings. Dropped some findings)