vendor.xilinx_spartan_3_6: set bitgen defaults to `-g Binary:Yes -g Compress`.
authorwhitequark <whitequark@whitequark.org>
Sun, 4 Aug 2019 14:16:02 +0000 (14:16 +0000)
committerwhitequark <whitequark@whitequark.org>
Sun, 4 Aug 2019 14:18:49 +0000 (14:18 +0000)
  * `-g Binary:Yes` should be overridable.
  * `-g Compress` is a good default.

nmigen/vendor/lattice_ice40.py
nmigen/vendor/xilinx_spartan_3_6.py

index 3ab0747b1bd7cac65038a1a251cc40cde541821e..16a981141df1afe46423188fe71109a73794c2e7 100644 (file)
@@ -24,8 +24,8 @@ class LatticeICE40Platform(TemplatedPlatform):
         * ``script_after_read``: inserts commands after ``read_ilang`` in Yosys script.
         * ``script_after_synth``: inserts commands after ``synth_ice40`` in Yosys script.
         * ``yosys_opts``: adds extra options for Yosys.
-        * ``nextpnr_opts``: adds extra and overrides default options (``--placer heap``)
-          for nextpnr.
+        * ``nextpnr_opts``: adds extra and overrides default options for nextpnr;
+          default options: ``--placer heap``.
 
     Build products:
         * ``{{name}}.rpt``: Yosys log.
index 4a44ffdff64d1d503f4dd960bd217128947eb057..374ff05b697a3db79ab511dea35b713938e9c99e 100644 (file)
@@ -154,9 +154,8 @@ class XilinxSpartan3Or6Platform(TemplatedPlatform):
         """,
         r"""
         {{get_tool("bitgen")}}
-            {{get_override("bitgen_opts")|default([])|options}}
+            {{get_override("bitgen_opts")|default(["-g Binary:Yes", "-g Compress"])|options}}
             -w
-            -g Binary:Yes
             {{name}}_par.ncd
             {{name}}.bit
         """