From: Luke Kenneth Casson Leighton <lkcl@lkcl.net>
Date: Sat, 5 Sep 2020 17:15:36 +0000 (+0100)
Subject: MSR read in INSN_READ only occurs for 1 cycle
X-Git-Tag: semi_working_ecp5~182
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=5a2c3f5b9f18b9958f5b19230d52e39461d41a3e;p=soc.git

MSR read in INSN_READ only occurs for 1 cycle
---

diff --git a/src/soc/simple/issuer.py b/src/soc/simple/issuer.py
index 5c3bb51a..d2b573bb 100644
--- a/src/soc/simple/issuer.py
+++ b/src/soc/simple/issuer.py
@@ -187,6 +187,7 @@ class TestIssuer(Elaboratable):
 
         # don't read msr every cycle
         comb += self.state_r_msr.ren.eq(0)
+        msr_read = Signal(reset=1)
 
         # connect up debug signals
         # TODO comb += core.icache_rst_i.eq(dbg.icache_rst_o)
@@ -227,6 +228,7 @@ class TestIssuer(Elaboratable):
 
                     # initiate read of MSR
                     comb += self.state_r_msr.ren.eq(1<<StateRegs.MSR)
+                    sync += msr_read.eq(0)
 
                     m.next = "INSN_READ" # move to "wait for bus" phase
                 with m.Else():
@@ -236,7 +238,9 @@ class TestIssuer(Elaboratable):
             # dummy pause to find out why simulation is not keeping up
             with m.State("INSN_READ"):
                 # one cycle later, msr read arrives
-                sync += cur_state.msr.eq(self.state_r_msr.data_o)
+                with m.If(~msr_read):
+                    sync += msr_read.eq(1)
+                    sync += cur_state.msr.eq(self.state_r_msr.data_o)
                 with m.If(self.imem.f_busy_o): # zzz...
                     # busy: stay in wait-read
                     comb += self.imem.a_valid_i.eq(1)
@@ -267,7 +271,7 @@ class TestIssuer(Elaboratable):
             with m.State("INSN_ACTIVE"):
                 with m.If(insn_type != MicrOp.OP_NOP):
                     comb += core_ivalid_i.eq(1) # instruction is valid
-                with m.If(self.state_nia.wen):
+                with m.If(self.state_nia.wen & (1<<StateRegs.PC)):
                     sync += pc_changed.eq(1)
                 with m.If(~core_busy_o): # instruction done!
                     # ok here we are not reading the branch unit.  TODO