RE: [Intel-wired-lan] [PATCH iwl-net] ice: Recalibrate PHY after settime64 on E825-C

"Loktionov, Aleksandr" <[email protected]>
Newsgroups org.kernel.vger.netdev,org.osuosl.intel-wired-lan
Message-ID <DS4PPF7551E65529058EA8F3867BDAFB22BE5DC2@DS4PPF7551E6552.namprd11.prod.outlook.com>

> -----Original Message-----
> From: Intel-wired-lan <[email protected]> On Behalf
> Of Tony Nguyen
> Sent: Wednesday, August 12, 2026 1:11 AM
> To: [email protected]
> Cc: Machnikowski, Maciej <[email protected]>;
> [email protected]; Keller, Jacob E <[email protected]>;
> Nguyen, Anthony L <[email protected]>
> Subject: [Intel-wired-lan] [PATCH iwl-net] ice: Recalibrate PHY after
> settime64 on E825-C
> 
> From: Maciek Machnikowski <[email protected]>
> 
> The PHY on E825-C requires recalibration after large jumps of the
> system time. Add missing PHY recalibration also on E825-C.
> 
> Fixes: 7cab44f1c35f ("ice: Introduce ETH56G PHY model for E825C
> products")
> Signed-off-by: Maciek Machnikowski <[email protected]>
> Signed-off-by: Tony Nguyen <[email protected]>
> ---
>  drivers/net/ethernet/intel/ice/ice_ptp.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/ethernet/intel/ice/ice_ptp.c
> b/drivers/net/ethernet/intel/ice/ice_ptp.c
> index 84b99cfc5675..bf625b9f4991 100644
> --- a/drivers/net/ethernet/intel/ice/ice_ptp.c
> +++ b/drivers/net/ethernet/intel/ice/ice_ptp.c
> @@ -1956,8 +1956,9 @@ ice_ptp_settime64(struct ptp_clock_info *info,
> const struct timespec64 *ts)
>  	/* Reenable periodic outputs */
>  	ice_ptp_enable_all_perout(pf);
> 
> -	/* Recalibrate and re-enable timestamp blocks for E822/E823 */
> -	if (hw->mac_type == ICE_MAC_GENERIC)
> +	/* Recalibrate and re-enable timestamp blocks for
> E822/E823/E825-C */
> +	if (hw->mac_type == ICE_MAC_GENERIC ||
> +	    hw->mac_type == ICE_MAC_GENERIC_3K_E825)
>  		ice_ptp_restart_all_phy(pf);
>  exit:
>  	if (err) {
> --
> 2.47.1

Reviewed-by: Aleksandr Loktionov <[email protected]>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.