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

The ice_ptp_configure_tx_tstamp function writes to PFINT_OICR_ENA to
configure it with the PFINT_OICR_TX_TSYN_M bit. The name of this
function is confusing because there are multiple other functions with
almost identical names.

Rename it to ice_ptp_cfg_tx_interrupt in order to make it more obvious
to the reader what action it performs.

Signed-off-by: Jacob Keller <jacob.e.kel...@intel.com>
Signed-off-by: Karol Kolacinski <karol.kolacin...@intel.com>
Reviewed-by: Jacob Keller <jacob.e.kel...@intel.com>
---
 drivers/net/ethernet/intel/ice/ice_ptp.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/drivers/net/ethernet/intel/ice/ice_ptp.c 
b/drivers/net/ethernet/intel/ice/ice_ptp.c
index d06840fe50ff..d4bc6dd0d76b 100644
--- a/drivers/net/ethernet/intel/ice/ice_ptp.c
+++ b/drivers/net/ethernet/intel/ice/ice_ptp.c
@@ -256,11 +256,11 @@ ice_verify_pin_e810t(struct ptp_clock_info *info, 
unsigned int pin,
 }
 
 /**
- * ice_ptp_configure_tx_tstamp - Enable or disable Tx timestamp interrupt
- * @pf: The PF pointer to search in
+ * ice_ptp_cfg_tx_interrupt - Configure Tx timestamp interrupt for the device
+ * @pf: Board private structure
  * @on: bool value for whether timestamp interrupt is enabled or disabled
  */
-static void ice_ptp_configure_tx_tstamp(struct ice_pf *pf, bool on)
+static void ice_ptp_cfg_tx_interrupt(struct ice_pf *pf, bool on)
 {
        u32 val;
 
@@ -295,7 +295,7 @@ static void ice_set_tx_tstamp(struct ice_pf *pf, bool on)
        }
 
        if (pf->ptp.tx_interrupt_mode == ICE_PTP_TX_INTERRUPT_SELF)
-               ice_ptp_configure_tx_tstamp(pf, on);
+               ice_ptp_cfg_tx_interrupt(pf, on);
 
        pf->ptp.tstamp_config.tx_type = on ? HWTSTAMP_TX_ON : HWTSTAMP_TX_OFF;
 }
@@ -2564,7 +2564,7 @@ void ice_ptp_reset(struct ice_pf *pf, enum ice_reset_req 
reset_type)
                ice_ptp_restart_all_phy(pf);
 
        if (ptp->tx_interrupt_mode)
-               ice_ptp_configure_tx_tstamp(pf, true);
+               ice_ptp_cfg_tx_interrupt(pf, true);
 
        /* Start periodic work going */
        kthread_queue_delayed_work(ptp->kworker, &ptp->work, 0);
@@ -2829,7 +2829,7 @@ static int ice_ptp_init_owner(struct ice_pf *pf)
                /* The clock owner for this device type handles the timestamp
                 * interrupt for all ports.
                 */
-               ice_ptp_configure_tx_tstamp(pf, true);
+               ice_ptp_cfg_tx_interrupt(pf, true);
 
                /* React on all quads interrupts for E82x */
                wr32(hw, PFINT_TSYN_MSK + (0x4 * hw->pf_id), (u32)0x1f);
@@ -2907,7 +2907,7 @@ static int ice_ptp_init_port(struct ice_pf *pf, struct 
ice_ptp_port *ptp_port)
                 * neither on own quad nor on others
                 */
                if (!ice_ptp_pf_handles_tx_interrupt(pf)) {
-                       ice_ptp_configure_tx_tstamp(pf, false);
+                       ice_ptp_cfg_tx_interrupt(pf, false);
                        wr32(hw, PFINT_TSYN_MSK + (0x4 * hw->pf_id), (u32)0x0);
                }
                kthread_init_delayed_work(&ptp_port->ov_work,
-- 
2.40.1

_______________________________________________
Intel-wired-lan mailing list
Intel-wired-lan@osuosl.org
https://lists.osuosl.org/mailman/listinfo/intel-wired-lan

Reply via email to