From 6eff2f003068320bb0243fb38864d28293be589e Mon Sep 17 00:00:00 2001
From: Alex Forencich <alex@alexforencich.com>
Date: Sun, 9 Jun 2019 22:03:24 -0700
Subject: [PATCH] Decouple transmit PTP tag enable and transmit PTP timestamp
 enable

---
 rtl/axis_baser_tx_64.v      | 8 ++++----
 rtl/axis_gmii_tx.v          | 8 ++++----
 rtl/axis_xgmii_tx_32.v      | 8 ++++----
 rtl/axis_xgmii_tx_64.v      | 8 ++++----
 rtl/eth_mac_10g.v           | 4 ++--
 rtl/eth_mac_1g.v            | 4 ++--
 rtl/eth_mac_phy_10g.v       | 4 ++--
 rtl/eth_mac_phy_10g_tx.v    | 4 ++--
 tb/test_axis_baser_tx_64.py | 4 ++--
 tb/test_axis_baser_tx_64.v  | 4 ++--
 tb/test_axis_gmii_tx.py     | 4 ++--
 tb/test_axis_gmii_tx.v      | 4 ++--
 tb/test_axis_xgmii_tx_32.py | 4 ++--
 tb/test_axis_xgmii_tx_32.v  | 4 ++--
 tb/test_axis_xgmii_tx_64.py | 4 ++--
 tb/test_axis_xgmii_tx_64.v  | 4 ++--
 tb/test_eth_mac_10g_32.py   | 4 ++--
 tb/test_eth_mac_10g_32.v    | 4 ++--
 tb/test_eth_mac_10g_64.py   | 4 ++--
 tb/test_eth_mac_10g_64.v    | 4 ++--
 tb/test_eth_mac_1g.py       | 4 ++--
 tb/test_eth_mac_1g.v        | 4 ++--
 tb/test_eth_mac_phy_10g.py  | 4 ++--
 tb/test_eth_mac_phy_10g.v   | 4 ++--
 24 files changed, 56 insertions(+), 56 deletions(-)

diff --git a/rtl/axis_baser_tx_64.v b/rtl/axis_baser_tx_64.v
index f76704b99..fcaa8ea15 100644
--- a/rtl/axis_baser_tx_64.v
+++ b/rtl/axis_baser_tx_64.v
@@ -41,9 +41,9 @@ module axis_baser_tx_64 #
     parameter PTP_PERIOD_FNS = 16'h6666,
     parameter PTP_TS_ENABLE = 0,
     parameter PTP_TS_WIDTH = 96,
-    parameter PTP_TAG_ENABLE = 0,
+    parameter PTP_TAG_ENABLE = PTP_TS_ENABLE,
     parameter PTP_TAG_WIDTH = 16,
