Skip to content

midend: Add Mempool mid-end (#34)

Sign in for the full log view
GitHub Actions / verible-verilog-lint failed Oct 31, 2023 in 0s

reviewdog [verible-verilog-lint] report

reported by reviewdog 🐶

Findings (4)

src/midend/idma_mp_split_midend.sv|60 col 14| Explicitly define a storage type for every parameter and localparam, (DmaRegionAddressBits). [Style: constants] [explicit-parameter-storage-type]
src/midend/idma_mp_split_midend.sv|102 col 3| enum types always should be named using typedef. [Style: typedef-enums] [typedef-enums]
src/midend/idma_mp_dist_midend.sv|65 col 14| Explicitly define a storage type for every parameter and localparam, (DmaRegionAddressBits). [Style: constants] [explicit-parameter-storage-type]
src/midend/idma_mp_dist_midend.sv|66 col 14| Explicitly define a storage type for every parameter and localparam, (FullRegionAddressBits). [Style: constants] [explicit-parameter-storage-type]

Filtered Findings (0)

Annotations

Check warning on line 60 in src/midend/idma_mp_split_midend.sv

See this annotation in the file changed.

@github-actions github-actions / verible-verilog-lint

[verible-verilog-lint] src/midend/idma_mp_split_midend.sv#L60

Explicitly define a storage type for every parameter and localparam, (DmaRegionAddressBits). [Style: constants] [explicit-parameter-storage-type]
Raw output
message:"Explicitly define a storage type for every parameter and localparam, (DmaRegionAddressBits). [Style: constants] [explicit-parameter-storage-type]"  location:{path:"src/midend/idma_mp_split_midend.sv"  range:{start:{line:60  column:14}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}

Check warning on line 102 in src/midend/idma_mp_split_midend.sv

See this annotation in the file changed.

@github-actions github-actions / verible-verilog-lint

[verible-verilog-lint] src/midend/idma_mp_split_midend.sv#L102

enum types always should be named using typedef. [Style: typedef-enums] [typedef-enums]
Raw output
message:"enum types always should be named using typedef. [Style: typedef-enums] [typedef-enums]"  location:{path:"src/midend/idma_mp_split_midend.sv"  range:{start:{line:102  column:3}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}

Check warning on line 65 in src/midend/idma_mp_dist_midend.sv

See this annotation in the file changed.

@github-actions github-actions / verible-verilog-lint

[verible-verilog-lint] src/midend/idma_mp_dist_midend.sv#L65

Explicitly define a storage type for every parameter and localparam, (DmaRegionAddressBits). [Style: constants] [explicit-parameter-storage-type]
Raw output
message:"Explicitly define a storage type for every parameter and localparam, (DmaRegionAddressBits). [Style: constants] [explicit-parameter-storage-type]"  location:{path:"src/midend/idma_mp_dist_midend.sv"  range:{start:{line:65  column:14}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}

Check warning on line 66 in src/midend/idma_mp_dist_midend.sv

See this annotation in the file changed.

@github-actions github-actions / verible-verilog-lint

[verible-verilog-lint] src/midend/idma_mp_dist_midend.sv#L66

Explicitly define a storage type for every parameter and localparam, (FullRegionAddressBits). [Style: constants] [explicit-parameter-storage-type]
Raw output
message:"Explicitly define a storage type for every parameter and localparam, (FullRegionAddressBits). [Style: constants] [explicit-parameter-storage-type]"  location:{path:"src/midend/idma_mp_dist_midend.sv"  range:{start:{line:66  column:14}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}