diff options
| author | YRabbit <rabbit@yrabbit.cyou> | 2022-12-09 12:55:22 +1000 |
|---|---|---|
| committer | YRabbit <rabbit@yrabbit.cyou> | 2022-12-09 12:55:22 +1000 |
| commit | aa8359c73e47ca3fed620dbab9ef7ce8d4a6732c (patch) | |
| tree | aeb5acbc8879a34f72486fc62bb4777c644e0555 /gowin/arch.cc | |
| parent | 0eb53d59d84ea1c46de7d9e25eb5a9901544202d (diff) | |
| download | nextpnr-aa8359c73e47ca3fed620dbab9ef7ce8d4a6732c.tar.gz nextpnr-aa8359c73e47ca3fed620dbab9ef7ce8d4a6732c.tar.bz2 nextpnr-aa8359c73e47ca3fed620dbab9ef7ce8d4a6732c.zip | |
gowin: BUGFIX: Correctly handle resets
When a single primitive occupies several cells, care must be taken when
manipulating the parameters of that primitive: when creating cells, each
cell must receive a copy of all the parameters and not modify them
unnecessarily. That is, if possible, it is better to make all parameter
changes before dividing the primitive into cells.
Signed-off-by: YRabbit <rabbit@yrabbit.cyou>
Diffstat (limited to 'gowin/arch.cc')
| -rw-r--r-- | gowin/arch.cc | 13 |
1 files changed, 0 insertions, 13 deletions
diff --git a/gowin/arch.cc b/gowin/arch.cc index 04cf7af5..461b64c8 100644 --- a/gowin/arch.cc +++ b/gowin/arch.cc @@ -2051,19 +2051,6 @@ void Arch::fix_pll_nets(Context *ctx) // This is general routing through CLK2 pip ci->setParam(id_FBSEL, Property("CLKFB0")); } while (0); - - // resets - Property pr_enable("ENABLE"), pr_disable("DISABLE"); - NetInfo *net = ci->getPort(id_RESET); - ci->setParam(id_RSTEN, pr_enable); - if (!port_used(ci, id_RESET) || net->name == id("$PACKER_VCC_NET") || net->name == id("$PACKER_GND_NET")) { - ci->setParam(id_RSTEN, pr_disable); - } - ci->setParam(id_PWDEN, pr_enable); - net = ci->getPort(id_RESET_P); - if (!port_used(ci, id_RESET_P) || net->name == id("$PACKER_VCC_NET") || net->name == id("$PACKER_GND_NET")) { - ci->setParam(id_PWDEN, pr_disable); - } } } |