From 43b1ed1bf62c08826c82bb80ec08651a1abc9e65 Mon Sep 17 00:00:00 2001 From: whitequark Date: Thu, 21 May 2020 09:49:42 +0000 Subject: [PATCH] vendor.intel: don't use `write_verilog -decimal`. See commit 702e41ba for details. --- nmigen/vendor/intel.py | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/nmigen/vendor/intel.py b/nmigen/vendor/intel.py index f99827e..8621661 100644 --- a/nmigen/vendor/intel.py +++ b/nmigen/vendor/intel.py @@ -68,15 +68,13 @@ class IntelPlatform(TemplatedPlatform): set -e{{verbose("x")}} {{emit_commands("sh")}} """, - # Quartus doesn't like constructs like (* keep = 32'd1 *), even though they mean the same - # thing as (* keep = 1 *); use -decimal to work around that. "{{name}}.v": r""" /* {{autogenerated}} */ - {{emit_verilog(["-decimal"])}} + {{emit_verilog()}} """, "{{name}}.debug.v": r""" /* {{autogenerated}} */ - {{emit_debug_verilog(["-decimal"])}} + {{emit_debug_verilog()}} """, "{{name}}.qsf": r""" # {{autogenerated}} @@ -146,7 +144,7 @@ class IntelPlatform(TemplatedPlatform): def add_clock_constraint(self, clock, frequency): super().add_clock_constraint(clock, frequency) - clock.attrs["keep"] = 1 + clock.attrs["keep"] = "true" # The altiobuf_* and altddio_* primitives are explained in the following Intel documents: # * https://www.intel.com/content/dam/www/programmable/us/en/pdfs/literature/ug/ug_altiobuf.pdf -- 2.30.2