diff options
author | Sylvain Munaut <tnt@246tNt.com> | 2019-04-11 13:50:36 +0200 |
---|---|---|
committer | Sylvain Munaut <tnt@246tNt.com> | 2019-04-11 13:52:23 +0200 |
commit | 6cb4e2e83bcf6173e646df10a4a14e32b8128ccb (patch) | |
tree | 4a384f2521e868d2e1f08af70f8fa88ac057e77c | |
parent | 22da6c8a65d43f91db254851da6135f0e9e801a0 (diff) | |
download | nextpnr-6cb4e2e83bcf6173e646df10a4a14e32b8128ccb.tar.gz nextpnr-6cb4e2e83bcf6173e646df10a4a14e32b8128ccb.tar.bz2 nextpnr-6cb4e2e83bcf6173e646df10a4a14e32b8128ccb.zip |
ice40/pack: During IO packing, remove any unused input connection
This is mostly for the benefit of PLL placement because the D_IN_x
ports are used for other purposes when PLL is enabled so we need to
make sure nothing is connected there already. (even an unused net is
too much)
Signed-off-by: Sylvain Munaut <tnt@246tNt.com>
-rw-r--r-- | ice40/pack.cc | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/ice40/pack.cc b/ice40/pack.cc index b1efe5b2..98cacad3 100644 --- a/ice40/pack.cc +++ b/ice40/pack.cc @@ -502,6 +502,19 @@ static void pack_io(Context *ctx) // Make it a normal SB_IO with global marker ci->type = ctx->id("SB_IO"); ci->attrs[ctx->id("GLOBAL")] = "1"; + } else if (is_sb_io(ctx, ci)) { + // Disconnect unused inputs + NetInfo *net_in0 = ci->ports.count(id_D_IN_0) ? ci->ports[id_D_IN_0].net : nullptr; + NetInfo *net_in1 = ci->ports.count(id_D_IN_1) ? ci->ports[id_D_IN_1].net : nullptr; + + if (net_in0 != nullptr && net_in0->users.size() == 0) { + delete_nets.insert(net_in0->name); + ci->ports[id_D_IN_0].net = nullptr; + } + if (net_in1 != nullptr && net_in1->users.size() == 0) { + delete_nets.insert(net_in1->name); + ci->ports[id_D_IN_1].net = nullptr; + } } } for (auto pcell : packed_cells) { |