From: whitequark <whitequark@whitequark.org>
Date: Tue, 24 Sep 2019 12:22:29 +0000 (+0000)
Subject: vendor.xilinx_spartan_3_6: explain why ASYNC_REG is used. NFC.
X-Git-Tag: v0.1rc1~55
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=b43d2d36e8cc966cb75768e4b51af82d343fe6ee;p=nmigen.git

vendor.xilinx_spartan_3_6: explain why ASYNC_REG is used. NFC.
---

diff --git a/nmigen/vendor/xilinx_spartan_3_6.py b/nmigen/vendor/xilinx_spartan_3_6.py
index eb0aac0..aafb3f8 100644
--- a/nmigen/vendor/xilinx_spartan_3_6.py
+++ b/nmigen/vendor/xilinx_spartan_3_6.py
@@ -411,6 +411,11 @@ class XilinxSpartan3Or6Platform(TemplatedPlatform):
             )
         return m
 
+    # The synchronizer implementations below apply the ASYNC_REG attribute. This attribute
+    # prevents inference of shift registers from synchronizer FFs, and constraints the FFs
+    # to be placed as close as possible, ideally in one CLB. This attribute only affects
+    # the synchronizer FFs themselves.
+
     def get_ff_sync(self, ff_sync):
         if ff_sync._max_input_delay is not None:
             raise NotImplementedError("Platform {!r} does not support constraining input delay "