> -----Original Message----- > From: Intel-wired-lan <intel-wired-lan-boun...@osuosl.org> On Behalf Of Karol > Kolacinski > Sent: Monday, January 8, 2024 6:17 PM > To: intel-wired-...@lists.osuosl.org > Cc: Keller, Jacob E <jacob.e.kel...@intel.com>; net...@vger.kernel.org; > Kolacinski, Karol <karol.kolacin...@intel.com>; Nguyen, Anthony L > <anthony.l.ngu...@intel.com>; Brandeburg, Jesse <jesse.brandeb...@intel.com> > Subject: [Intel-wired-lan] [PATCH v5 iwl-next 5/6] ice: factor out > ice_ptp_rebuild_owner() > > From: Jacob Keller <jacob.e.kel...@intel.com> > > The ice_ptp_reset() function uses a goto to skip past clock owner > operations if performing a PF reset or if the device is not the clock > owner. This is a bit confusing. Factor this out into > ice_ptp_rebuild_owner() instead. > > The ice_ptp_reset() function is called by ice_rebuild() to restore PTP > functionality after a device reset. Follow the convention set by the > ice_main.c file and rename this function to ice_ptp_rebuild(), in the > same way that we have ice_prepare_for_reset() and > ice_ptp_prepare_for_reset(). > > 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_main.c | 2 +- > drivers/net/ethernet/intel/ice/ice_ptp.c | 66 ++++++++++++++--------- > drivers/net/ethernet/intel/ice/ice_ptp.h | 6 +-- > 3 files changed, 44 insertions(+), 30 deletions(-) >
Tested-by: Pucha Himasekhar Reddy <himasekharx.reddy.pu...@intel.com> (A Contingent worker at Intel)
