You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Retrieving address in the ar channel and transferring data in the r channel can be performed simultaneously. It's recommended to use pipeline instead of state machine, which will improve bandwidth.
Issues to be Fixed
Handling of the
WARPtype in burst of AXI4 protocol is missing.https://github.com/oscc-ip/sram/blob/c73ed0c37e9a87ab0e62f007589dc583b5697c04/rtl/axi4_sram_fsm.sv#L172-L173
https://github.com/oscc-ip/sram/blob/c73ed0c37e9a87ab0e62f007589dc583b5697c04/rtl/axi4_sram_fsm.sv#L195-L196
sram.bm_iandaxi4.wstrbhave the same polarity, but here it's inverted.https://github.com/oscc-ip/sram/blob/c73ed0c37e9a87ab0e62f007589dc583b5697c04/rtl/mem_ctrl.sv#L57
Recommendations for Improvement
Using
#1for delay is unnecessary as it doesn't take effect during synthesis.https://github.com/oscc-ip/sram/blob/c73ed0c37e9a87ab0e62f007589dc583b5697c04/rtl/axi4_sram_fsm.sv#L227-L230
Retrieving address in the
archannel and transferring data in therchannel can be performed simultaneously. It's recommended to use pipeline instead of state machine, which will improve bandwidth.