[PATCH v1 1/1] net/ixgbe: fix PTP initialization for E610

[PATCH v1 1/1] net/ixgbe: fix PTP initialization for E610

[Anatoly Burakov]

Current codepath does not have case labels for E610 when initializing PTP.
Add them in relevant places.

Fixes: 316637762a5f ("net/ixgbe/base: enable E610 device")

Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
---
 drivers/net/ixgbe/ixgbe_ethdev.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/drivers/net/ixgbe/ixgbe_ethdev.c
index eb431889c3..d02d1e43a3 100644
--- a/drivers/net/ixgbe/ixgbe_ethdev.c
+++ b/drivers/net/ixgbe/ixgbe_ethdev.c
@@ -6741,6 +6741,7 @@ ixgbe_read_systime_cyclecounter(struct rte_eth_dev *dev)
 	case ixgbe_mac_X550:
 	case ixgbe_mac_X550EM_x:
 	case ixgbe_mac_X550EM_a:
+	case ixgbe_mac_E610:
 		/* SYSTIMEL stores ns and SYSTIMEH stores seconds. */
 		systime_cycles = (uint64_t)IXGBE_READ_REG(hw, IXGBE_SYSTIML);
 		systime_cycles += (uint64_t)IXGBE_READ_REG(hw, IXGBE_SYSTIMH)
@@ -6765,6 +6766,7 @@ ixgbe_read_rx_tstamp_cyclecounter(struct rte_eth_dev *dev)
 	case ixgbe_mac_X550:
 	case ixgbe_mac_X550EM_x:
 	case ixgbe_mac_X550EM_a:
+	case ixgbe_mac_E610:
 		/* RXSTMPL stores ns and RXSTMPH stores seconds. */
 		rx_tstamp_cycles = (uint64_t)IXGBE_READ_REG(hw, IXGBE_RXSTMPL);
 		rx_tstamp_cycles += (uint64_t)IXGBE_READ_REG(hw, IXGBE_RXSTMPH)
@@ -6790,6 +6792,7 @@ ixgbe_read_tx_tstamp_cyclecounter(struct rte_eth_dev *dev)
 	case ixgbe_mac_X550:
 	case ixgbe_mac_X550EM_x:
 	case ixgbe_mac_X550EM_a:
+	case ixgbe_mac_E610:
 		/* TXSTMPL stores ns and TXSTMPH stores seconds. */
 		tx_tstamp_cycles = (uint64_t)IXGBE_READ_REG(hw, IXGBE_TXSTMPL);
 		tx_tstamp_cycles += (uint64_t)IXGBE_READ_REG(hw, IXGBE_TXSTMPH)
@@ -6838,6 +6841,7 @@ ixgbe_start_timecounters(struct rte_eth_dev *dev)
 	case ixgbe_mac_X550:
 	case ixgbe_mac_X550EM_x:
 	case ixgbe_mac_X550EM_a:
+	case ixgbe_mac_E610:
 		/* Independent of link speed. */
 		incval = 1;
 		/* Cycles read will be interpreted as ns. */
-- 
2.43.5

RE: [PATCH v1 1/1] net/ixgbe: fix PTP initialization for E610

[Xu, HailinX]

> -----Original Message-----
> From: Anatoly Burakov <anatoly.burakov@intel.com>
> Sent: Friday, November 22, 2024 11:17 PM
> To: dev@dpdk.org; Medvedkin, Vladimir <vladimir.medvedkin@intel.com>;
> Kwapulinski, Piotr <piotr.kwapulinski@intel.com>; Carolyn Wyborny
> <carolyn.wyborny@intel.com>; Jagielski, Jedrzej <jedrzej.jagielski@intel.com>
> Subject: [PATCH v1 1/1] net/ixgbe: fix PTP initialization for E610
> 
> Current codepath does not have case labels for E610 when initializing PTP.
> Add them in relevant places.
> 
> Fixes: 316637762a5f ("net/ixgbe/base: enable E610 device")
> 
> Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
> ---
Tested-by: Hailin Xu <hailinx.xu@intel.com>

Re: [PATCH v1 1/1] net/ixgbe: fix PTP initialization for E610

[Bruce Richardson]

On Tue, Nov 26, 2024 at 08:32:09AM +0000, Xu, HailinX wrote:
> > -----Original Message-----
> > From: Anatoly Burakov <anatoly.burakov@intel.com>
> > Sent: Friday, November 22, 2024 11:17 PM
> > To: dev@dpdk.org; Medvedkin, Vladimir <vladimir.medvedkin@intel.com>;
> > Kwapulinski, Piotr <piotr.kwapulinski@intel.com>; Carolyn Wyborny
> > <carolyn.wyborny@intel.com>; Jagielski, Jedrzej <jedrzej.jagielski@intel.com>
> > Subject: [PATCH v1 1/1] net/ixgbe: fix PTP initialization for E610
> > 
> > Current codepath does not have case labels for E610 when initializing PTP.
> > Add them in relevant places.
> > 
> > Fixes: 316637762a5f ("net/ixgbe/base: enable E610 device")
> > 
> > Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
> > ---
> Tested-by: Hailin Xu <hailinx.xu@intel.com>

Applied to dpdk-next-net-intel

Thanks,
/Bruce