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

[lc_ctrl/top_earlgrey] Disable RAW unlock feature and additional DV checks #21320

Closed

[ottool] Support chips with volatile RAW unlock disabled

db8f9b8
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Closed

[lc_ctrl/top_earlgrey] Disable RAW unlock feature and additional DV checks #21320

[ottool] Support chips with volatile RAW unlock disabled
db8f9b8
Select commit
Loading
Failed to load commit list.
GitHub Actions / verible-verilog-lint succeeded Feb 15, 2024 in 0s

reviewdog [verible-verilog-lint] report

reported by reviewdog 🐶

Findings (0)
Filtered Findings (290)

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/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/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/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/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/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/vendor/google_riscv-dv/src/riscv_illegal_instr.sv|145 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/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/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/spi_device/pre_dv/tb/spid_status_tb.sv|95 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/spid_status_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/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/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/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/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/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/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/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/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/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/ip/otbn/pre_sca/alma/rtl/ram_1p.v|9 col 13| Explicitly define a storage type for every parameter and localparam, (DataWidth). [Style: constants] [explicit-parameter-storage-type]
hw/ip/otbn/pre_sca/alma/rtl/ram_1p.v|10 col 13| Explicitly define a storage type for every parameter and localparam, (AddrWidth). [Style: constants] [explicit-parameter-storage-type]
hw/ip/otbn/pre_sca/alma/rtl/ram_1p.v|11 col 13| Explicitly define a storage type for every parameter and localparam, (Depth). [Style: constants] [explicit-parameter-storage-type]
hw/ip/otbn/pre_sca/alma/rtl/ram_1p.v|24 col 27| 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/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_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/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/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/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/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/dv/uvm/icache/dv/tb/tb.sv|225 col 101| Line length exceeds max: 100; is: 108 [Style: line-length] [line-length]
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/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/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/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/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/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/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/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/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/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/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/top_earlgrey/ip/ast/rtl/vcaon_pgd.sv|34 col 1| Use 'always_comb' instead of 'always @'. [Style: combinational-logic] [always-comb]
hw/top_earlgrey/dv/env/seq_lib/chip_padctrl_attributes_vseq.sv|256 col 101| Line length exceeds max: 100; is: 101 [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/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/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/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/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/vendor/pulp_riscv_dbg/tb/tb_top_verilator.sv|31 col 16| Explicitly define a storage type for every parameter and localparam, (CLUSTER_ID). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/tb_top_verilator.sv|32 col 16| Explicitly define a storage type for every parameter and localparam, (CORE_ID). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/tb_top_verilator.sv|34 col 16| Explicitly define a storage type for every parameter and localparam, (CORE_MHARTID). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/tb_top_verilator.sv|35 col 16| Explicitly define a storage type for every parameter and localparam, (NrHarts). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/tb_top_verilator.sv|37 col 16| Explicitly define a storage type for every parameter and localparam, (HARTINFO). [Style: constants] [explicit-parameter-storage-type]
hw/vendor/pulp_riscv_dbg/tb/tb_top_verilator.sv|96 col 12| Declare packed dimension range in little-endian (decreasing) order, e.g. [N-1:0]. [Style: packed-ordering] [packed-dimensions-range-ordering]
hw/vendor/pulp_riscv_dbg/tb/tb_top_verilator.sv|98 col 12| Declare packed dimension range in little-endian (decreasing) order, e.g. [N-1:0]. [Style: packed-ordering] [packed-dimensions-range-ordering]
hw/vendor/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/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_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/vendor/google_riscv-dv/src/riscv_load_store_instr_lib.sv|606 col 101| Line length exceeds max: 100; is: 101 [Style: line-length] [line-length]
hw/vendor/lowrisc_ibex/vendor/google_riscv-dv/src/riscv_load_store_instr_lib.sv|620 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/vendor/lowrisc_ibex/vendor/google_riscv-dv/src/riscv_load_store_instr_lib.sv|639 col 7| Explicitly define a default case for every case statement. [Style: case-statements] [case-missing-default]
hw/ip/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/otbn/dv/verilator/otbn_top_sim.sv|506 col 101| Line length exceeds max: 100; is: 103 [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/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/dv/sv/dv_lib/dv_base_env.sv|37 col 101| Line length exceeds max: 100; is: 102 [Style: line-length] [line-length]
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/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/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]
... (Too many findings. Dropped some findings)