From 700f76c599b2d7ef4a6c986597d79ba245208b9c Mon Sep 17 00:00:00 2001
From: Florent Kermarrec <florent@enjoy-digital.fr>
Date: Wed, 5 Sep 2018 08:50:28 +0200
Subject: [PATCH] frontend/axi: add resp signals

---
 litedram/frontend/axi.py | 14 +++++++++++++-
 litedram/frontend/dma.py |  2 +-
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/litedram/frontend/axi.py b/litedram/frontend/axi.py
index 29190b4..2b452ca 100644
--- a/litedram/frontend/axi.py
+++ b/litedram/frontend/axi.py
@@ -12,6 +12,7 @@ Features:
 Limitations:
 - Write response always supposed to be ready.
 - Last signals not used.
+- Response always okay.
 - No reordering.
 """
 
@@ -28,6 +29,13 @@ burst_types = {
     "reserved": 0b11
 }
 
+resp_types = {
+    "okay":   0b00,
+    "exokay": 0b01,
+    "slverr": 0b10,
+    "decerr": 0b11
+}
+
 def ax_description(address_width, id_width):
     return [
         ("addr",  address_width),
@@ -45,11 +53,13 @@ def w_description(data_width):
 
 def b_description(id_width):
     return [
+        ("resp", 2),
         ("id", id_width)
     ]
 
 def r_description(data_width, id_width):
     return [
+        ("resp", 2),
         ("data", data_width),
         ("id", id_width)
     ]
@@ -158,6 +168,7 @@ class LiteDRAMAXI2NativeW(Module):
             id_buffer.sink.valid.eq(aw.valid & aw.ready),
             id_buffer.sink.id.eq(aw.id),
             axi.b.valid.eq(axi.w.valid & axi.w.ready), # Note: Write response always supposed to be ready.
+            axi.b.resp.eq(resp_types["okay"]),
             axi.b.id.eq(id_buffer.source.id),
             id_buffer.source.ready.eq(axi.b.valid & axi.b.ready)
         ]
@@ -246,7 +257,8 @@ class LiteDRAMAXI2NativeR(Module):
         # Read data
         self.comb += [
             port.rdata.connect(r_buffer.sink),
-            r_buffer.source.connect(axi.r, omit={"id"})
+            r_buffer.source.connect(axi.r, omit={"id"}),
+            axi.r.resp.eq(resp_types["okay"])
         ]
 
 
diff --git a/litedram/frontend/dma.py b/litedram/frontend/dma.py
index b5c7145..bba640d 100644
--- a/litedram/frontend/dma.py
+++ b/litedram/frontend/dma.py
@@ -83,7 +83,7 @@ class LiteDRAMDMAReader(Module):
         self.submodules += fifo
 
         self.comb += [
-            rdata.connect(fifo.sink, omit={"bank", "id"}),
+            rdata.connect(fifo.sink, omit={"bank", "id", "resp"}),
             fifo.source.connect(source),
             data_dequeued.eq(source.valid & source.ready)
         ]