M-Labs/artiq-zynq
10
1
Fork 19
Code Issues 12 Pull Requests Releases Wiki Activity

sram: fixed wrong assumptions on some signals

Browse Source
This commit is contained in:
mwojcik 2021-08-13 14:58:18 +02:00
parent bfe0c34f57
commit b6dd5bea68
1 changed files with 6 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/gateware/aux_controller.py
View File

@ -37,8 +37,6 @@ class SRAM(Module):
# probably will get removed
self.addr_base = CSRStorage(32)
self.trigger_stb = Signal()
# Dout : Data received from CPU, output by SRAM <- port.dat_r
# Din : Data driven into SRAM, written into CPU <- port.dat_w
# When stb assert, index shows word being read/written, dout/din holds <- will be removed
@ -78,14 +76,15 @@ class SRAM(Module):
)
)
read_fsm.act("READ_START",
If(r.valid,
r.ready.eq(1),
r.data.eq(port.dat_r),
r.valid.eq(1),
If(r.ready,
r.data.eq(port.dat_r), # that should be always updated, right?
NextState("READ"))
)
read_fsm.act("READ",
If(r.last & r.valid, # that's a smart way of skipping "LAST" state
r.data.eq(port.dat_r),
If(r.last & r.ready, # that's a smart way of skipping "LAST" state
NextState("IDLE")
)
)
@ -95,7 +94,7 @@ class SRAM(Module):
self.dout_index.eq(0),
r.ready.eq(0), # shall it be reset too on IDLE?
ar.ready.eq(0)
).Else(If(r.valid & read_fsm.ongoing("READ"),
).Else(If(r.ready & read_fsm.ongoing("READ"),
self.dout_index.eq(self.dout_index+1),
port.adr.eq(port.adr + self.dout_index), # update address in the port
If(self.dout_index==ar.len, r.last.eq(1)) # and update last