From 3d7214cb701123f03146b3f5e1fe93b8a820f9e6 Mon Sep 17 00:00:00 2001 From: whitequark Date: Sun, 4 Aug 2019 23:23:06 +0000 Subject: [PATCH] vendor.xilinx_spartan_3_6: reconsider bitgen defaults. 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 | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/nmigen/vendor/xilinx_spartan_3_6.py b/nmigen/vendor/xilinx_spartan_3_6.py index 374ff05..3b3b9bf 100644 --- a/nmigen/vendor/xilinx_spartan_3_6.py +++ b/nmigen/vendor/xilinx_spartan_3_6.py @@ -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 """ -- 2.30.2