Use RenameClockDomains decorator instead of add_submodule

milkymist/dvisampler/analysis.py

@@ -137,8 +137,9 @@ class FrameExtraction(Module, AutoCSR):
 			vsync_r.eq(self.vsync)
 		]
 
-		fifo = AsyncFIFO(layout_len(frame_layout), 512)
-		self.add_submodule(fifo, {"write": "pix", "read": "sys"})
+		fifo = RenameClockDomains(AsyncFIFO(layout_len(frame_layout), 512),
+			{"write": "pix", "read": "sys"})
+		self.submodules += fifo
 		self.comb += [
 			fifo.we.eq(fifo_stb),
 			fifo.din.eq(fifo_in.raw_bits()),

milkymist/dvisampler/chansync.py

@@ -56,8 +56,8 @@ class ChanSync(Module, AutoCSR):
 
 			###
 		
-			syncbuffer = _SyncBuffer(layout_len(channel_layout), depth)
-			self.add_submodule(syncbuffer, "pix")
+			syncbuffer = RenameClockDomains(_SyncBuffer(layout_len(channel_layout), depth), "pix")
+			self.submodules += syncbuffer
 			self.comb += [
 				syncbuffer.din.eq(data_in.raw_bits()),
 				data_out.raw_bits().eq(syncbuffer.dout)

milkymist/dvisampler/debug.py

@@ -25,8 +25,9 @@ class RawDVISampler(Module, AutoCSR):
 			self.data0_cap.serdesstrobe.eq(self.clocking.serdesstrobe)
 		]
 
-		fifo = AsyncFIFO(10, 256)
-		self.add_submodule(fifo, {"write": "pix", "read": "sys"})
+		fifo = RenameClockDomains(AsyncFIFO(10, 256),
+			{"write": "pix", "read": "sys"})
+		self.submodules += fifo
 		self.comb += [
 			fifo.din.eq(self.data0_cap.d),
 			fifo.we.eq(1)

milkymist/framebuffer/lib.py

@@ -128,8 +128,9 @@ class FIFO(Module):
 		###
 
 		data_width = 2+2*3*bpc_dac
-		fifo = AsyncFIFO(data_width, 512)
-		self.add_submodule(fifo, {"write": "sys", "read": "vga"})
+		fifo = RenameClockDomains(AsyncFIFO(data_width, 512),
+			{"write": "sys", "read": "vga"})
+		self.submodules += fifo
 		fifo_in = self.dac.payload
 		fifo_out = Record(dac_layout)
 		self.comb += [

tb/dvisampler/chansync.py

@@ -7,8 +7,7 @@ class TB(Module):
 	def __init__(self, test_seq_it):
 		self.test_seq_it = test_seq_it
 
-		self.chansync = ChanSync()
-		self.add_submodule(self.chansync, {"pix": "sys"})
+		self.submodules.chansync = RenameClockDomains(ChanSync(), {"pix": "sys"})
 		self.comb += self.chansync.valid_i.eq(1)
 
 	def do_simulation(self, s):