[Cryptech-Commits] [user/shatov/modexpng] 02/14: * more consistent port names * optional two-stage pipeline for A&B ports

[git at cryptech.is]

This is an automated email from the git hooks/post-receive script.

meisterpaul1 at yandex.ru pushed a commit to branch master
in repository user/shatov/modexpng.

commit 6b335ce4fcf6934caf124dd55365973dc7f6e2e1
Author: Pavel V. Shatov (Meister) <meisterpaul1 at yandex.ru>
AuthorDate: Thu Jan 30 19:46:23 2020 +0300

     * more consistent port names
     * optional two-stage pipeline for A&B ports
---
 rtl/modexpng_dsp_slice_addsub_wrapper_generic.v | 38 ++++++++++++++---------
 rtl/modexpng_dsp_slice_addsub_wrapper_xilinx.v  | 41 +++++++++++++++----------
 2 files changed, 48 insertions(+), 31 deletions(-)

diff --git a/rtl/modexpng_dsp_slice_addsub_wrapper_generic.v b/rtl/modexpng_dsp_slice_addsub_wrapper_generic.v
index 0ef45fe..d5b294c 100644
--- a/rtl/modexpng_dsp_slice_addsub_wrapper_generic.v
+++ b/rtl/modexpng_dsp_slice_addsub_wrapper_generic.v
@@ -30,13 +30,18 @@
 //
 //======================================================================
 
