From: Oleg Akhrem <oleg.akh...@intel.com>

Removed redundant code. The *clk_freq and *clk_src are not modified.

Signed-off-by: Oleg Akhrem <oleg.akh...@intel.com>
Signed-off-by: Dhanya Pillai <dhanya.r.pil...@intel.com>
---
 drivers/net/intel/ice/base/ice_ptp_hw.c | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/drivers/net/intel/ice/base/ice_ptp_hw.c 
b/drivers/net/intel/ice/base/ice_ptp_hw.c
index 1e92e5ff21..7d16965674 100644
--- a/drivers/net/intel/ice/base/ice_ptp_hw.c
+++ b/drivers/net/intel/ice/base/ice_ptp_hw.c
@@ -520,9 +520,6 @@ ice_cfg_cgu_pll_e825c(struct ice_hw *hw, enum 
ice_time_ref_freq *clk_freq,
                  ice_clk_src_str(dw23.field.time_ref_sel),
                  ice_clk_freq_str(dw9.field.time_ref_freq_sel));
 
-       *clk_freq = (enum ice_time_ref_freq)dw9.field.time_ref_freq_sel;
-       *clk_src = (enum ice_clk_src)dw23.field.time_ref_sel;
-
        return 0;
 }
 
@@ -798,11 +795,11 @@ static int ice_init_cgu_e82x(struct ice_hw *hw)
                ice_warn(hw, "Failed to lock TS PLL to predefined frequency. 
Retrying with fallback frequency.\n");
 
                /* Try to lock to internal 25 MHz TCXO as a fallback */
+               time_ref_freq = ICE_TIME_REF_FREQ_25_000;
+               clk_src = ICE_CLK_SRC_TCX0;
                if (hw->phy_model == ICE_PHY_ETH56G)
                        time_ref_freq = ICE_TIME_REF_FREQ_156_250;
-               else
-                       time_ref_freq = ICE_TIME_REF_FREQ_25_000;
-               clk_src = ICE_CLK_SRC_TCX0;
+
                if (ice_is_e825c(hw))
                        err = ice_cfg_cgu_pll_e825c(hw, &time_ref_freq,
                                                    &clk_src);
-- 
2.43.0

Reply via email to