From 9d2d38ee20c7a7769866473dc31758c88e9245d1 Mon Sep 17 00:00:00 2001 From: whitequark Date: Tue, 24 Sep 2019 12:22:29 +0000 Subject: [PATCH] vendor.xilinx_spartan_3_6: explain why ASYNC_REG is used. NFC. --- nmigen/vendor/xilinx_spartan_3_6.py | 5 +++++ 1 file changed, 5 insertions(+) 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 " -- 2.30.2