[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250508194825.3058929-1-vladimir.oltean@nxp.com>
Date: Thu, 8 May 2025 22:48:22 +0300
From: Vladimir Oltean <vladimir.oltean@....com>
To: netdev@...r.kernel.org
Cc: Köry Maincent <kory.maincent@...tlin.com>,
Andrew Lunn <andrew@...n.ch>,
Siddharth Vadapalli <s-vadapalli@...com>,
Roger Quadros <rogerq@...nel.org>,
"David S. Miller" <davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>,
Paolo Abeni <pabeni@...hat.com>,
Simon Horman <horms@...nel.org>,
Richard Cochran <richardcochran@...il.com>,
Russell King <linux@...linux.org.uk>,
linux-kernel@...r.kernel.org,
linux-omap@...r.kernel.org
Subject: [PATCH net-next 0/3] Convert cpsw and cpsw_new to ndo_hwtstamp_get() and ndo_hwtstamp_set()
This is probably the most difficult conversion to the new timestamping
API, because the cpsw driver makes some efforts to hide the existence of
two slave interfaces with two attached PHYs from the network stack, and
the whole point of the new timestamping API is to let the core stack
make more decisions. The cpsw_new driver doesn't do that.
Patch 2/3 is the main conversion.
Patch 3/3 is a subsequent cleanup/ simplification. It would have helped
my understanding if the code dealing with PHY ioctls wasn't common
between cpsw and cpsw_new, because there are important differences
between them.
Patch 1/3 is probably a minor fix, but in lack of a real-life report
I've targeted it to net-next (leaving the possibility for someone to
later request a backport). It would be very disruptive to my flow, at
this stage, to send this patch to 'net' and resend the rest after the
merge back into net-next.
I hope the changes are correct and reasonable, they are purely based on
static analysis and have only been compile-tested.
Vladimir Oltean (3):
net: cpsw: return proper RX timestamping filter in cpsw_hwtstamp_get()
net: cpsw: convert to ndo_hwtstamp_get() and ndo_hwtstamp_set()
net: cpsw: isolate cpsw_ndo_ioctl() to just the old driver
drivers/net/ethernet/ti/cpsw.c | 25 ++++++++++
drivers/net/ethernet/ti/cpsw_new.c | 4 +-
drivers/net/ethernet/ti/cpsw_priv.c | 71 +++++++++++------------------
drivers/net/ethernet/ti/cpsw_priv.h | 6 ++-
4 files changed, 59 insertions(+), 47 deletions(-)
--
2.43.0
Powered by blists - more mailing lists