-    parameter USER_WIDTH = (PTP_TS_ENABLE && PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1
+    parameter USER_WIDTH = (PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1
 )
 (
     input  wire                      clk,
@@ -238,8 +238,8 @@ assign encoded_tx_data = encoded_tx_data_reg;
 assign encoded_tx_hdr = encoded_tx_hdr_reg;
 
 assign m_axis_ptp_ts = PTP_TS_ENABLE ? m_axis_ptp_ts_reg : 0;
-assign m_axis_ptp_ts_tag = PTP_TS_ENABLE && PTP_TAG_ENABLE ? m_axis_ptp_ts_tag_reg : 0;
-assign m_axis_ptp_ts_valid = PTP_TS_ENABLE ? m_axis_ptp_ts_valid_reg : 1'b0;
+assign m_axis_ptp_ts_tag = PTP_TAG_ENABLE ? m_axis_ptp_ts_tag_reg : 0;
+assign m_axis_ptp_ts_valid = PTP_TS_ENABLE || PTP_TAG_ENABLE ? m_axis_ptp_ts_valid_reg : 1'b0;
 
 assign start_packet = start_packet_reg;
 assign error_underflow = error_underflow_reg;
diff --git a/rtl/axis_gmii_tx.v b/rtl/axis_gmii_tx.v
index e16ef2ed8..de4c43998 100644
--- a/rtl/axis_gmii_tx.v
+++ b/rtl/axis_gmii_tx.v
@@ -36,9 +36,9 @@ module axis_gmii_tx #
     parameter MIN_FRAME_LENGTH = 64,
     parameter PTP_TS_ENABLE = 0,
     parameter PTP_TS_WIDTH = 96,
-    parameter PTP_TAG_ENABLE = 0,
+    parameter PTP_TAG_ENABLE = PTP_TS_ENABLE,
     parameter PTP_TAG_WIDTH = 16,
-    parameter USER_WIDTH = (PTP_TS_ENABLE && PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1
+    parameter USER_WIDTH = (PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1
 )
 (
     input  wire                      clk,
@@ -144,8 +144,8 @@ assign gmii_tx_en = gmii_tx_en_reg;
 assign gmii_tx_er = gmii_tx_er_reg;
 
 assign m_axis_ptp_ts = PTP_TS_ENABLE ? m_axis_ptp_ts_reg : 0;
-assign m_axis_ptp_ts_tag = PTP_TS_ENABLE && PTP_TAG_ENABLE ? m_axis_ptp_ts_tag_reg : 0;
-assign m_axis_ptp_ts_valid = PTP_TS_ENABLE ? m_axis_ptp_ts_valid_reg : 1'b0;
+assign m_axis_ptp_ts_tag = PTP_TAG_ENABLE ? m_axis_ptp_ts_tag_reg : 0;
+assign m_axis_ptp_ts_valid = PTP_TS_ENABLE || PTP_TAG_ENABLE ? m_axis_ptp_ts_valid_reg : 1'b0;
 
 assign start_packet = start_packet_reg;
 assign error_underflow = error_underflow_reg;
diff --git a/rtl/axis_xgmii_tx_32.v b/rtl/axis_xgmii_tx_32.v
index 0eae6d15e..ead5affff 100644
--- a/rtl/axis_xgmii_tx_32.v
+++ b/rtl/axis_xgmii_tx_32.v
@@ -39,9 +39,9 @@ module axis_xgmii_tx_32 #
     parameter MIN_FRAME_LENGTH = 64,
     parameter PTP_TS_ENABLE = 0,
     parameter PTP_TS_WIDTH = 96,
-    parameter PTP_TAG_ENABLE = 0,
+    parameter PTP_TAG_ENABLE = PTP_TS_ENABLE,
     parameter PTP_TAG_WIDTH = 16,
-    parameter USER_WIDTH = (PTP_TS_ENABLE && PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1
+    parameter USER_WIDTH = (PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1
 )
 (
     input  wire                      clk,
@@ -171,8 +171,8 @@ assign xgmii_txd = xgmii_txd_reg;
 assign xgmii_txc = xgmii_txc_reg;
 
 assign m_axis_ptp_ts = PTP_TS_ENABLE ? m_axis_ptp_ts_reg : 0;
-assign m_axis_ptp_ts_tag = PTP_TS_ENABLE && PTP_TAG_ENABLE ? m_axis_ptp_ts_tag_reg : 0;
-assign m_axis_ptp_ts_valid = PTP_TS_ENABLE ? m_axis_ptp_ts_valid_reg : 1'b0;
+assign m_axis_ptp_ts_tag = PTP_TAG_ENABLE ? m_axis_ptp_ts_tag_reg : 0;
+assign m_axis_ptp_ts_valid = PTP_TS_ENABLE || PTP_TAG_ENABLE ? m_axis_ptp_ts_valid_reg : 1'b0;
 
 assign start_packet = start_packet_reg;
 assign error_underflow = error_underflow_reg;
diff --git a/rtl/axis_xgmii_tx_64.v b/rtl/axis_xgmii_tx_64.v
index 6bb73a1a6..fb1c9b394 100644
--- a/rtl/axis_xgmii_tx_64.v
+++ b/rtl/axis_xgmii_tx_64.v
@@ -41,9 +41,9 @@ module axis_xgmii_tx_64 #
     parameter PTP_PERIOD_FNS = 16'h6666,
     parameter PTP_TS_ENABLE = 0,
     parameter PTP_TS_WIDTH = 96,
-    parameter PTP_TAG_ENABLE = 0,
+    parameter PTP_TAG_ENABLE = PTP_TS_ENABLE,
     parameter PTP_TAG_WIDTH = 16,
-    parameter USER_WIDTH = (PTP_TS_ENABLE && PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1
+    parameter USER_WIDTH = (PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1
 )
 (
     input  wire                      clk,
@@ -183,8 +183,8 @@ assign xgmii_txd = xgmii_txd_reg;
 assign xgmii_txc = xgmii_txc_reg;
 
 assign m_axis_ptp_ts = PTP_TS_ENABLE ? m_axis_ptp_ts_reg : 0;
-assign m_axis_ptp_ts_tag = PTP_TS_ENABLE && PTP_TAG_ENABLE ? m_axis_ptp_ts_tag_reg : 0;
-assign m_axis_ptp_ts_valid = PTP_TS_ENABLE ? m_axis_ptp_ts_valid_reg : 1'b0;
+assign m_axis_ptp_ts_tag = PTP_TAG_ENABLE ? m_axis_ptp_ts_tag_reg : 0;
+assign m_axis_ptp_ts_valid = PTP_TS_ENABLE || PTP_TAG_ENABLE ? m_axis_ptp_ts_valid_reg : 1'b0;
 
 assign start_packet = start_packet_reg;
 assign error_underflow = error_underflow_reg;
diff --git a/rtl/eth_mac_10g.v b/rtl/eth_mac_10g.v
index 9601a2e13..187389bce 100644
--- a/rtl/eth_mac_10g.v
+++ b/rtl/eth_mac_10g.v
@@ -41,11 +41,11 @@ module eth_mac_10g #
     parameter PTP_PERIOD_FNS = 16'h6666,
     parameter TX_PTP_TS_ENABLE = 0,
     parameter TX_PTP_TS_WIDTH = 96,
-    parameter TX_PTP_TAG_ENABLE = 0,
+    parameter TX_PTP_TAG_ENABLE = TX_PTP_TS_ENABLE,
     parameter TX_PTP_TAG_WIDTH = 16,
     parameter RX_PTP_TS_ENABLE = 0,
     parameter RX_PTP_TS_WIDTH = 96,
-    parameter TX_USER_WIDTH = (TX_PTP_TS_ENABLE && TX_PTP_TAG_ENABLE ? TX_PTP_TAG_WIDTH : 0) + 1,
+    parameter TX_USER_WIDTH = (TX_PTP_TAG_ENABLE ? TX_PTP_TAG_WIDTH : 0) + 1,
     parameter RX_USER_WIDTH = (RX_PTP_TS_ENABLE ? RX_PTP_TS_WIDTH : 0) + 1
 )
 (
diff --git a/rtl/eth_mac_1g.v b/rtl/eth_mac_1g.v
index 7eefcdf3f..4f1075c19 100644
--- a/rtl/eth_mac_1g.v
+++ b/rtl/eth_mac_1g.v
@@ -36,11 +36,11 @@ module eth_mac_1g #
     parameter MIN_FRAME_LENGTH = 64,
     parameter TX_PTP_TS_ENABLE = 0,
     parameter TX_PTP_TS_WIDTH = 96,
-    parameter TX_PTP_TAG_ENABLE = 0,
+    parameter TX_PTP_TAG_ENABLE = TX_PTP_TS_ENABLE,
     parameter TX_PTP_TAG_WIDTH = 16,
     parameter RX_PTP_TS_ENABLE = 0,
     parameter RX_PTP_TS_WIDTH = 96,
-    parameter TX_USER_WIDTH = (TX_PTP_TS_ENABLE && TX_PTP_TAG_ENABLE ? TX_PTP_TAG_WIDTH : 0) + 1,
+    parameter TX_USER_WIDTH = (TX_PTP_TAG_ENABLE ? TX_PTP_TAG_WIDTH : 0) + 1,
     parameter RX_USER_WIDTH = (RX_PTP_TS_ENABLE ? RX_PTP_TS_WIDTH : 0) + 1
 )
 (
diff --git a/rtl/eth_mac_phy_10g.v b/rtl/eth_mac_phy_10g.v
index a1b16ade1..1811e5f79 100644
--- a/rtl/eth_mac_phy_10g.v
+++ b/rtl/eth_mac_phy_10g.v
@@ -42,11 +42,11 @@ module eth_mac_phy_10g #
     parameter PTP_PERIOD_FNS = 16'h6666,
     parameter TX_PTP_TS_ENABLE = 0,
     parameter TX_PTP_TS_WIDTH = 96,
-    parameter TX_PTP_TAG_ENABLE = 0,
+    parameter TX_PTP_TAG_ENABLE = TX_PTP_TS_ENABLE,
     parameter TX_PTP_TAG_WIDTH = 16,
     parameter RX_PTP_TS_ENABLE = 0,
     parameter RX_PTP_TS_WIDTH = 96,
-    parameter TX_USER_WIDTH = (TX_PTP_TS_ENABLE && TX_PTP_TAG_ENABLE ? TX_PTP_TAG_WIDTH : 0) + 1,
+    parameter TX_USER_WIDTH = (TX_PTP_TAG_ENABLE ? TX_PTP_TAG_WIDTH : 0) + 1,
     parameter RX_USER_WIDTH = (RX_PTP_TS_ENABLE ? RX_PTP_TS_WIDTH : 0) + 1,
     parameter BIT_REVERSE = 0,
     parameter SCRAMBLER_DISABLE = 0,
diff --git a/rtl/eth_mac_phy_10g_tx.v b/rtl/eth_mac_phy_10g_tx.v
index 9a497e2d5..4b80b8cc5 100644
--- a/rtl/eth_mac_phy_10g_tx.v
+++ b/rtl/eth_mac_phy_10g_tx.v
@@ -42,9 +42,9 @@ module eth_mac_phy_10g_tx #
     parameter PTP_PERIOD_FNS = 16'h6666,
     parameter PTP_TS_ENABLE = 0,
     parameter PTP_TS_WIDTH = 96,
-    parameter PTP_TAG_ENABLE = 0,
+    parameter PTP_TAG_ENABLE = PTP_TS_ENABLE,
     parameter PTP_TAG_WIDTH = 16,
-    parameter USER_WIDTH = (PTP_TS_ENABLE && PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1,
+    parameter USER_WIDTH = (PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1,
     parameter BIT_REVERSE = 0,
     parameter SCRAMBLER_DISABLE = 0,
     parameter PRBS31_ENABLE = 0
diff --git a/tb/test_axis_baser_tx_64.py b/tb/test_axis_baser_tx_64.py
index f807214ef..62269ecb6 100755
--- a/tb/test_axis_baser_tx_64.py
+++ b/tb/test_axis_baser_tx_64.py
@@ -56,9 +56,9 @@ def bench():
     PTP_PERIOD_FNS = 0x6666
     PTP_TS_ENABLE = 0
     PTP_TS_WIDTH = 96
-    PTP_TAG_ENABLE = 0
+    PTP_TAG_ENABLE = PTP_TS_ENABLE
     PTP_TAG_WIDTH = 16
-    USER_WIDTH = (PTP_TAG_WIDTH if PTP_TS_ENABLE and PTP_TAG_ENABLE else 0) + 1
+    USER_WIDTH = (PTP_TAG_WIDTH if PTP_TAG_ENABLE else 0) + 1
 
     # Inputs
     clk = Signal(bool(0))
diff --git a/tb/test_axis_baser_tx_64.v b/tb/test_axis_baser_tx_64.v
index 5aac11293..1c62d6bdd 100644
--- a/tb/test_axis_baser_tx_64.v
+++ b/tb/test_axis_baser_tx_64.v
@@ -42,9 +42,9 @@ parameter PTP_PERIOD_NS = 4'h6;
 parameter PTP_PERIOD_FNS = 16'h6666;
 parameter PTP_TS_ENABLE = 0;
 parameter PTP_TS_WIDTH = 96;
-parameter PTP_TAG_ENABLE = 0;
+parameter PTP_TAG_ENABLE = PTP_TS_ENABLE;
 parameter PTP_TAG_WIDTH = 16;
-parameter USER_WIDTH = (PTP_TS_ENABLE && PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1;
+parameter USER_WIDTH = (PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1;
 
 // Inputs
 reg clk = 0;
diff --git a/tb/test_axis_gmii_tx.py b/tb/test_axis_gmii_tx.py
index 041fef7d0..1c47c6a57 100755
--- a/tb/test_axis_gmii_tx.py
+++ b/tb/test_axis_gmii_tx.py
@@ -51,9 +51,9 @@ def bench():
     MIN_FRAME_LENGTH = 64
     PTP_TS_ENABLE = 0
     PTP_TS_WIDTH = 96
-    PTP_TAG_ENABLE = 0
+    PTP_TAG_ENABLE = PTP_TS_ENABLE
     PTP_TAG_WIDTH = 16
-    USER_WIDTH = (PTP_TAG_WIDTH if PTP_TS_ENABLE and PTP_TAG_ENABLE else 0) + 1
+    USER_WIDTH = (PTP_TAG_WIDTH if PTP_TAG_ENABLE else 0) + 1
 
     # Inputs
     clk = Signal(bool(0))
diff --git a/tb/test_axis_gmii_tx.v b/tb/test_axis_gmii_tx.v
index 44ea2a234..f24b03804 100644
--- a/tb/test_axis_gmii_tx.v
+++ b/tb/test_axis_gmii_tx.v
@@ -37,9 +37,9 @@ parameter ENABLE_PADDING = 1;
 parameter MIN_FRAME_LENGTH = 64;
 parameter PTP_TS_ENABLE = 0;
 parameter PTP_TS_WIDTH = 96;
-parameter PTP_TAG_ENABLE = 0;
+parameter PTP_TAG_ENABLE = PTP_TS_ENABLE;
 parameter PTP_TAG_WIDTH = 16;
-parameter USER_WIDTH = (PTP_TS_ENABLE && PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1;
+parameter USER_WIDTH = (PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1;
 
 // Inputs
 reg clk = 0;
diff --git a/tb/test_axis_xgmii_tx_32.py b/tb/test_axis_xgmii_tx_32.py
index 41dc13631..6f3f48edd 100755
--- a/tb/test_axis_xgmii_tx_32.py
+++ b/tb/test_axis_xgmii_tx_32.py
@@ -54,9 +54,9 @@ def bench():
     MIN_FRAME_LENGTH = 64
     PTP_TS_ENABLE = 0
     PTP_TS_WIDTH = 96
-    PTP_TAG_ENABLE = 0
+    PTP_TAG_ENABLE = PTP_TS_ENABLE
     PTP_TAG_WIDTH = 16
-    USER_WIDTH = (PTP_TAG_WIDTH if PTP_TS_ENABLE and PTP_TAG_ENABLE else 0) + 1
+    USER_WIDTH = (PTP_TAG_WIDTH if PTP_TAG_ENABLE else 0) + 1
 
     # Inputs
     clk = Signal(bool(0))
diff --git a/tb/test_axis_xgmii_tx_32.v b/tb/test_axis_xgmii_tx_32.v
index 7af249a4e..9f2615378 100644
--- a/tb/test_axis_xgmii_tx_32.v
+++ b/tb/test_axis_xgmii_tx_32.v
@@ -40,9 +40,9 @@ parameter ENABLE_DIC = 1;
 parameter MIN_FRAME_LENGTH = 64;
 parameter PTP_TS_ENABLE = 0;
 parameter PTP_TS_WIDTH = 96;
-parameter PTP_TAG_ENABLE = 0;
+parameter PTP_TAG_ENABLE = PTP_TS_ENABLE;
 parameter PTP_TAG_WIDTH = 16;
-parameter USER_WIDTH = (PTP_TS_ENABLE && PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1;
+parameter USER_WIDTH = (PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1;
 
 // Inputs
 reg clk = 0;
diff --git a/tb/test_axis_xgmii_tx_64.py b/tb/test_axis_xgmii_tx_64.py
index 0845a88a0..aa547938c 100755
--- a/tb/test_axis_xgmii_tx_64.py
+++ b/tb/test_axis_xgmii_tx_64.py
@@ -56,9 +56,9 @@ def bench():
     PTP_PERIOD_FNS = 0x6666
     PTP_TS_ENABLE = 0
     PTP_TS_WIDTH = 96
-    PTP_TAG_ENABLE = 0
+    PTP_TAG_ENABLE = PTP_TS_ENABLE
     PTP_TAG_WIDTH = 16
-    USER_WIDTH = (PTP_TAG_WIDTH if PTP_TS_ENABLE and PTP_TAG_ENABLE else 0) + 1
+    USER_WIDTH = (PTP_TAG_WIDTH if PTP_TAG_ENABLE else 0) + 1
 
     # Inputs
     clk = Signal(bool(0))
diff --git a/tb/test_axis_xgmii_tx_64.v b/tb/test_axis_xgmii_tx_64.v
index 8fdc43426..e840edd11 100644
--- a/tb/test_axis_xgmii_tx_64.v
+++ b/tb/test_axis_xgmii_tx_64.v
@@ -42,9 +42,9 @@ parameter PTP_PERIOD_NS = 4'h6;
 parameter PTP_PERIOD_FNS = 16'h6666;
 parameter PTP_TS_ENABLE = 0;
 parameter PTP_TS_WIDTH = 96;
-parameter PTP_TAG_ENABLE = 0;
+parameter PTP_TAG_ENABLE = PTP_TS_ENABLE;
 parameter PTP_TAG_WIDTH = 16;
-parameter USER_WIDTH = (PTP_TS_ENABLE && PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1;
+parameter USER_WIDTH = (PTP_TAG_ENABLE ? PTP_TAG_WIDTH : 0) + 1;
 
 // Inputs
 reg clk = 0;
diff --git a/tb/test_eth_mac_10g_32.py b/tb/test_eth_mac_10g_32.py
index ee290e30c..f63897201 100755
--- a/tb/test_eth_mac_10g_32.py
+++ b/tb/test_eth_mac_10g_32.py
@@ -56,11 +56,11 @@ def bench():
     MIN_FRAME_LENGTH = 64
     TX_PTP_TS_ENABLE = 0
     TX_PTP_TS_WIDTH = 96
-    TX_PTP_TAG_ENABLE = 0
+    TX_PTP_TAG_ENABLE = TX_PTP_TS_ENABLE
     TX_PTP_TAG_WIDTH = 16
     RX_PTP_TS_ENABLE = 0
     RX_PTP_TS_WIDTH = 96
-    TX_USER_WIDTH = (TX_PTP_TAG_WIDTH if TX_PTP_TS_ENABLE and TX_PTP_TAG_ENABLE else 0) + 1
+    TX_USER_WIDTH = (TX_PTP_TAG_WIDTH if TX_PTP_TAG_ENABLE else 0) + 1
     RX_USER_WIDTH = (RX_PTP_TS_WIDTH if RX_PTP_TS_ENABLE else 0) + 1
 
     # Inputs
diff --git a/tb/test_eth_mac_10g_32.v b/tb/test_eth_mac_10g_32.v
index 0a98ef33e..dd93fafc6 100644
--- a/tb/test_eth_mac_10g_32.v
+++ b/tb/test_eth_mac_10g_32.v
@@ -40,11 +40,11 @@ parameter ENABLE_DIC = 1;
 parameter MIN_FRAME_LENGTH = 64;
 parameter TX_PTP_TS_ENABLE = 0;
 parameter TX_PTP_TS_WIDTH = 96;
-parameter TX_PTP_TAG_ENABLE = 0;
+parameter TX_PTP_TAG_ENABLE = TX_PTP_TS_ENABLE;
 parameter TX_PTP_TAG_WIDTH = 16;
 parameter RX_PTP_TS_ENABLE = 0;
 parameter RX_PTP_TS_WIDTH = 96;
-parameter TX_USER_WIDTH = (TX_PTP_TS_ENABLE && TX_PTP_TAG_ENABLE ? TX_PTP_TAG_WIDTH : 0) + 1;
+parameter TX_USER_WIDTH = (TX_PTP_TAG_ENABLE ? TX_PTP_TAG_WIDTH : 0) + 1;
 parameter RX_USER_WIDTH = (RX_PTP_TS_ENABLE ? RX_PTP_TS_WIDTH : 0) + 1;
 
 // Inputs
diff --git a/tb/test_eth_mac_10g_64.py b/tb/test_eth_mac_10g_64.py
index 2662ed6c6..6b7e960b4 100755
--- a/tb/test_eth_mac_10g_64.py
+++ b/tb/test_eth_mac_10g_64.py
@@ -58,11 +58,11 @@ def bench():
     PTP_PERIOD_FNS = 0x6666
     TX_PTP_TS_ENABLE = 0
     TX_PTP_TS_WIDTH = 96
-    TX_PTP_TAG_ENABLE = 0
+    TX_PTP_TAG_ENABLE = TX_PTP_TS_ENABLE
     TX_PTP_TAG_WIDTH = 16
     RX_PTP_TS_ENABLE = 0
     RX_PTP_TS_WIDTH = 96
-    TX_USER_WIDTH = (TX_PTP_TAG_WIDTH if TX_PTP_TS_ENABLE and TX_PTP_TAG_ENABLE else 0) + 1
+    TX_USER_WIDTH = (TX_PTP_TAG_WIDTH if TX_PTP_TAG_ENABLE else 0) + 1
     RX_USER_WIDTH = (RX_PTP_TS_WIDTH if RX_PTP_TS_ENABLE else 0) + 1
 
     # Inputs
diff --git a/tb/test_eth_mac_10g_64.v b/tb/test_eth_mac_10g_64.v
index db0585b3b..20ffa25e0 100644
--- a/tb/test_eth_mac_10g_64.v
+++ b/tb/test_eth_mac_10g_64.v
@@ -42,11 +42,11 @@ parameter PTP_PERIOD_NS = 4'h6;
 parameter PTP_PERIOD_FNS = 16'h6666;
 parameter TX_PTP_TS_ENABLE = 0;
 parameter TX_PTP_TS_WIDTH = 96;
-parameter TX_PTP_TAG_ENABLE = 0;
+parameter TX_PTP_TAG_ENABLE = TX_PTP_TS_ENABLE;
 parameter TX_PTP_TAG_WIDTH = 16;
 parameter RX_PTP_TS_ENABLE = 0;
 parameter RX_PTP_TS_WIDTH = 96;
-parameter TX_USER_WIDTH = (TX_PTP_TS_ENABLE && TX_PTP_TAG_ENABLE ? TX_PTP_TAG_WIDTH : 0) + 1;
+parameter TX_USER_WIDTH = (TX_PTP_TAG_ENABLE ? TX_PTP_TAG_WIDTH : 0) + 1;
 parameter RX_USER_WIDTH = (RX_PTP_TS_ENABLE ? RX_PTP_TS_WIDTH : 0) + 1;
 
 // Inputs
diff --git a/tb/test_eth_mac_1g.py b/tb/test_eth_mac_1g.py
index 8a978e8d2..cf2ab16bf 100755
--- a/tb/test_eth_mac_1g.py
+++ b/tb/test_eth_mac_1g.py
@@ -53,11 +53,11 @@ def bench():
     MIN_FRAME_LENGTH = 64
     TX_PTP_TS_ENABLE = 0
     TX_PTP_TS_WIDTH = 96
-    TX_PTP_TAG_ENABLE = 0
+    TX_PTP_TAG_ENABLE = TX_PTP_TS_ENABLE
     TX_PTP_TAG_WIDTH = 16
     RX_PTP_TS_ENABLE = 0
     RX_PTP_TS_WIDTH = 96
-    TX_USER_WIDTH = (TX_PTP_TAG_WIDTH if TX_PTP_TS_ENABLE and TX_PTP_TAG_ENABLE else 0) + 1
+    TX_USER_WIDTH = (TX_PTP_TAG_WIDTH if TX_PTP_TAG_ENABLE else 0) + 1
     RX_USER_WIDTH = (RX_PTP_TS_WIDTH if RX_PTP_TS_ENABLE else 0) + 1
 
     # Inputs
diff --git a/tb/test_eth_mac_1g.v b/tb/test_eth_mac_1g.v
index 25b63dbb2..5790affca 100644
--- a/tb/test_eth_mac_1g.v
+++ b/tb/test_eth_mac_1g.v
@@ -37,11 +37,11 @@ parameter ENABLE_PADDING = 1;
 parameter MIN_FRAME_LENGTH = 64;
 parameter TX_PTP_TS_ENABLE = 0;
 parameter TX_PTP_TS_WIDTH = 96;
-parameter TX_PTP_TAG_ENABLE = 0;
+parameter TX_PTP_TAG_ENABLE = TX_PTP_TS_ENABLE;
 parameter TX_PTP_TAG_WIDTH = 16;
 parameter RX_PTP_TS_ENABLE = 0;
 parameter RX_PTP_TS_WIDTH = 96;
-parameter TX_USER_WIDTH = (TX_PTP_TS_ENABLE && TX_PTP_TAG_ENABLE ? TX_PTP_TAG_WIDTH : 0) + 1;
+parameter TX_USER_WIDTH = (TX_PTP_TAG_ENABLE ? TX_PTP_TAG_WIDTH : 0) + 1;
 parameter RX_USER_WIDTH = (RX_PTP_TS_ENABLE ? RX_PTP_TS_WIDTH : 0) + 1;
 
 // Inputs
diff --git a/tb/test_eth_mac_phy_10g.py b/tb/test_eth_mac_phy_10g.py
index 255aa6f0f..5f5f43e6c 100755
--- a/tb/test_eth_mac_phy_10g.py
+++ b/tb/test_eth_mac_phy_10g.py
@@ -66,11 +66,11 @@ def bench():
     PTP_PERIOD_FNS = 0x6666
     TX_PTP_TS_ENABLE = 0
     TX_PTP_TS_WIDTH = 96
-    TX_PTP_TAG_ENABLE = 0
+    TX_PTP_TAG_ENABLE = TX_PTP_TS_ENABLE
     TX_PTP_TAG_WIDTH = 16
     RX_PTP_TS_ENABLE = 0
     RX_PTP_TS_WIDTH = 96
-    TX_USER_WIDTH = (TX_PTP_TAG_WIDTH if TX_PTP_TS_ENABLE and TX_PTP_TAG_ENABLE else 0) + 1
+    TX_USER_WIDTH = (TX_PTP_TAG_WIDTH if TX_PTP_TAG_ENABLE else 0) + 1
     RX_USER_WIDTH = (RX_PTP_TS_WIDTH if RX_PTP_TS_ENABLE else 0) + 1
     BIT_REVERSE = 0
     SCRAMBLER_DISABLE = 0
diff --git a/tb/test_eth_mac_phy_10g.v b/tb/test_eth_mac_phy_10g.v
index 6a73719e9..45911b60f 100644
--- a/tb/test_eth_mac_phy_10g.v
+++ b/tb/test_eth_mac_phy_10g.v
@@ -43,11 +43,11 @@ parameter PTP_PERIOD_NS = 4'h6;
 parameter PTP_PERIOD_FNS = 16'h6666;
 parameter TX_PTP_TS_ENABLE = 0;
 parameter TX_PTP_TS_WIDTH = 96;
-parameter TX_PTP_TAG_ENABLE = 0;
+parameter TX_PTP_TAG_ENABLE = TX_PTP_TS_ENABLE;
 parameter TX_PTP_TAG_WIDTH = 16;
 parameter RX_PTP_TS_ENABLE = 0;
 parameter RX_PTP_TS_WIDTH = 96;
-parameter TX_USER_WIDTH = (TX_PTP_TS_ENABLE && TX_PTP_TAG_ENABLE ? TX_PTP_TAG_WIDTH : 0) + 1;
+parameter TX_USER_WIDTH = (TX_PTP_TAG_ENABLE ? TX_PTP_TAG_WIDTH : 0) + 1;
 parameter RX_USER_WIDTH = (RX_PTP_TS_ENABLE ? RX_PTP_TS_WIDTH : 0) + 1;
 parameter BIT_REVERSE = 0;
 parameter SCRAMBLER_DISABLE = 0;