On 3/10/2025 4:12 AM, Karol Kolacinski wrote:
Collect TSPLL related functions and definitions and move them to
a separate file to have all TSPLL functionality in one place.

Move CGU related functions and definitions to ice_common.*

You lost changes with the move, specifically 4c9f13a65426 ("ice: use string choice helpers"). Please restore them. Also, please use the change/helper in patch 4.

Checkpatch reports missing SPDX:

WARNING: Missing or malformed SPDX-License-Identifier tag in line 1
#1323: FILE: drivers/net/ethernet/intel/ice/ice_tspll.c:1:

WARNING: Missing or malformed SPDX-License-Identifier tag in line 1
#1972: FILE: drivers/net/ethernet/intel/ice/ice_tspll.h:1:


Reviewed-by: Michal Kubiak <michal.kub...@intel.com>
Reviewed-by: Milena Olech <milena-ol...@intel.com>
Signed-off-by: Karol Kolacinski <karol.kolacin...@intel.com>
---
  drivers/net/ethernet/intel/ice/Makefile       |   2 +-
  drivers/net/ethernet/intel/ice/ice.h          |   1 +
  drivers/net/ethernet/intel/ice/ice_cgu_regs.h | 181 -----
  drivers/net/ethernet/intel/ice/ice_common.c   |  61 ++
  drivers/net/ethernet/intel/ice/ice_common.h   | 176 +++++
  drivers/net/ethernet/intel/ice/ice_ptp.c      |   1 -
  .../net/ethernet/intel/ice/ice_ptp_consts.h   | 161 -----
  drivers/net/ethernet/intel/ice/ice_ptp_hw.c   | 542 ---------------
  drivers/net/ethernet/intel/ice/ice_ptp_hw.h   |  43 --
  drivers/net/ethernet/intel/ice/ice_tspll.c    | 643 ++++++++++++++++++
  drivers/net/ethernet/intel/ice/ice_tspll.h    |  43 ++
  11 files changed, 925 insertions(+), 929 deletions(-)
  delete mode 100644 drivers/net/ethernet/intel/ice/ice_cgu_regs.h
  create mode 100644 drivers/net/ethernet/intel/ice/ice_tspll.c
  create mode 100644 drivers/net/ethernet/intel/ice/ice_tspll.h


Reply via email to