From: Pepijn de Vos <pepijndevos@gmail.com>
Date: Tue, 19 Nov 2019 14:53:44 +0000 (+0100)
Subject: Remove dff init altogether
X-Git-Tag: working-ls180~956^2
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=8ab412eb16b1d4f98117247bf85e0c37627ee459;p=yosys.git

Remove dff init altogether

The hardware does not actually support it.
In reality it is always initialised to its reset value.
---

diff --git a/techlibs/gowin/cells_map.v b/techlibs/gowin/cells_map.v
index 881c2e9bb..9845e56a7 100644
--- a/techlibs/gowin/cells_map.v
+++ b/techlibs/gowin/cells_map.v
@@ -1,9 +1,10 @@
-//TODO all DFF* have INIT
+//All DFF* have INIT, but the hardware is always initialised to the reset
+//value regardless. The parameter is ignored.
 
 // DFFN      D Flip-Flop with Negative-Edge Clock
 module  \$_DFF_N_ (input D, C, output Q); DFFN _TECHMAP_REPLACE_ (.D(D), .Q(Q), .CLK(C)); endmodule
 // DFF       D Flip-Flop
-module  \$_DFF_P_ #(parameter INIT = 1'b0) (input D, C, output Q); DFF  #(.INIT(INIT)) _TECHMAP_REPLACE_ (.D(D), .Q(Q), .CLK(C)); endmodule
+module  \$_DFF_P_ (input D, C, output Q); DFF _TECHMAP_REPLACE_ (.D(D), .Q(Q), .CLK(C)); endmodule
 
 // DFFE      D Flip-Flop with Clock Enable
 module  \$_DFFE_PP_ (input D, C, E, output Q); DFFE _TECHMAP_REPLACE_ (.D(D), .Q(Q), .CLK(C), .CE(E)); endmodule
diff --git a/techlibs/gowin/synth_gowin.cc b/techlibs/gowin/synth_gowin.cc
index bb6d44e00..3c1426414 100644
--- a/techlibs/gowin/synth_gowin.cc
+++ b/techlibs/gowin/synth_gowin.cc
@@ -238,7 +238,6 @@ struct SynthGowinPass : public ScriptPass
 			run("hilomap -singleton -hicell VCC V -locell GND G");
 			run("iopadmap -bits -inpad IBUF O:I -outpad OBUF I:O "
 				"-toutpad TBUF OEN:I:O -tinoutpad IOBUF OEN:O:I:IO", "(unless -noiopads)");
-			run("dffinit  -ff DFF Q INIT");
 			run("clean");
 
 		}