-
Notifications
You must be signed in to change notification settings - Fork 73
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
Add trigger logic tests in LSU #224
Conversation
input logic [31:0] lsu_addr_m, // address | ||
input logic [31:0] store_data_m, // store data |
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 [31:0] lsu_addr_m, // address | |
input logic [31:0] store_data_m, // store data | |
input logic [31:0] lsu_addr_m, // address | |
input logic [31:0] store_data_m, // store data |
input logic [31:0] lsu_addr_m, // address | ||
input logic [31:0] store_data_m, // store data | ||
|
||
output logic [3:0] lsu_trigger_match_m // match result |
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 [3:0] lsu_trigger_match_m // match result | |
output logic [3:0] lsu_trigger_match_m // match result |
assign lsu_pkt_m.word = 1; // lsu_word; | ||
assign lsu_pkt_m.half = 1; // lsu_half; | ||
assign lsu_pkt_m.dma = 0; // lsu_dma; | ||
assign lsu_pkt_m.valid = 1; // lsu_valid; | ||
assign lsu_pkt_m.store = 1; // lsu_store; |
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 🐶
assign lsu_pkt_m.word = 1; // lsu_word; | |
assign lsu_pkt_m.half = 1; // lsu_half; | |
assign lsu_pkt_m.dma = 0; // lsu_dma; | |
assign lsu_pkt_m.valid = 1; // lsu_valid; | |
assign lsu_pkt_m.store = 1; // lsu_store; | |
assign lsu_pkt_m.word = 1; // lsu_word; | |
assign lsu_pkt_m.half = 1; // lsu_half; | |
assign lsu_pkt_m.dma = 0; // lsu_dma; | |
assign lsu_pkt_m.valid = 1; // lsu_valid; | |
assign lsu_pkt_m.store = 1; // lsu_store; |
el2_lsu_trigger tu ( | ||
.* | ||
); |
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 🐶
el2_lsu_trigger tu ( | |
.* | |
); | |
el2_lsu_trigger tu (.*); |
27f805e
to
2675bce
Compare
Links to coverage and verification reports for this PR (#224) are available at https://chipsalliance.github.io/Cores-VeeR-EL2/ |
2675bce
to
b090da2
Compare
b090da2
to
55ff09d
Compare
Links to coverage and verification reports for this PR (#224) are available at https://chipsalliance.github.io/Cores-VeeR-EL2/ |
LGTM |
The PR extends verification tests of LSU. A similar test was already used in DEC.