All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jacob Keller <jacob.e.keller@intel.com>
To: intel-wired-lan@osuosl.org
Subject: [Intel-wired-lan] [net-next PATCH 0/9] implement support for PTP on E822 hardware
Date: Mon, 11 Oct 2021 14:54:24 -0700	[thread overview]
Message-ID: <20211011215433.1525457-1-jacob.e.keller@intel.com> (raw)

Extend the ice driver implementation to support PTP for the E822 based
devices.

This includes a few cleanup patches, that fix some minor issues spotted
while preparing them. In addition, there are some slight refactors to ease
the addition of E822 support, followed by adding the new hardware
implementation ice_ptp_hw.c.

There are a few major differences with E822 support compared to E810
support:

*) The E822 device has a Clock Generation Unit which must be initialized in
order to generate proper clock frequencies on the output that drives the PTP
hardware clock registers

*) The E822 PHY is a bit different and requires a more complex
initialization procedure which must be rerun any time the link configuration
changes.

*) The E822 devices support enhanced timestamp calibration by making use of
a process called Vernier offset measurement. This allows the hardware to
measure phase offset related to the PHY clocks for Serdes and FEC, reducing
the inaccuracy of the timestamp relative to the actual packet transmission
and receipt. Making use of this requires data gathered from the first
transmitted and received packets, and waiting for the PHY to complete the
calibration measurements. This is done as part of a new kthread, ov_work.
Note that to avoid delay in enabling timestamps, we start the PHY in
'bypass' mode which allows timestamps to be captured without the Vernier
calibration measurement. Once the first packets have been sent and received,
we then complete the calibration setup and exit bypass mode and begin using
the more precise timestamps. According to the datasheet, timestamps without
calibration data can be incorrect relative to actual receipt or transmission
by up to 1 clock cycle (~1.25 nanoseconds), while calibrated timestamps
should be correct to within 1/8th of a clock cycle (~0.15 nanoseconds).

*) E822 devices support crosstimestamping via PCIe PTM, which we enable when
available on the platform.

There is a fair amount of logic required to perform PHY and CGU
initialization, which is the vast majority of the new code, but it is fairly
self contained within ice_ptp_hw.c, with the exception of monitoring for
offset validity being handled by a kthread.

Changes since previous submission (v3):
* Fix a bug with SIOCSHWTSTAMP failing to update returned filter to
  HWTSTAMP_FILTER_ALL
* Fix an issue with the Tx timestamp slot assignment using modulo instead
  of division
* Rebased on top of Intel Wired LAN dev-queue
* Dropped the patches which were previously applied as fixes

Changes since v2:
* introduced helper functions to enable and disable clock outputs

Changes since v1:
* Fixed build against non-X86 arch for the PCIe PTM support


Jacob Keller (9):
  ice: introduce ice_base_incval function
  ice: PTP: move setting of tstamp_config
  ice: use 'int err' instead of 'int status' in ice_ptp_hw.c
  ice: introduce ice_ptp_init_phc function
  ice: convert clk_freq capability into time_ref
  ice: implement basic E822 PTP support
  ice: ensure the hardware Clock Generation Unit is configured
  ice: exit bypass mode once hardware finishes timestamp calibration
  ice: support crosstimestamping on E822 devices if supported

 drivers/net/ethernet/intel/ice/ice_cgu_regs.h |  116 +
 drivers/net/ethernet/intel/ice/ice_common.c   |   12 +
 .../net/ethernet/intel/ice/ice_hw_autogen.h   |    9 +
 drivers/net/ethernet/intel/ice/ice_main.c     |    7 +
 drivers/net/ethernet/intel/ice/ice_ptp.c      |  651 +++-
 drivers/net/ethernet/intel/ice/ice_ptp.h      |   30 +-
 .../net/ethernet/intel/ice/ice_ptp_consts.h   |  374 +++
 drivers/net/ethernet/intel/ice/ice_ptp_hw.c   | 2794 ++++++++++++++++-
 drivers/net/ethernet/intel/ice/ice_ptp_hw.h   |  345 ++
 drivers/net/ethernet/intel/ice/ice_type.h     |   23 +-
 10 files changed, 4160 insertions(+), 201 deletions(-)
 create mode 100644 drivers/net/ethernet/intel/ice/ice_cgu_regs.h
 create mode 100644 drivers/net/ethernet/intel/ice/ice_ptp_consts.h


base-commit: 0df65b003e9431e69d4e695681c67aedaf4a84ac
-- 
2.31.1.331.gb0c09ab8796f


             reply	other threads:[~2021-10-11 21:54 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-11 21:54 Jacob Keller [this message]
2021-10-11 21:54 ` [Intel-wired-lan] [net-next PATCH 1/9] ice: introduce ice_base_incval function Jacob Keller
2021-10-11 21:54 ` [Intel-wired-lan] [net-next PATCH 2/9] ice: PTP: move setting of tstamp_config Jacob Keller
2021-10-11 21:54 ` [Intel-wired-lan] [net-next PATCH 3/9] ice: use 'int err' instead of 'int status' in ice_ptp_hw.c Jacob Keller
2021-10-11 21:54 ` [Intel-wired-lan] [net-next PATCH 4/9] ice: introduce ice_ptp_init_phc function Jacob Keller
2021-10-11 21:54 ` [Intel-wired-lan] [net-next PATCH 5/9] ice: convert clk_freq capability into time_ref Jacob Keller
2021-10-11 21:54 ` [Intel-wired-lan] [net-next PATCH 6/9] ice: implement basic E822 PTP support Jacob Keller
2021-10-11 21:54 ` [Intel-wired-lan] [net-next PATCH 7/9] ice: ensure the hardware Clock Generation Unit is configured Jacob Keller
2021-10-11 21:54 ` [Intel-wired-lan] [net-next PATCH 8/9] ice: exit bypass mode once hardware finishes timestamp calibration Jacob Keller
2021-10-11 21:54 ` [Intel-wired-lan] [net-next PATCH 9/9] ice: support crosstimestamping on E822 devices if supported Jacob Keller
2021-10-12  0:22   ` Vinicius Costa Gomes
2021-10-12  0:58     ` Keller, Jacob E

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=20211011215433.1525457-1-jacob.e.keller@intel.com \
    --to=jacob.e.keller@intel.com \
    --cc=intel-wired-lan@osuosl.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.