From: Jacob Keller <jacob.e.keller@intel.com>
To: Intel Wired LAN <intel-wired-lan@lists.osuosl.org>,
Anthony Nguyen <anthony.l.nguyen@intel.com>,
netdev <netdev@vger.kernel.org>
Cc: Jacob Keller <jacob.e.keller@intel.com>,
Michal Kubiak <michal.kubiak@intel.com>,
Aleksandr Loktionov <aleksandr.loktionov@intel.com>,
Karol Kolacinski <karol.kolacinski@intel.com>,
Przemek Kitszel <przemyslaw.kitszel@intel.com>,
Milena Olech <milena.olech@intel.com>,
Paul Menzel <pmenzel@molgen.mpg.de>
Subject: [PATCH v4 00/15] ice: Separate TSPLL from PTP and clean up
Date: Thu, 01 May 2025 15:54:11 -0700 [thread overview]
Message-ID: <20250501-kk-tspll-improvements-alignment-v4-0-24c83d0ce7a8@intel.com> (raw)
Separate TSPLL related functions and definitions from all PTP-related
files and clean up the code by implementing multiple helpers.
Adjust TSPLL wait times and fall back to TCXO on lock failure to ensure
proper init flow of TSPLL.
Change default clock source for E825-C from TCXO to TIME_REF if its
available.
Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
---
Changes in v4:
- Rebase onto Intel Wired LAN dev-queue
- Restore use of string choice helpers
- Restore use of ice_debug()
- Improve title of commit to change the default clock source
- Link to v3: https://lore.kernel.org/r/20250430-kk-tspll-improvements-alignment-v3-0-ab8472e86204@intel.com
Changes in v3:
- Add SPDX headers to new files
- Fix .dest_dev assignments in patch #1
- Drop accidental re-sizing of e82x misc15 field in v2 #2
- Split v2 patch #6, with behavioral changes in new patch #3 and #8
- Split v2 patch #3, with removal of E825-C array to its own patch #4
- Fix kernel doc warnings
- Add a patch to also update the default E825-C clock source to TIME_REF
- Link to v2: https://lore.kernel.org/intel-wired-lan/20250409122830.1977644-12-karol.kolacinski@intel.com/
---
Jacob Keller (4):
ice: fix E825-C TSPLL register definitions
ice: clear time_sync_en field for E825-C during reprogramming
ice: read TSPLL registers again before reporting status
ice: default to TIME_REF instead of TXCO on E825-C
Karol Kolacinski (11):
ice: move TSPLL functions to a separate file
ice: rename TSPLL and CGU functions and definitions
ice: remove ice_tspll_params_e825 definitions
ice: use designated initializers for TSPLL consts
ice: add TSPLL log config helper
ice: add ICE_READ/WRITE_CGU_REG_OR_DIE helpers
ice: use bitfields instead of unions for CGU regs
ice: add multiple TSPLL helpers
ice: wait before enabling TSPLL
ice: fall back to TCXO on TSPLL lock fail
ice: move TSPLL init calls to ice_ptp.c
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.h | 58 +++
drivers/net/ethernet/intel/ice/ice_ptp_consts.h | 177 +-------
drivers/net/ethernet/intel/ice/ice_ptp_hw.h | 54 +--
drivers/net/ethernet/intel/ice/ice_tspll.h | 31 ++
drivers/net/ethernet/intel/ice/ice_type.h | 20 +-
drivers/net/ethernet/intel/ice/ice_common.c | 71 ++-
drivers/net/ethernet/intel/ice/ice_ptp.c | 14 +-
drivers/net/ethernet/intel/ice/ice_ptp_hw.c | 564 +-----------------------
drivers/net/ethernet/intel/ice/ice_tspll.c | 518 ++++++++++++++++++++++
drivers/net/ethernet/intel/ice/Makefile | 2 +-
12 files changed, 712 insertions(+), 979 deletions(-)
---
base-commit: e3945f0d3dac6bf6231a5501766c3614cd6f7f45
change-id: 20250417-kk-tspll-improvements-alignment-2cb078adf961
Best regards,
--
Jacob Keller <jacob.e.keller@intel.com>
next reply other threads:[~2025-05-01 22:54 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-01 22:54 Jacob Keller [this message]
2025-05-01 22:54 ` [PATCH v4 01/15] ice: move TSPLL functions to a separate file Jacob Keller
2025-05-22 14:13 ` [Intel-wired-lan] " Rinitha, SX
2025-05-01 22:54 ` [PATCH v4 02/15] ice: rename TSPLL and CGU functions and definitions Jacob Keller
2025-05-22 14:15 ` [Intel-wired-lan] " Rinitha, SX
2025-05-01 22:54 ` [PATCH v4 03/15] ice: fix E825-C TSPLL register definitions Jacob Keller
2025-05-22 14:18 ` [Intel-wired-lan] " Rinitha, SX
2025-05-01 22:54 ` [PATCH v4 04/15] ice: remove ice_tspll_params_e825 definitions Jacob Keller
2025-05-22 14:19 ` [Intel-wired-lan] " Rinitha, SX
2025-05-01 22:54 ` [PATCH v4 05/15] ice: use designated initializers for TSPLL consts Jacob Keller
2025-05-22 14:21 ` [Intel-wired-lan] " Rinitha, SX
2025-05-01 22:54 ` [PATCH v4 06/15] ice: add TSPLL log config helper Jacob Keller
2025-05-22 14:23 ` [Intel-wired-lan] " Rinitha, SX
2025-05-01 22:54 ` [PATCH v4 07/15] ice: add ICE_READ/WRITE_CGU_REG_OR_DIE helpers Jacob Keller
2025-05-22 14:24 ` [Intel-wired-lan] " Rinitha, SX
2025-05-01 22:54 ` [PATCH v4 08/15] ice: clear time_sync_en field for E825-C during reprogramming Jacob Keller
2025-05-22 14:27 ` [Intel-wired-lan] " Rinitha, SX
2025-05-01 22:54 ` [PATCH v4 09/15] ice: read TSPLL registers again before reporting status Jacob Keller
2025-05-22 14:28 ` [Intel-wired-lan] " Rinitha, SX
2025-05-01 22:54 ` [PATCH v4 10/15] ice: use bitfields instead of unions for CGU regs Jacob Keller
2025-05-22 14:30 ` [Intel-wired-lan] " Rinitha, SX
2025-05-01 22:54 ` [PATCH v4 11/15] ice: add multiple TSPLL helpers Jacob Keller
2025-05-22 14:32 ` [Intel-wired-lan] " Rinitha, SX
2025-05-01 22:54 ` [PATCH v4 12/15] ice: wait before enabling TSPLL Jacob Keller
2025-05-22 14:34 ` [Intel-wired-lan] " Rinitha, SX
2025-05-01 22:54 ` [PATCH v4 13/15] ice: fall back to TCXO on TSPLL lock fail Jacob Keller
2025-05-22 14:36 ` [Intel-wired-lan] " Rinitha, SX
2025-05-01 22:54 ` [PATCH v4 14/15] ice: move TSPLL init calls to ice_ptp.c Jacob Keller
2025-05-22 14:38 ` [Intel-wired-lan] " Rinitha, SX
2025-05-01 22:54 ` [PATCH v4 15/15] ice: default to TIME_REF instead of TXCO on E825-C Jacob Keller
2025-05-22 14:40 ` [Intel-wired-lan] " Rinitha, SX
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250501-kk-tspll-improvements-alignment-v4-0-24c83d0ce7a8@intel.com \
--to=jacob.e.keller@intel.com \
--cc=aleksandr.loktionov@intel.com \
--cc=anthony.l.nguyen@intel.com \
--cc=intel-wired-lan@lists.osuosl.org \
--cc=karol.kolacinski@intel.com \
--cc=michal.kubiak@intel.com \
--cc=milena.olech@intel.com \
--cc=netdev@vger.kernel.org \
--cc=pmenzel@molgen.mpg.de \
--cc=przemyslaw.kitszel@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).