From f0b5c6721627947025527d9ed9c9304cc291c530 Mon Sep 17 00:00:00 2001
From: Tim 'mithro' Ansell <me@mith.ro>
Date: Mon, 30 Dec 2019 19:25:14 +0100
Subject: [PATCH] Allow specifying the same clock constraint multiple times.

(As long as the clock values actually match.)
---
 litex/build/lattice/icestorm.py | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/litex/build/lattice/icestorm.py b/litex/build/lattice/icestorm.py
index b999e483b..c78574086 100644
--- a/litex/build/lattice/icestorm.py
+++ b/litex/build/lattice/icestorm.py
@@ -193,7 +193,12 @@ class LatticeIceStormToolchain:
         return v_output.ns
 
     def add_period_constraint(self, platform, clk, period):
+        clk_ns = 1e3/period
         clk.attr.add("keep")
         if clk in self.clocks:
-            raise ValueError("A period constraint already exists")
-        self.clocks[clk] = 1e3/period
+            if clk_ns != self.clocks[clk]:
+                raise ValueError(
+                    "A period constraint already exists"
+                    "(wanted: {:.2f}ns, got {:.2f}ns)".format(
+                        clk_ns, self.clocks[clk]))
+        self.clocks[clk] = clk_ns