[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20200510191240.413699-1-andrew@lunn.ch>
Date: Sun, 10 May 2020 21:12:29 +0200
From: Andrew Lunn <andrew@...n.ch>
To: David Miller <davem@...emloft.net>
Cc: netdev <netdev@...r.kernel.org>,
Florian Fainelli <f.fainelli@...il.com>,
Heiner Kallweit <hkallweit1@...il.com>,
Chris Healy <cphealy@...il.com>,
Michal Kubecek <mkubecek@...e.cz>, Andrew Lunn <andrew@...n.ch>
Subject: [PATCH net-next v4 00/10] Ethernet Cable test support
any copper Ethernet PHY have support for performing diagnostics of
the cable. Are the cable shorted, broken, not plugged into anything at
the other end? And they can report roughly how far along the cable any
fault is.
Add infrastructure in ethtool and phylib support for triggering a
cable test and reporting the results. The Marvell 1G PHY driver is
then extended to make use of this infrastructure.
For testing, a modified ethtool(1) can be found here:
https://github.com/lunn/ethtool.git feature/cable-test-v4. This also
contains extra code for TDR dump, which will be added to the kernel in
a later patch series.
Thanks to Chris Healy for extensive testing.
v2:
See individual patches but:
Remove _REPLY messages
Change length into a u32
Grammar fixes
Rename functions for consistency
Extack for cable test already running
Remove ethnl_cable_test_act_ops
Add status attributes
Rename pairs from numbers to letters
v3:
See individual patches but:
Remove ETHTOOL_MSG_CABLE_TEST_ACT_REPLY from documentation
Remove unused cable_test_get_policy
Fixed example in document
Add ETHTOOL_A_CABLE_NEST_* enum
Add ETHTOOL_MSG_CABLE_TEST_NTF to documentation
Poison phydev->skb
Return -EMSGSIZE when ethnl_bcastmsg_put() fails
Return valid error code when nla_nest_start() fails
Use u8 for results
Actually put u32 length into message
s/mavell/marvell/g
Remove include of <uapi/linux/ethtool_netlink.h>
EMSGSIZE when ethnl_bcastmsg_put() fails
Print an error message on failure, since this is a void function.
v4:
See individual patches but:
Remove unwanted blank line
ENOTSUPP->EOPNOTSUPP
Move EINVAL->EMSGSIZE fix to correct patch
Andrew Lunn (10):
net: phy: Add cable test support to state machine
net: phy: Add support for polling cable test
net: ethtool: netlink: Add support for triggering a cable test
net: ethtool: Add attributes for cable test reports
net: ethtool: Make helpers public
net: ethtool: Add infrastructure for reporting cable test results
net: ethtool: Add helpers for reporting test results
net: phy: marvell: Add cable test support
net: phy: Put interface into oper testing during cable test
net: phy: Send notifier when starting the cable test
Documentation/networking/ethtool-netlink.rst | 57 +++++-
drivers/net/phy/marvell.c | 201 +++++++++++++++++++
drivers/net/phy/phy.c | 106 ++++++++++
include/linux/ethtool_netlink.h | 33 +++
include/linux/phy.h | 42 ++++
include/uapi/linux/ethtool_netlink.h | 71 +++++++
net/Kconfig | 1 +
net/ethtool/Makefile | 2 +-
net/ethtool/cabletest.c | 201 +++++++++++++++++++
net/ethtool/netlink.c | 9 +-
net/ethtool/netlink.h | 3 +
11 files changed, 722 insertions(+), 4 deletions(-)
create mode 100644 net/ethtool/cabletest.c
--
2.26.2
Powered by blists - more mailing lists