From 5889e626680f9e7f95ba8968d5247b92380720a4 Mon Sep 17 00:00:00 2001 From: whitequark Date: Wed, 21 Aug 2019 21:32:12 +0000 Subject: [PATCH] vendor.lattice_ice40: remove `--placer heap` default option. It is not the place of nMigen to decide on this default, since both SA and HeAP have valid uses that are not covered by the other. --- nmigen/vendor/lattice_ice40.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/nmigen/vendor/lattice_ice40.py b/nmigen/vendor/lattice_ice40.py index 45aaac4..5c1880b 100644 --- a/nmigen/vendor/lattice_ice40.py +++ b/nmigen/vendor/lattice_ice40.py @@ -24,8 +24,7 @@ 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 for ``nextpnr-ice40``; - default options: ``--placer heap``. + * ``nextpnr_opts``: adds extra options for ``nextpnr-ice40``. Build products: * ``{{name}}.rpt``: Yosys log. @@ -107,7 +106,7 @@ class LatticeICE40Platform(TemplatedPlatform): r""" {{get_tool("nextpnr-ice40")}} {{quiet("--quiet")}} - {{get_override("nextpnr_opts")|default(["--placer","heap"])|options}} + {{get_override("nextpnr_opts")|options}} --log {{name}}.tim {{platform._nextpnr_device_options[platform.device]}} --package -- 2.30.2