From: Andrew Lunn <andrew@lunn.ch>
To: Wei Fang <wei.fang@nxp.com>
Cc: robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org,
richardcochran@gmail.com, claudiu.manoil@nxp.com,
vladimir.oltean@nxp.com, xiaoning.wang@nxp.com,
andrew+netdev@lunn.ch, davem@davemloft.net, edumazet@google.com,
kuba@kernel.org, pabeni@redhat.com, vadim.fedorenko@linux.dev,
Frank.Li@nxp.com, shawnguo@kernel.org, s.hauer@pengutronix.de,
festevam@gmail.com, fushi.peng@nxp.com,
devicetree@vger.kernel.org, netdev@vger.kernel.org,
linux-kernel@vger.kernel.org, imx@lists.linux.dev,
kernel@pengutronix.de
Subject: Re: [PATCH v5 net-next 08/15] ptp: netc: add debugfs support to loop back pulse signal
Date: Mon, 25 Aug 2025 14:36:31 +0200 [thread overview]
Message-ID: <13c4fe5a-878c-4a08-87df-f5bb96f0b589@lunn.ch> (raw)
In-Reply-To: <20250825041532.1067315-9-wei.fang@nxp.com>
On Mon, Aug 25, 2025 at 12:15:25PM +0800, Wei Fang wrote:
> The NETC Timer supports to loop back the output pulse signal of Fiper-n
> into Trigger-n input, so that we can leverage this feature to validate
> some other features without external hardware support.
This seems like it should be a common feature which more PTP clocks
will support? Did you search around and see if there are other devices
with this?
Ah:
ptp_qoriq_debugfs.c: if (!debugfs_create_file_unsafe("fiper1-loopback", 0600, root,
ptp_qoriq_debugfs.c: if (!debugfs_create_file_unsafe("fiper2-loopback", 0600, root,
Rather than hiding this in debugfs where each vendor will implement it
differently, i'm just think it might make sense to have an official
kernel API for it.
Andrew
next prev parent reply other threads:[~2025-08-25 12:36 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-25 4:15 [PATCH v5 net-next 00/15] Add NETC Timer PTP driver and add PTP support for i.MX95 Wei Fang
2025-08-25 4:15 ` [PATCH v5 net-next 01/15] dt-bindings: ptp: add NETC Timer PTP clock Wei Fang
2025-08-25 15:05 ` Frank Li
2025-08-25 4:15 ` [PATCH v5 net-next 02/15] dt-bindings: net: move ptp-timer property to ethernet-controller.yaml Wei Fang
2025-08-25 4:15 ` [PATCH v5 net-next 03/15] ptp: add helpers to get the phc_index by of_node or dev Wei Fang
2025-08-25 4:15 ` [PATCH v5 net-next 04/15] ptp: netc: add NETC V4 Timer PTP driver support Wei Fang
2025-08-25 15:06 ` Frank Li
2025-08-25 4:15 ` [PATCH v5 net-next 05/15] ptp: netc: add PTP_CLK_REQ_PPS support Wei Fang
2025-08-25 15:09 ` Frank Li
2025-08-25 4:15 ` [PATCH v5 net-next 06/15] ptp: netc: add periodic pulse output support Wei Fang
2025-08-25 4:15 ` [PATCH v5 net-next 07/15] ptp: netc: add external trigger stamp support Wei Fang
2025-08-25 4:15 ` [PATCH v5 net-next 08/15] ptp: netc: add debugfs support to loop back pulse signal Wei Fang
2025-08-25 12:36 ` Andrew Lunn [this message]
2025-08-26 1:50 ` Wei Fang
2025-08-26 2:22 ` Andrew Lunn
2025-08-26 5:40 ` Wei Fang
2025-08-25 4:15 ` [PATCH v5 net-next 09/15] MAINTAINERS: add NETC Timer PTP clock driver section Wei Fang
2025-08-25 4:15 ` [PATCH v5 net-next 10/15] net: enetc: save the parsed information of PTP packet to skb->cb Wei Fang
2025-08-25 4:15 ` [PATCH v5 net-next 11/15] net: enetc: extract enetc_update_ptp_sync_msg() to handle PTP Sync packets Wei Fang
2025-08-25 4:15 ` [PATCH v5 net-next 12/15] net: enetc: remove unnecessary CONFIG_FSL_ENETC_PTP_CLOCK check Wei Fang
2025-08-25 4:15 ` [PATCH v5 net-next 13/15] net: enetc: add PTP synchronization support for ENETC v4 Wei Fang
2025-08-25 15:17 ` Frank Li
2025-08-26 2:11 ` Wei Fang
2025-08-25 4:15 ` [PATCH v5 net-next 14/15] net: enetc: don't update sync packet checksum if checksum offload is used Wei Fang
2025-08-25 4:15 ` [PATCH v5 15/15] arm64: dts: imx95: add standard PCI device compatible string to NETC Timer Wei Fang
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=13c4fe5a-878c-4a08-87df-f5bb96f0b589@lunn.ch \
--to=andrew@lunn.ch \
--cc=Frank.Li@nxp.com \
--cc=andrew+netdev@lunn.ch \
--cc=claudiu.manoil@nxp.com \
--cc=conor+dt@kernel.org \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=edumazet@google.com \
--cc=festevam@gmail.com \
--cc=fushi.peng@nxp.com \
--cc=imx@lists.linux.dev \
--cc=kernel@pengutronix.de \
--cc=krzk+dt@kernel.org \
--cc=kuba@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=richardcochran@gmail.com \
--cc=robh@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=shawnguo@kernel.org \
--cc=vadim.fedorenko@linux.dev \
--cc=vladimir.oltean@nxp.com \
--cc=wei.fang@nxp.com \
--cc=xiaoning.wang@nxp.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).