Skip to content

Commit

Permalink
chenge the AxiMaxReads in axi_dw_converter to 8, to meet the vivado i…
Browse files Browse the repository at this point in the history
…mplementation timing constraint
  • Loading branch information
Aquaticfuller committed Dec 29, 2023
1 parent 8ea9652 commit ce42f85
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion hw/c910_pkg.sv
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ package c910_pkg;
// localparam AxiIdWidthSlaves = AxiIdWidthMaster + $clog2(NrMasters); // 5
// localparam AxiUserWidth = 1;

// // 2n(Non-cacheable/Device) + 28(cacheable) read + 2n(Non-cacheable/Device) + 32(cacheable) write
// // 8n(Non-cacheable/Device) + 28(cacheable) read + 8n(Non-cacheable/Device) + 32(cacheable) write
// localparam AxiMaxMstTrans = 8 * NrMasters + 28 + 8 * NrMasters + 32;

endpackage
5 changes: 3 additions & 2 deletions hw/cheshire_soc.sv
Original file line number Diff line number Diff line change
Expand Up @@ -677,7 +677,8 @@ module cheshire_soc
// plic
.plic_hartx_mint_req_i ({1'b0, xeip[i].m}),
.plic_hartx_sint_req_i ({1'b0, xeip[i].s}),

// debug request (async)
.debug_req_i ( '0 ),
// External interrupts to c910 internal plic
.ext_int_i ( '0 /*{39'b0, intr.ext}*/ ),
// JTAG
Expand Down Expand Up @@ -791,7 +792,7 @@ module cheshire_soc


axi_dw_converter #(
.AxiMaxReads ( Cfg.AxiMaxMstTrans ), // Number of outstanding reads
.AxiMaxReads ( 8 ),// soc910_pkg::AxiMaxMstReadTrans is too large // Number of outstanding reads
.AxiSlvPortDataWidth ( soc910_pkg::AxiDataWidth ), // Data width of the slv port
.AxiMstPortDataWidth ( Cfg.AxiDataWidth ), // Data width of the mst port
.AxiAddrWidth ( Cfg.AddrWidth ), // Address width
Expand Down

0 comments on commit ce42f85

Please sign in to comment.