Skip to content

[spatz_controller] latch issue fix #150

[spatz_controller] latch issue fix

[spatz_controller] latch issue fix #150

GitHub Actions / verible-verilog-lint failed Nov 15, 2024 in 0s

reviewdog [verible-verilog-lint] report

reported by reviewdog 🐶

Findings (17)

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]
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|423 col 1| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
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|294 col 53| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]
hw/ip/spatz/src/spatz.sv|342 col 1| Remove trailing spaces. [Style: trailing-spaces] [no-trailing-spaces]

Filtered Findings (0)

Annotations

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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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"}

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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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 423 in hw/ip/spatz/src/spatz_controller.sv

See this annotation in the file changed.

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

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

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:423  column:1}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}  suggestions:{range:{start:{line:423  column:1}  end:{line:424}}  text:"\n"}

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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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

See this annotation in the file changed.

@github-actions 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 294 in hw/ip/spatz/src/spatz.sv

See this annotation in the file changed.

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

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

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:294  column:53}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}  suggestions:{range:{start:{line:294  column:53}  end:{line:295}}  text:"      vrf_wdata_buf[VFU_VD_WD] = vrf_buf_data.wdata;\n"}

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

See this annotation in the file changed.

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

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

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:342  column:1}}}  severity:WARNING  source:{name:"verible-verilog-lint"  url:"https://github.com/chipsalliance/verible"}  suggestions:{range:{start:{line:342  column:1}  end:{line:343}}  text:"\n"}