From: Jacob Keller <jacob.e.kel...@intel.com>

A couple of E822 PTP functions use ICE_PTP_READ_TIME on the main timer to avoid
any side effects when programming a single PHY port. Match the ETH56G code by
using ICE_PTP_NOP instead, which better reflects the intended behavior.

Signed-off-by: Jacob Keller <jacob.e.kel...@intel.com>
Signed-off-by: Ian Stokes <ian.sto...@intel.com>
---
 drivers/net/ice/base/ice_ptp_hw.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/net/ice/base/ice_ptp_hw.c 
b/drivers/net/ice/base/ice_ptp_hw.c
index 27451a5897..e4b621591d 100644
--- a/drivers/net/ice/base/ice_ptp_hw.c
+++ b/drivers/net/ice/base/ice_ptp_hw.c
@@ -4114,8 +4114,8 @@ static int ice_sync_phy_timer_e822(struct ice_hw *hw, u8 
port)
        if (status)
                goto err_unlock;
 
-       /* Init PHC mstr/src cmd for exec during sync */
-       ice_ptp_src_cmd(hw, ICE_PTP_READ_TIME);
+       /* Do not perform any action on the main timer */
+       ice_ptp_src_cmd(hw, ICE_PTP_NOP);
 
        /* Issue the sync to activate the time adjustment */
        ice_ptp_exec_tmr_cmd(hw);
@@ -4242,8 +4242,8 @@ ice_start_phy_timer_e822(struct ice_hw *hw, u8 port, bool 
bypass)
        if (status)
                return status;
 
-       /* Init PHC mstr/src cmd for exec during sync */
-       ice_ptp_src_cmd(hw, ICE_PTP_READ_TIME);
+       /* Do not perform any action on the main timer */
+       ice_ptp_src_cmd(hw, ICE_PTP_NOP);
 
        ice_ptp_exec_tmr_cmd(hw);
 
-- 
2.43.0

Reply via email to