vendor.xilinx_spartan_3_6: reconsider bitgen defaults.
authorwhitequark <whitequark@whitequark.org>
Sun, 4 Aug 2019 23:23:06 +0000 (23:23 +0000)
committerwhitequark <whitequark@whitequark.org>
Sun, 4 Aug 2019 23:28:09 +0000 (23:28 +0000)
Previously changed in 27063a3b.

I haven't realized the .bin file is the same as the .bit file without
a small header. That means generating it is free and it's just easier
to let programming tools to be able to always rely on its existence.

nmigen/vendor/xilinx_spartan_3_6.py

index 374ff05b697a3db79ab511dea35b713938e9c99e..3b3b9bfde37f433d0c810dd0c66ab1c811283ae9 100644 (file)
@@ -33,7 +33,8 @@ class XilinxSpartan3Or6Platform(TemplatedPlatform):
         * ``ngdbuild_opts``: adds extra options for NGDBuild.
         * ``map_opts``: adds extra options for MAP.
         * ``par_opts``: adds extra options for PAR.
-        * ``bitgen_opts``: adds extra options for BitGen.
+        * ``bitgen_opts``: adds extra and overrides default options for BitGen;
+          default options: ``-g Compress``.
 
     Build products:
         * ``{{name}}.srp``: synthesis report.
@@ -154,8 +155,9 @@ class XilinxSpartan3Or6Platform(TemplatedPlatform):
         """,
         r"""
         {{get_tool("bitgen")}}
-            {{get_override("bitgen_opts")|default(["-g Binary:Yes", "-g Compress"])|options}}
+            {{get_override("bitgen_opts")|default(["-g Compress"])|options}}
             -w
+            -g Binary:Yes
             {{name}}_par.ncd
             {{name}}.bit
         """