-
Notifications
You must be signed in to change notification settings - Fork 705
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Complete Design Document #1865
Complete Design Document #1865
Conversation
fix id_stage implement the black_list ports temporary Generate black list ports rename python script Black reformat
Fix BHT image
core/alu.sv
Outdated
// Asynchronous reset active low - SUBSYSTEM | ||
input logic rst_ni, | ||
// FU data needed to execute instruction - ISSUE_STAGE | ||
input fu_data_t fu_data_i, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[verible-verilog-format] reported by reviewdog 🐶
input fu_data_t fu_data_i, | |
input fu_data_t fu_data_i, |
core/branch_unit.sv
Outdated
input logic is_compressed_instr_i, | ||
input logic fu_valid_i, // any functional unit is valid, check that there is no accidental mis-predict | ||
// any functional unit is valid, check that there is no accidental mis-predict - TO_BE_COMPLETED | ||
input logic fu_valid_i, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[verible-verilog-format] reported by reviewdog 🐶
input logic fu_valid_i, | |
input logic fu_valid_i, |
core/csr_buffer.sv
Outdated
// CSR FU is ready - ISSUE_STAGE | ||
output logic csr_ready_o, | ||
// CSR instruction is valid - ISSUE_STAGE | ||
input logic csr_valid_i, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[verible-verilog-format] reported by reviewdog 🐶
input logic csr_valid_i, | |
input logic csr_valid_i, |
core/csr_buffer.sv
Outdated
// CSR buffer result - ISSUE_STAGE | ||
output riscv::xlen_t csr_result_o, | ||
// commit the pending CSR OP - TO_BE_COMPLETED | ||
input logic csr_commit_i, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[verible-verilog-format] reported by reviewdog 🐶
input logic csr_commit_i, | |
input logic csr_commit_i, |
core/cvxif_fu.sv
Outdated
input logic clk_i, | ||
input logic rst_ni, | ||
// Subsystem Clock - SUBSYSTEM | ||
input logic clk_i, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[verible-verilog-format] reported by reviewdog 🐶
input logic clk_i, | |
input logic clk_i, |
core/multiplier.sv
Outdated
// Subsystem Clock - SUBSYSTEM | ||
input logic clk_i, | ||
// Asynchronous reset active low - SUBSYSTEM | ||
input logic rst_ni, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[verible-verilog-format] reported by reviewdog 🐶
input logic rst_ni, | |
input logic rst_ni, |
core/multiplier.sv
Outdated
output logic mult_ready_o, | ||
output logic [TRANS_ID_BITS-1:0] mult_trans_id_o | ||
// Multiplier transaction ID - Mult | ||
output logic [TRANS_ID_BITS-1:0] mult_trans_id_o |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[verible-verilog-format] reported by reviewdog 🐶
output logic [TRANS_ID_BITS-1:0] mult_trans_id_o | |
output logic [TRANS_ID_BITS-1:0] mult_trans_id_o |
core/scoreboard.sv
Outdated
|
||
// RVFI | ||
// Transaction ID at which to write the result back - TO_BE_COMPLETED | ||
input logic [CVA6Cfg.NrWbPorts-1:0][ariane_pkg::TRANS_ID_BITS-1:0] trans_id_i, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[verible-verilog-format] reported by reviewdog 🐶
input logic [CVA6Cfg.NrWbPorts-1:0][ariane_pkg::TRANS_ID_BITS-1:0] trans_id_i, | |
input logic [CVA6Cfg.NrWbPorts-1:0][ariane_pkg::TRANS_ID_BITS-1:0] trans_id_i, |
core/serdiv.sv
Outdated
// handshake | ||
input logic in_vld_i, // there is a cycle delay from in_rdy_o->in_vld_i, see issue_read_operands.sv stage | ||
// Serdiv instruction is valid - Mult | ||
input logic in_vld_i, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[verible-verilog-format] reported by reviewdog 🐶
input logic in_vld_i, | |
input logic in_vld_i, |
core/store_unit.sv
Outdated
input logic dtlb_hit_i, // will be one in the same cycle translation_req was asserted if it hits | ||
// address checker | ||
// Data TLB hit - lsu | ||
input logic dtlb_hit_i, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[verible-verilog-format] reported by reviewdog 🐶
input logic dtlb_hit_i, | |
input logic dtlb_hit_i, |
✔️ successful run, report available here. |
❌ failed run, report available here. |
Thnaks to Côme for the review, this has improve the a lot the specification