targets/ulx3s, versa_ecp5: use ECP5PLL

litex/boards/targets/ulx3s.py

@@ -7,6 +7,7 @@ from migen.genlib.resetsync import AsyncResetSynchronizer
 
 from litex.boards.platforms import ulx3s
 
+from litex.soc.cores.clock import *
 from litex.soc.integration.soc_sdram import *
 from litex.soc.integration.builder import *
 
@@ -22,11 +23,16 @@ class _CRG(Module):
 
         # # #
 
+        # clk / rst
         clk25 = platform.request("clk25")
         rst = platform.request("rst")
+        platform.add_period_constraint(clk25, 40.0)
 
-        # sys_clk
+        # pll
-        self.comb += self.cd_sys.clk.eq(clk25)
+        self.submodules.pll = pll = ECP5PLL()
+        self.comb += pll.reset.eq(rst)
+        pll.register_clkin(clk25, 25e6)
+        pll.create_clkout(self.cd_sys, 50e6)
         # FIXME: AsyncResetSynchronizer needs FD1S3BX support.
         #self.specials += AsyncResetSynchronizer(self.cd_sys, rst)
         self.comb += self.cd_sys.rst.eq(rst)

litex/boards/targets/versa_ecp5.py

@@ -7,6 +7,7 @@ from migen.genlib.resetsync import AsyncResetSynchronizer
 
 from litex.boards.platforms import versa_ecp5
 
+from litex.soc.cores.clock import *
 from litex.soc.integration.soc_sdram import *
 from litex.soc.integration.builder import *
 
@@ -17,22 +18,21 @@ from litedram.core.controller import ControllerSettings
 
 class _CRG(Module):
     def __init__(self, platform):
-        self.clock_domains.cd_sys_4x = ClockDomain(reset_less=True)
         self.clock_domains.cd_sys = ClockDomain()
         self.clock_domains.cd_sys_ps = ClockDomain()
 
         # # #
 
+        # clk / rst
         clk100 = platform.request("clk100")
         rst_n = platform.request("rst_n")
+        platform.add_period_constraint(clk100, 10.0)
 
-        # sys_4x_clk divider
+        # pll
-        self.comb += self.cd_sys_4x.clk.eq(clk100)
+        self.submodules.pll = pll = ECP5PLL()
-        sys_4x_divider = Signal(1)
+        self.comb += pll.reset.eq(~rst_n)
-        self.sync.sys_4x += sys_4x_divider.eq(sys_4x_divider + 1)
+        pll.register_clkin(clk100, 100e6)
-
+        pll.create_clkout(self.cd_sys, 50e6)
-        # sys_clk
-        self.comb += self.cd_sys.clk.eq(sys_4x_divider[-1])
         # FIXME: AsyncResetSynchronizer needs FD1S3BX support.
         #self.specials += AsyncResetSynchronizer(self.cd_sys, rst)
         self.comb += self.cd_sys.rst.eq(~rst_n)
@@ -40,7 +40,7 @@ class _CRG(Module):
         # sys_clk phase shifted (for sdram)
         sdram_ps_clk = self.cd_sys.clk
         # FIXME: phase shift with luts, needs PLL support.
-        sdram_ps_luts = 2
+        sdram_ps_luts = 5
         for i in range(sdram_ps_luts):
             new_sdram_ps_clk = Signal()
             self.specials += Instance("LUT4",