From: Eddie Hung <eddie@fpgeh.com>
Date: Mon, 10 Jun 2019 21:34:16 +0000 (-0700)
Subject: Revert "shregmap -tech xilinx_dynamic to work -params and -enpol"
X-Git-Tag: working-ls180~1208^2~171
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=7d27e1e4312499b72d253470c97cfbf98e4c9d8e;p=yosys.git

Revert "shregmap -tech xilinx_dynamic to work -params and -enpol"

This reverts commit 45d1bdf83ae6d51628e917b66f1b6043c8a3baee.
---

diff --git a/passes/techmap/shregmap.cc b/passes/techmap/shregmap.cc
index 91a942c39..60b04be6f 100644
--- a/passes/techmap/shregmap.cc
+++ b/passes/techmap/shregmap.cc
@@ -56,7 +56,7 @@ struct ShregmapOptions
 
 struct ShregmapTechGreenpak4 : ShregmapTech
 {
-	virtual bool analyze(vector<int> &taps, const vector<SigBit> &/*qbits*/) override
+	bool analyze(vector<int> &taps, const vector<SigBit> &/*qbits*/)
 	{
 		if (GetSize(taps) > 2 && taps[0] == 0 && taps[2] < 17) {
 			taps.clear();
@@ -71,7 +71,7 @@ struct ShregmapTechGreenpak4 : ShregmapTech
 		return true;
 	}
 
-	virtual bool fixup(Cell *cell, dict<int, SigBit> &taps) override
+	bool fixup(Cell *cell, dict<int, SigBit> &taps)
 	{
 		auto D = cell->getPort("\\D");
 		auto C = cell->getPort("\\C");
@@ -212,24 +212,8 @@ struct ShregmapTechXilinx7Dynamic : ShregmapTech
 		newcell->set_src_attribute(cell->get_src_attribute());
 		newcell->setParam("\\DEPTH", cell->getParam("\\DEPTH"));
 		newcell->setParam("\\INIT", cell->getParam("\\INIT"));
-
-		if (cell->type.in("$__SHREG_DFF_N_", "$__SHREG_DFF_P_",
-					"$__SHREG_DFFE_NN_", "$__SHREG_DFFE_NP_", "$__SHREG_DFFE_PN_", "$__SHREG_DFFE_PP_")) {
-			int param_clkpol = -1;
-			int param_enpol = 2;
-			if (cell->type == "$__SHREG_DFF_N_") param_clkpol = 0;
-			else if (cell->type == "$__SHREG_DFF_P_") param_clkpol = 1;
-			else if (cell->type == "$__SHREG_DFFE_NN_") param_clkpol = 0, param_enpol = 0;
-			else if (cell->type == "$__SHREG_DFFE_NP_") param_clkpol = 0, param_enpol = 1;
-			else if (cell->type == "$__SHREG_DFFE_PN_") param_clkpol = 1, param_enpol = 0;
-			else if (cell->type == "$__SHREG_DFFE_PP_") param_clkpol = 1, param_enpol = 1;
-			else log_abort();
-
-			log_assert(param_clkpol >= 0);
-			cell->setParam("\\CLKPOL", param_clkpol);
-			cell->setParam("\\ENPOL", param_enpol);
-		}
-		else log_abort();
+		newcell->setParam("\\CLKPOL", cell->getParam("\\CLKPOL"));
+		newcell->setParam("\\ENPOL", cell->getParam("\\ENPOL"));
 
 		newcell->setPort("\\C", cell->getPort("\\C"));
 		newcell->setPort("\\D", cell->getPort("\\D"));
@@ -681,12 +665,8 @@ struct ShregmapPass : public Pass {
 				}
 				else if (tech == "xilinx_dynamic") {
 					opts.init = true;
-					opts.ffcells["$_DFF_P_"] = make_pair(IdString("\\D"), IdString("\\Q"));
-					opts.ffcells["$_DFF_N_"] = make_pair(IdString("\\D"), IdString("\\Q"));
-					opts.ffcells["$_DFFE_PP_"] = make_pair(IdString("\\D"), IdString("\\Q"));
-					opts.ffcells["$_DFFE_PN_"] = make_pair(IdString("\\D"), IdString("\\Q"));
-					opts.ffcells["$_DFFE_NP_"] = make_pair(IdString("\\D"), IdString("\\Q"));
-					opts.ffcells["$_DFFE_NN_"] = make_pair(IdString("\\D"), IdString("\\Q"));
+					opts.params = true;
+					enpol = "any_or_none";
 					opts.tech = new ShregmapTechXilinx7Dynamic(opts);
 				} else {
 					argidx--;