[WIP] add generated iDMA code to allow bender inclusion #244
reviewdog [verible-verilog-lint] report
reported by reviewdog 🐶
Findings (26)
src/pulp_idma_wrap.sv|33 col 41| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
src/pulp_idma_wrap.sv|36 col 28| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
src/pulp_idma_wrap.sv|38 col 52| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
src/pulp_idma_wrap.sv|41 col 27| Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
src/pulp_idma_wrap.sv|46 col 70| 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|47 col 65| 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|48 col 67| 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|48 col 101| Line length exceeds max: 100; is: 112 [Style: line-length] [line-length]
src/pulp_idma_wrap.sv|56 col 25| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
src/pulp_idma_wrap.sv|360 col 31| Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
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|364 col 31| Non-type localparam names must be styled with CamelCase [Style: constants] [parameter-name-style]
src/pulp_idma_wrap.sv|366 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|366 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|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|593 col 50| All generate block statements must have a label [Style: generate-statements] [generate-label]
src/pulp_idma_wrap.sv|594 col 19| All generate block statements must have a label [Style: generate-statements] [generate-label]
src/pulp_idma_wrap.sv|621 col 14| All generate block statements must have a label [Style: generate-statements] [generate-label]
src/pulp_idma_wrap.sv|632 col 50| All generate block statements must have a label [Style: generate-statements] [generate-label]
src/pulp_idma_wrap.sv|697 col 20| All generate block statements must have a label [Style: generate-statements] [generate-label]
src/pulp_idma_wrap.sv|697 col 101| Line length exceeds max: 100; is: 106 [Style: line-length] [line-length]
src/pulp_idma_wrap.sv|698 col 77| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
Filtered Findings (0)
Annotations
Check warning on line 33 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L33
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:33 column:41}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"} suggestions:{range:{start:{line:33 column:41} end:{line:36}} 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 36 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L36
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:36 column:28}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
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:52}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 41 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L41
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:41 column:27}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 46 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L46
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:46 column:70}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 47 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L47
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:47 column:65}}} 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:67}}} 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
Line length exceeds max: 100; is: 112 [Style: line-length] [line-length]
Raw output
message:"Line length exceeds max: 100; is: 112 [Style: line-length] [line-length]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:48 column:101}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 56 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L56
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:56 column:25}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"} suggestions:{range:{start:{line:56 column:25} end:{line:57}} text:"); // verilog_format: on\n"}
Check warning on line 360 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L360
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:360 column:31}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
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 364 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L364
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:364 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
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:366 column:29}}} 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
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:366 column:61}}} 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 593 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L593
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:593 column:50}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 594 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L594
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:594 column:19}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 621 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L621
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:621 column:14}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 632 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L632
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:632 column:50}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 697 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L697
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:697 column:20}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}
Check warning on line 697 in src/pulp_idma_wrap.sv
github-actions / verible-verilog-lint
[verible-verilog-lint] src/pulp_idma_wrap.sv#L697
Line length exceeds max: 100; is: 106 [Style: line-length] [line-length]
Raw output
message:"Line length exceeds max: 100; is: 106 [Style: line-length] [line-length]" location:{path:"src/pulp_idma_wrap.sv" range:{start:{line:697 column:101}}} severity:WARNING source:{name:"verible-verilog-lint" url:"https://github.com/chipsalliance/verible"}