[hw/ip/spatz] also buffer rsp from vfu #181

GitHub Actions / verible-verilog-lint failed Dec 19, 2024 in 0s

reviewdog [verible-verilog-lint] report

reported by reviewdog 🐶

Findings (20)

hw/ip/spatz/src/spatz.sv|101 col 24| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz.sv|102 col 23| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz.sv|104 col 1| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz.sv|352 col 1| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz_vrf.sv|99 col 1| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz_vrf.sv|105 col 1| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz_controller.sv|284 col 1| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz_controller.sv|287 col 29| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz_controller.sv|288 col 1| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz_controller.sv|292 col 87| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz_controller.sv|298 col 1| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz_controller.sv|307 col 1| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz_controller.sv|331 col 1| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz_controller.sv|335 col 1| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz_controller.sv|339 col 1| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz_controller.sv|434 col 1| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz_vlsu.sv|446 col 31| Localparam name does not match the naming convention defined by regex pattern: (([A-Z0-9]+[a-z0-9])+(_[0-9]+)?) [Style: constants] [parameter-name-style]
hw/ip/spatz/src/spatz_vlsu.sv|681 col 31| Localparam name does not match the naming convention defined by regex pattern: (([A-Z0-9]+[a-z0-9])+([0-9]+)?) [Style: constants] [parameter-name-style]
hw/ip/spatz/src/spatz_vlsu.sv|719 col 31| Localparam name does not match the naming convention defined by regex pattern: (([A-Z0-9]+[a-z0-9]*)+([0-9]+)?) [Style: constants] [parameter-name-style]
hw/ip/spatz/src/spatz_vlsu.sv|1045 col 31| Localparam name does not match the naming convention defined by regex pattern: (([A-Z0-9]+[a-z0-9]*)+(_[0-9]+)?) [Style: constants] [parameter-name-style]

Filtered Findings (0)

Annotations

Check warning on line 101 in hw/ip/spatz/src/spatz.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz.sv#L101

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz.sv"  range:{start:{line:101  column:24}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}  suggestions:{range:{start:{line:101  column:24}  end:{line:104}}  text:"  logic vrf_vfu_wvalid;\n  logic vrf_buf_ready;\n\n"}

Check warning on line 102 in hw/ip/spatz/src/spatz.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz.sv#L102

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz.sv"  range:{start:{line:102  column:23}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}

Check warning on line 104 in hw/ip/spatz/src/spatz.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz.sv#L104

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz.sv"  range:{start:{line:104  column:1}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}

Check warning on line 352 in hw/ip/spatz/src/spatz.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz.sv#L352

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz.sv"  range:{start:{line:352  column:1}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}  suggestions:{range:{start:{line:352  column:1}  end:{line:353}}  text:"\n"}

Check warning on line 99 in hw/ip/spatz/src/spatz_vrf.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_vrf.sv#L99

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz_vrf.sv"  range:{start:{line:99  column:1}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}  suggestions:{range:{start:{line:99  column:1}  end:{line:100}}  text:"\n"}

Check warning on line 105 in hw/ip/spatz/src/spatz_vrf.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_vrf.sv#L105

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz_vrf.sv"  range:{start:{line:105  column:1}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}  suggestions:{range:{start:{line:105  column:1}  end:{line:106}}  text:"\n"}

Check warning on line 284 in hw/ip/spatz/src/spatz_controller.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_controller.sv#L284

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz_controller.sv"  range:{start:{line:284  column:1}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}  suggestions:{range:{start:{line:284  column:1}  end:{line:287}}  text:"\n      automatic logic intID;\n\n"}

Check warning on line 287 in hw/ip/spatz/src/spatz_controller.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_controller.sv#L287

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz_controller.sv"  range:{start:{line:287  column:29}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}

Check warning on line 288 in hw/ip/spatz/src/spatz_controller.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_controller.sv#L288

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz_controller.sv"  range:{start:{line:288  column:1}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}

Check warning on line 292 in hw/ip/spatz/src/spatz_controller.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_controller.sv#L292

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz_controller.sv"  range:{start:{line:292  column:87}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}  suggestions:{range:{start:{line:292  column:87}  end:{line:293}}  text:"      end else if (port inside {SB_VLSU_VD_RD0, SB_VLSU_VD_RD1, SB_VLSU_VD_WD1}) begin\n"}

