[WIP] Fix compilation & optimization of PULP iDMA wrap, register FE
reviewdog [verible-verilog-lint] report
reported by reviewdog 🐶
Findings (27)
src/pulp_idma_wrap.sv|21 col 1| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
src/pulp_idma_wrap.sv|35 col 41| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
src/pulp_idma_wrap.sv|38 col 28| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
src/pulp_idma_wrap.sv|40 col 52| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
src/pulp_idma_wrap.sv|43 col 27| Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
src/pulp_idma_wrap.sv|48 col 48| 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]
src/pulp_idma_wrap.sv|49 col 45| 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]
src/pulp_idma_wrap.sv|50 col 46| 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]
src/pulp_idma_wrap.sv|58 col 25| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
src/pulp_idma_wrap.sv|363 col 31| Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
src/pulp_idma_wrap.sv|366 col 31| Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
src/pulp_idma_wrap.sv|367 col 31| Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
src/pulp_idma_wrap.sv|469 col 31| Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
src/pulp_idma_wrap.sv|472 col 31| Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
src/pulp_idma_wrap.sv|473 col 31| Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
src/pulp_idma_wrap.sv|475 col 29| Explicitly define static or automatic lifetime for non-class functions [Style: function-task-explicit-lifetime] [explicit-function-lifetime]
src/pulp_idma_wrap.sv|475 col 61| Explicitly define a storage type for every function parameter. [Style: function-task-argument-types] [explicit-function-task-parameter-type]
src/pulp_idma_wrap.sv|488 col 101| Line length exceeds max: 100; is: 121 [Style: line-length] [line-length]
src/pulp_idma_wrap.sv|493 col 101| Line length exceeds max: 100; is: 118 [Style: line-length] [line-length]
src/pulp_idma_wrap.sv|589 col 50| All generate block statements must have a label [Style: generate-statements] [generate-label]
src/pulp_idma_wrap.sv|590 col 19| All generate block statements must have a label [Style: generate-statements] [generate-label]
src/pulp_idma_wrap.sv|591 col 37| Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
src/pulp_idma_wrap.sv|616 col 14| All generate block statements must have a label [Style: generate-statements] [generate-label]
src/pulp_idma_wrap.sv|627 col 50| All generate block statements must have a label [Style: generate-statements] [generate-label]
src/pulp_idma_wrap.sv|692 col 20| All generate block statements must have a label [Style: generate-statements] [generate-label]
src/pulp_idma_wrap.sv|692 col 101| Line length exceeds max: 100; is: 106 [Style: line-length] [line-length]
src/pulp_idma_wrap.sv|693 col 77| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
Filtered Findings (0)
Annotations
Check warning on line 21 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L21
Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
Raw output
message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:21 column:1}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"} suggestions:{range:{start:{line:21 column:1} end:{line:22}} text:"\n"}
Check warning on line 35 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L35
Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
Raw output
message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:35 column:41}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"} suggestions:{range:{start:{line:35 column:41} end:{line:38}} text:" // bidirectional streams: range 1 to 8\n // queue depth per stream\n // mux read ports between tcdm-tcdm and tcdm-axi?\n"}
Check warning on line 38 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L38
Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
Raw output
message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:38 column:28}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 40 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L40
Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
Raw output
message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:40 column:52}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 43 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L43
Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
Raw output
message:"Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:43 column:27}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 48 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L48
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]
Raw output
message:"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]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:48 column:48}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 49 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L49
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]
Raw output
message:"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]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:49 column:45}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 50 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L50
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]
Raw output
message:"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]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:50 column:46}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 58 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L58
Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
Raw output
message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:58 column:25}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"} suggestions:{range:{start:{line:58 column:25} end:{line:59}} text:"); // verilog_format: on\n"}
Check warning on line 363 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L363
Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
Raw output
message:"Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:363 column:31}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 366 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L366
Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
Raw output
message:"Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:366 column:31}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 367 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L367
Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
Raw output
message:"Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:367 column:31}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 469 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L469
Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
Raw output
message:"Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:469 column:31}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 472 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L472
Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
Raw output
message:"Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:472 column:31}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 473 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L473
Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
Raw output
message:"Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:473 column:31}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 475 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L475
Explicitly define static or automatic lifetime for non-class functions [Style: function-task-explicit-lifetime] [explicit-function-lifetime]
Raw output
message:"Explicitly define static or automatic lifetime for non-class functions [Style: function-task-explicit-lifetime] [explicit-function-lifetime]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:475 column:29}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 475 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L475
Explicitly define a storage type for every function parameter. [Style: function-task-argument-types] [explicit-function-task-parameter-type]
Raw output
message:"Explicitly define a storage type for every function parameter. [Style: function-task-argument-types] [explicit-function-task-parameter-type]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:475 column:61}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 488 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L488
Line length exceeds max: 100; is: 121 [Style: line-length] [line-length]
Raw output
message:"Line length exceeds max: 100; is: 121 [Style: line-length] [line-length]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:488 column:101}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 493 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L493
Line length exceeds max: 100; is: 118 [Style: line-length] [line-length]
Raw output
message:"Line length exceeds max: 100; is: 118 [Style: line-length] [line-length]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:493 column:101}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 589 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L589
All generate block statements must have a label [Style: generate-statements] [generate-label]
Raw output
message:"All generate block statements must have a label [Style: generate-statements] [generate-label]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:589 column:50}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 590 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L590
All generate block statements must have a label [Style: generate-statements] [generate-label]
Raw output
message:"All generate block statements must have a label [Style: generate-statements] [generate-label]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:590 column:19}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 591 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L591
Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
Raw output
message:"Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:591 column:37}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 616 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L616
All generate block statements must have a label [Style: generate-statements] [generate-label]
Raw output
message:"All generate block statements must have a label [Style: generate-statements] [generate-label]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:616 column:14}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 627 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L627
All generate block statements must have a label [Style: generate-statements] [generate-label]
Raw output
message:"All generate block statements must have a label [Style: generate-statements] [generate-label]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:627 column:50}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 692 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L692
All generate block statements must have a label [Style: generate-statements] [generate-label]
Raw output
message:"All generate block statements must have a label [Style: generate-statements] [generate-label]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:692 column:20}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}