Skip to content

Two new PTP Hardware Clock features

Richard Cochran netdev 3 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest4 patches 4 repliesIn review
  2. v24 patches 5 repliesSuperseded
  3. v13 patches 11 repliesSuperseded

Patches in v3 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4ptp: remember the adjusted frequencydrivers/ptp/ptp_clock.c drivers/ptp/ptp_private.h+2 -00 replies
2/4ptp: provide the clock's adjusted frequencydrivers/ptp/ptp_clock.c+5 -00 replies
3/4ptp: link the phc device to its parent devicedrivers/net/ethernet/freescale/gianfar_ptp.c drivers/net/ethernet/intel/igb/igb_ptp.c drivers/net/ethernet/intel/ixgbe/ixgbe_ptp.c +6 more+18 -112 replies
4/4ptp: clarify the clock_name sysfs attributeDocumentation/ABI/testing/sysfs-ptp include/linux/ptp_clock_kernel.h+8 -21 reply

Reviews trailers per patch

1 of 4 patches has a Reviewed-by or Acked-by.

PatchBen HutchingsJeff Kirsher
1/4
2/4
3/4A-b A-b
4/4

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v3 thread

Cover letter [PATCH V3 net-next 0/4] Two new PTP Hardware Clock features

This patch series adds two new features to the PHC code.

* ChangeLog
** V3
- Use the correct parent device in the solarflare driver.
- Expand the sysfs documentation of clock_name.
- Also document the clock_name field in the header file.
** V2
- Preserves the clock_name attribute as it was meant to be, instead of making any changes to it.
- Covers the registration API change in the brand new solarflare phc device, which was overlooked in V1.

The first two patches let a user program find out the previously dialed frequency adjustment. This is primarily useful when restarting a PTP service, since without this information, the presumably correct adjustment will bias the new frequency estimation.

The third patch links the phc class device to its parent device within the driver model and sysfs.

The fourth patch adds a bit more documentation of the sysfs clock_name attribute. This should help clarify the naming scheme.

Thanks,
Richard

Richard Cochran (4):
ptp: remember the adjusted frequency
ptp: provide the clock's adjusted frequency
ptp: link the phc device to its parent device
ptp: clarify the clock_name sysfs attribute

 Documentation/ABI/testing/sysfs-ptp          |    6 +++++-
 drivers/net/ethernet/freescale/gianfar_ptp.c |    2 +-
 drivers/net/ethernet/intel/igb/igb_ptp.c     |    3 ++-
 drivers/net/ethernet/intel/ixgbe/ixgbe_ptp.c |    3 ++-
 drivers/net/ethernet/sfc/ptp.c               |    3 ++-
 drivers/net/phy/dp83640.c                    |    2 +-
 drivers/ptp/ptp_clock.c                      |   11 +++++++++--
 drivers/ptp/ptp_ixp46x.c                     |    2 +-
 drivers/ptp/ptp_pch.c                        |    2 +-
 drivers/ptp/ptp_private.h                    |    1 +
 include/linux/ptp_clock_kernel.h             |   11 ++++++++---
 11 files changed, 33 insertions(+), 13 deletions(-)
Signature · 1 line
1.7.2.5