Check warning on line 298 in hw/ip/spatz/src/spatz_controller.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_controller.sv#L298

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz_controller.sv"  range:{start:{line:298  column:1}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}  suggestions:{range:{start:{line:298  column:1}  end:{line:299}}  text:"\n"}

Check warning on line 307 in hw/ip/spatz/src/spatz_controller.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_controller.sv#L307

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz_controller.sv"  range:{start:{line:307  column:1}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}  suggestions:{range:{start:{line:307  column:1}  end:{line:308}}  text:"\n"}

Check warning on line 331 in hw/ip/spatz/src/spatz_controller.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_controller.sv#L331

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz_controller.sv"  range:{start:{line:331  column:1}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}  suggestions:{range:{start:{line:331  column:1}  end:{line:332}}  text:"\n"}

Check warning on line 335 in hw/ip/spatz/src/spatz_controller.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_controller.sv#L335

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz_controller.sv"  range:{start:{line:335  column:1}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}  suggestions:{range:{start:{line:335  column:1}  end:{line:336}}  text:"\n"}

Check warning on line 339 in hw/ip/spatz/src/spatz_controller.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_controller.sv#L339

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz_controller.sv"  range:{start:{line:339  column:1}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}  suggestions:{range:{start:{line:339  column:1}  end:{line:340}}  text:"\n"}

Check warning on line 434 in hw/ip/spatz/src/spatz_controller.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_controller.sv#L434

Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Raw output


            message:"Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]"  location:{path:"hw/ip/spatz/src/spatz_controller.sv"  range:{start:{line:434  column:1}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}  suggestions:{range:{start:{line:434  column:1}  end:{line:435}}  text:"\n"}

Check warning on line 446 in hw/ip/spatz/src/spatz_vlsu.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_vlsu.sv#L446

Localparam name does not match the naming convention defined by regex pattern: (([A-Z0-9]+[a-z0-9]*)+(_[0-9]+)?) [Style: constants] [parameter-name-style]

Raw output


            message:"Localparam name does not match the naming convention defined by regex pattern: (([A-Z0-9]+[a-z0-9]*)+(_[0-9]+)?) [Style: constants] [parameter-name-style]"  location:{path:"hw/ip/spatz/src/spatz_vlsu.sv"  range:{start:{line:446  column:31}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}

Check warning on line 681 in hw/ip/spatz/src/spatz_vlsu.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_vlsu.sv#L681

Localparam name does not match the naming convention defined by regex pattern: (([A-Z0-9]+[a-z0-9]*)+(_[0-9]+)?) [Style: constants] [parameter-name-style]

Raw output


            message:"Localparam name does not match the naming convention defined by regex pattern: (([A-Z0-9]+[a-z0-9]*)+(_[0-9]+)?) [Style: constants] [parameter-name-style]"  location:{path:"hw/ip/spatz/src/spatz_vlsu.sv"  range:{start:{line:681  column:31}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}

Check warning on line 719 in hw/ip/spatz/src/spatz_vlsu.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_vlsu.sv#L719

Localparam name does not match the naming convention defined by regex pattern: (([A-Z0-9]+[a-z0-9]*)+(_[0-9]+)?) [Style: constants] [parameter-name-style]

Raw output


            message:"Localparam name does not match the naming convention defined by regex pattern: (([A-Z0-9]+[a-z0-9]*)+(_[0-9]+)?) [Style: constants] [parameter-name-style]"  location:{path:"hw/ip/spatz/src/spatz_vlsu.sv"  range:{start:{line:719  column:31}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}

Check warning on line 1045 in hw/ip/spatz/src/spatz_vlsu.sv

github-actions / verible-verilog-lint

[verible-verilog-lint] hw/ip/spatz/src/spatz_vlsu.sv#L1045

Localparam name does not match the naming convention defined by regex pattern: (([A-Z0-9]+[a-z0-9]*)+(_[0-9]+)?) [Style: constants] [parameter-name-style]

Raw output


            message:"Localparam name does not match the naming convention defined by regex pattern: (([A-Z0-9]+[a-z0-9]*)+(_[0-9]+)?) [Style: constants] [parameter-name-style]"  location:{path:"hw/ip/spatz/src/spatz_vlsu.sv"  range:{start:{line:1045  column:31}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}

View more details on GitHub Actions

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[hw/ip/spatz] also buffer rsp from vfu #181

verible-verilog-lint