-module modexpng_dsp_slice_addsub_wrapper_generic
+module modexpng_dsp_slice_addsub_wrapper_generic #
+(
+    AB_REG = 1
+)
 (
     clk,
-    ce_abc,
+    ce_ab1,
+    ce_ab2,
+    ce_c,
     ce_p,
     ce_ctrl,
-    x, y, p,
+    ab, c, p,
     op_mode,
     alu_mode,
     carry_in_sel,
@@ -48,11 +53,13 @@ module modexpng_dsp_slice_addsub_wrapper_generic
     `include "modexpng_dsp48e1.vh"
 
     input                              clk;
-    input                              ce_abc;
+    input                              ce_ab1;
+    input                              ce_ab2;
+    input                              ce_c;
     input                              ce_p;
     input                              ce_ctrl;
-    input  [         DSP48E1_C_W -1:0] x;
-    input  [         DSP48E1_C_W -1:0] y;
+    input  [         DSP48E1_C_W -1:0] ab;
+    input  [         DSP48E1_C_W -1:0] c;
     output [         DSP48E1_P_W -1:0] p;
     input  [    DSP48E1_OPMODE_W -1:0] op_mode;
     input  [   DSP48E1_ALUMODE_W -1:0] alu_mode;
@@ -65,19 +72,22 @@ module modexpng_dsp_slice_addsub_wrapper_generic
     //
     // Internal Registers
     //
-    reg [         DSP48E1_C_W -1:0] reg_x;
-    reg [         DSP48E1_C_W -1:0] reg_y;
+    reg [         DSP48E1_C_W -1:0] reg_ab1;
+    reg [         DSP48E1_C_W -1:0] reg_ab2;
+    reg [         DSP48E1_C_W -1:0] reg_c;
 
     reg [    DSP48E1_OPMODE_W -1:0] reg_op_mode;
     reg [   DSP48E1_ALUMODE_W -1:0] reg_alu_mode;
     reg [DSP48E1_CARRYINSEL_W -1:0] reg_carry_in_sel;
 
-    always @(posedge clk)
+    always @(posedge clk) begin
         //
-        if (ce_abc) begin
-            reg_x <= x;
-            reg_y <= y;
-        end
+        if (ce_ab1) reg_ab1 <= ab;
+        if (ce_ab2) reg_ab2 <= AB_REG == 2 ? reg_ab1 : ab;
+        if (ce_c) reg_c <= c;
+        //
+    end
+
         
     always @(posedge clk)
         //
@@ -205,7 +215,7 @@ module modexpng_dsp_slice_addsub_wrapper_generic
     always @(posedge clk)
         //
         if (ce_p) {reg_carry_out, reg_p} <=
-            calc_p(reg_alu_mode, reg_op_mode, reg_carry_in_sel, reg_p, reg_x, reg_y, reg_carry_out, casc_p_in);
+            calc_p(reg_alu_mode, reg_op_mode, reg_carry_in_sel, reg_p, reg_ab2, reg_c, reg_carry_out, casc_p_in);
 
 
     //
diff --git a/rtl/modexpng_dsp_slice_addsub_wrapper_xilinx.v b/rtl/modexpng_dsp_slice_addsub_wrapper_xilinx.v
index fee8216..b8dcbce 100644
--- a/rtl/modexpng_dsp_slice_addsub_wrapper_xilinx.v
+++ b/rtl/modexpng_dsp_slice_addsub_wrapper_xilinx.v
@@ -30,13 +30,18 @@
 //
 //======================================================================
 
-module modexpng_dsp_slice_addsub_wrapper_xilinx
+module modexpng_dsp_slice_addsub_wrapper_xilinx #
+(
+    AB_REG = 1
+)
 (
     clk,
-    ce_abc,
+    ce_ab1,
+    ce_ab2,
+    ce_c,
     ce_p,
     ce_ctrl,
-    x, y, p,
+    ab, c, p,
     op_mode,
     alu_mode,
     carry_in_sel,
@@ -48,11 +53,13 @@ module modexpng_dsp_slice_addsub_wrapper_xilinx
     `include "modexpng_dsp48e1.vh"
         
     input                              clk;
-    input                              ce_abc;
+    input                              ce_ab1;
+    input                              ce_ab2;
+    input                              ce_c;
     input                              ce_p;
     input                              ce_ctrl;
-    input  [         DSP48E1_C_W -1:0] x;
-    input  [         DSP48E1_C_W -1:0] y;
+    input  [         DSP48E1_C_W -1:0] ab;
+    input  [         DSP48E1_C_W -1:0] c;
     output [         DSP48E1_P_W -1:0] p;
     input  [    DSP48E1_OPMODE_W -1:0] op_mode;
     input  [   DSP48E1_ALUMODE_W -1:0] alu_mode;
@@ -67,22 +74,22 @@ module modexpng_dsp_slice_addsub_wrapper_xilinx
     wire [       DSP48E1_P_W -1:0] p_int;
     wire [DSP48E1_CARRYOUT_W -1:0] carry_out_int;
 
-    assign {a_int, b_int} = {x};
-    assign {c_int}        = {y};
+    assign {a_int, b_int} = {ab};
+    assign {c_int}        = {c};
     assign {p}            = {p_int};
     assign {carry_out}    = {carry_out_int[DSP48E1_CARRYOUT_W-1]};
 
     DSP48E1 #
     (
-        .AREG                   (1),
-        .BREG                   (1),
+        .AREG                   (AB_REG),
+        .BREG                   (AB_REG),
         .CREG                   (1),
         .DREG                   (0),
         .ADREG                  (0),
         .MREG                   (0),
         .PREG                   (1),
-        .ACASCREG               (1),
-        .BCASCREG               (1),
+        .ACASCREG               (AB_REG),
+        .BCASCREG               (AB_REG),
         .INMODEREG              (0),
         .OPMODEREG              (1),
         .ALUMODEREG             (1),
@@ -108,12 +115,12 @@ module modexpng_dsp_slice_addsub_wrapper_xilinx
     (
         .CLK                (clk),
     
-        .CEA1               (1'b0),
-        .CEB1               (1'b0),
-        .CEA2               (ce_abc),
-        .CEB2               (ce_abc),
+        .CEA1               (ce_ab1),
+        .CEB1               (ce_ab1),
+        .CEA2               (ce_ab2),
+        .CEB2               (ce_ab2),
         .CEAD               (1'b0),
-        .CEC                (ce_abc),
+        .CEC                (ce_c),
         .CED                (1'b0),
         .CEM                (1'b0),
         .CEP                (ce_p),