From: Florent Kermarrec <florent@enjoy-digital.fr>
Date: Thu, 27 Jul 2017 16:22:01 +0000 (+0200)
Subject: soc/cores: fix vivado issue with SPIRegister (at least with Vivado 2017.x+, mosi... 
X-Git-Tag: 24jan2021_ls180~1810
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=c02de1632b9cbe269aeb76ef248f2a1869bcb7f9;p=litex.git

soc/cores: fix vivado issue with SPIRegister (at least with Vivado 2017.x+, mosi was not generated correctly), create cs_n signal if pads does not exists
---

diff --git a/litex/soc/cores/spi.py b/litex/soc/cores/spi.py
index d5f92789..bda81f04 100644
--- a/litex/soc/cores/spi.py
+++ b/litex/soc/cores/spi.py
@@ -45,18 +45,19 @@ class SPIRegister(Module):
             self.o.eq(Mux(self.lsb, self.data[0], self.data[-1])),
         ]
         self.sync += [
-            If(self.shift,
-                If(self.lsb,
-                    self.data[:-1].eq(self.data[1:]),
-                ).Else(
-                    self.data[1:].eq(self.data[:-1]),
+            If(self.lsb,
+                If(self.shift,
+                    self.data[:-1].eq(self.data[1:])
+                ),
+                If(self.sample,
+                    self.data[0].eq(self.i)
                 )
-            ),
-            If(self.sample,
-                If(self.lsb,
-                    self.data[-1].eq(self.i),
-                ).Else(
-                    self.data[0].eq(self.i),
+            ).Else(
+                If(self.shift,
+                    self.data[1:].eq(self.data[:-1]),
+                ),
+                If(self.sample,
+                    self.data[0].eq(self.i)
                 )
             )
         ]
@@ -317,14 +318,17 @@ class SPIMasterCore(Module):
         ]
 
         # I/O
-        if hasattr(pads, "cs_n"):
-            cs_n_t = TSTriple(len(pads.cs_n))
-            self.specials += cs_n_t.get_tristate(pads.cs_n)
-            self.comb += [
-                cs_n_t.oe.eq(~self.config.offline),
-                cs_n_t.o.eq((cs & Replicate(machine.cs, len(cs))) ^
-                            Replicate(~self.config.cs_polarity, len(cs))),
-            ]
+        if not hasattr(pads, "cs_n"):
+            self.cs_n = Signal()
+        else:
+            self.cs_n = pads.cs_n
+        cs_n_t = TSTriple(len(self.cs_n))
+        self.specials += cs_n_t.get_tristate(self.cs_n)
+        self.comb += [
+            cs_n_t.oe.eq(~self.config.offline),
+            cs_n_t.o.eq((cs & Replicate(machine.cs, len(cs))) ^
+                        Replicate(~self.config.cs_polarity, len(cs))),
+        ]
 
         clk_t = TSTriple()
         self.specials += clk_t.get_tristate(pads.clk)
@@ -342,6 +346,7 @@ class SPIMasterCore(Module):
             machine.reg.i.eq(Mux(self.config.half_duplex, mosi_t.i,
                              getattr(pads, "miso", mosi_t.i))),
         ]
+        self.mosi_t = mosi_t
 
 
 class SPIMaster(Module, AutoCSR):