From patchwork Sun May 17 19:58:49 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andrew Lunn X-Patchwork-Id: 219078 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.6 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH, MAILING_LIST_MULTI, SIGNED_OFF_BY, SPF_HELO_NONE, SPF_PASS, USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4A4CBC433E1 for ; Sun, 17 May 2020 19:59:13 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 24F5B20758 for ; Sun, 17 May 2020 19:59:13 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=lunn.ch header.i=@lunn.ch header.b="OXz4cwS9" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726937AbgEQT7M (ORCPT ); Sun, 17 May 2020 15:59:12 -0400 Received: from vps0.lunn.ch ([185.16.172.187]:36288 "EHLO vps0.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726269AbgEQT7K (ORCPT ); Sun, 17 May 2020 15:59:10 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lunn.ch; s=20171124; h=Content-Transfer-Encoding:MIME-Version:References:In-Reply-To: Message-Id:Date:Subject:Cc:To:From:Sender:Reply-To:Content-Type:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=oaDA9ZCIa6yPBOnTpethJou/ZSNdHxhbvXntfhYi+cc=; b=OXz4cwS9wQxE5kPNhd8p8ulhJ/ hIFgTfS5MXeO3gELcV5jabSQuiRoaH4G4RE7N8TdMo3xxBAQi7a/M6myTenChsb+F111mj6ThOxPV 0d3iuX8RSDbKC5Oh6sl9bt3/fxJ3ArdAIrlTm+r18AQaO2JFBYVFsYiqyrkuXaW5Onng=; Received: from andrew by vps0.lunn.ch with local (Exim 4.93) (envelope-from ) id 1jaPR9-002YpN-DE; Sun, 17 May 2020 21:59:03 +0200 From: Andrew Lunn To: David Miller Cc: netdev , Florian Fainelli , Heiner Kallweit , Chris Healy , Michal Kubecek , Andrew Lunn Subject: [PATCH net-next 5/7] net: ethtool: Allow PHY cable test TDR data to configured Date: Sun, 17 May 2020 21:58:49 +0200 Message-Id: <20200517195851.610435-6-andrew@lunn.ch> X-Mailer: git-send-email 2.26.2 In-Reply-To: <20200517195851.610435-1-andrew@lunn.ch> References: <20200517195851.610435-1-andrew@lunn.ch> MIME-Version: 1.0 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Allow the user to configure where on the cable the TDR data should be retrieved, in terms of first and last sample, and the step between samples. Also add the ability to ask for TDR data for just one pair. If this configuration is not provided, it defaults to 0-150m at 1m intervals for all pairs. Signed-off-by: Andrew Lunn --- drivers/net/phy/marvell.c | 49 ++++++++++++++------ drivers/net/phy/phy.c | 6 ++- include/linux/phy.h | 10 +++-- include/uapi/linux/ethtool_netlink.h | 4 ++ net/ethtool/cabletest.c | 67 +++++++++++++++++++++++++++- 5 files changed, 115 insertions(+), 21 deletions(-) diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c index e7994f5b506e..f5d9a932db9a 100644 --- a/drivers/net/phy/marvell.c +++ b/drivers/net/phy/marvell.c @@ -198,6 +198,7 @@ #define MII_VCT5_CTRL_PEEK_HYST_DEFAULT 3 #define MII_VCT5_SAMPLE_POINT_DISTANCE 0x18 +#define MII_VCT5_SAMPLE_POINT_DISTANCE_MAX 511 #define MII_VCT5_TX_PULSE_CTRL 0x1c #define MII_VCT5_TX_PULSE_CTRL_DONT_WAIT_LINK_DOWN BIT(12) #define MII_VCT5_TX_PULSE_CTRL_PULSE_WIDTH_128nS (0x0 << 10) @@ -270,6 +271,10 @@ struct marvell_priv { char *hwmon_name; struct device *hwmon_dev; bool cable_test_tdr; + u32 first; + u32 last; + u32 step; + s8 pair; }; static int marvell_read_page(struct phy_device *phydev) @@ -1783,12 +1788,13 @@ static int marvell_vct5_amplitude(struct phy_device *phydev, int pair) } static int marvell_vct5_amplitude_distance(struct phy_device *phydev, - int meters) + int meters, int pair) { - int mV_pair0, mV_pair1, mV_pair2, mV_pair3; int distance; u16 reg; int err; + int mV; + int i; distance = meters * 1000 / 805; @@ -1812,21 +1818,20 @@ static int marvell_vct5_amplitude_distance(struct phy_device *phydev, if (err) return err; - mV_pair0 = marvell_vct5_amplitude(phydev, 0); - mV_pair1 = marvell_vct5_amplitude(phydev, 1); - mV_pair2 = marvell_vct5_amplitude(phydev, 2); - mV_pair3 = marvell_vct5_amplitude(phydev, 3); + for (i = 0; i < 4; i++) { + if (pair != PHY_PAIR_ALL && i != pair) + continue; - ethnl_cable_test_amplitude(phydev, ETHTOOL_A_CABLE_PAIR_A, mV_pair0); - ethnl_cable_test_amplitude(phydev, ETHTOOL_A_CABLE_PAIR_B, mV_pair1); - ethnl_cable_test_amplitude(phydev, ETHTOOL_A_CABLE_PAIR_C, mV_pair2); - ethnl_cable_test_amplitude(phydev, ETHTOOL_A_CABLE_PAIR_D, mV_pair3); + mV = marvell_vct5_amplitude(phydev, i); + ethnl_cable_test_amplitude(phydev, i, mV); + } return 0; } static int marvell_vct5_amplitude_graph(struct phy_device *phydev) { + struct marvell_priv *priv = phydev->priv; int meters; int err; u16 reg; @@ -1841,8 +1846,11 @@ static int marvell_vct5_amplitude_graph(struct phy_device *phydev) if (err) return err; - for (meters = 0; meters <= 100; meters++) { - err = marvell_vct5_amplitude_distance(phydev, meters); + for (meters = priv->first; + meters <= priv->last; + meters += priv->step) { + err = marvell_vct5_amplitude_distance(phydev, meters, + priv->pair); if (err) return err; } @@ -1916,11 +1924,19 @@ static int marvell_vct7_cable_test_start(struct phy_device *phydev) MII_VCT7_CTRL_CENTIMETERS); } -static int marvell_vct5_cable_test_tdr_start(struct phy_device *phydev) +static int marvell_vct5_cable_test_tdr_start(struct phy_device *phydev, + u32 first, u32 last, u32 step, + s8 pair) { struct marvell_priv *priv = phydev->priv; int ret; + if (first > MII_VCT5_SAMPLE_POINT_DISTANCE_MAX) + return -EINVAL; + + if (last > MII_VCT5_SAMPLE_POINT_DISTANCE_MAX) + return -EINVAL; + /* Disable VCT7 */ ret = phy_write_paged(phydev, MII_MARVELL_VCT7_PAGE, MII_VCT7_CTRL, 0); @@ -1932,11 +1948,16 @@ static int marvell_vct5_cable_test_tdr_start(struct phy_device *phydev) return ret; priv->cable_test_tdr = true; + priv->first = first; + priv->last = last; + priv->step = step; + priv->pair = pair; + ret = ethnl_cable_test_pulse(phydev, 1000); if (ret) return ret; - return ethnl_cable_test_step(phydev, 0, 100, 1); + return ethnl_cable_test_step(phydev, first, last, step); } static int marvell_vct7_distance_to_length(int distance, bool meter) diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c index 1f794c1716cf..92f3f1d1ddaf 100644 --- a/drivers/net/phy/phy.c +++ b/drivers/net/phy/phy.c @@ -553,7 +553,8 @@ int phy_start_cable_test(struct phy_device *phydev, EXPORT_SYMBOL(phy_start_cable_test); int phy_start_cable_test_tdr(struct phy_device *phydev, - struct netlink_ext_ack *extack) + struct netlink_ext_ack *extack, + u32 first, u32 last, u32 step, s8 pair) { struct net_device *dev = phydev->attached_dev; int err = -ENOMEM; @@ -590,7 +591,8 @@ int phy_start_cable_test_tdr(struct phy_device *phydev, phy_link_down(phydev, true); netif_testing_on(dev); - err = phydev->drv->cable_test_tdr_start(phydev); + err = phydev->drv->cable_test_tdr_start(phydev, first, last, step, + pair); if (err) { netif_testing_off(dev); phy_link_up(phydev); diff --git a/include/linux/phy.h b/include/linux/phy.h index c35d45fe74ce..c1a1fa8f33f5 100644 --- a/include/linux/phy.h +++ b/include/linux/phy.h @@ -706,8 +706,10 @@ struct phy_driver { /* Start a cable test */ int (*cable_test_start)(struct phy_device *dev); +#define PHY_PAIR_ALL -1 /* Start a raw TDR cable test */ - int (*cable_test_tdr_start)(struct phy_device *dev); + int (*cable_test_tdr_start)(struct phy_device *dev, + u32 first, u32 last, u32 step, s8 pair); /* Once per second, or on interrupt, request the status of the * test. @@ -1260,7 +1262,8 @@ int phy_reset_after_clk_enable(struct phy_device *phydev); int phy_start_cable_test(struct phy_device *phydev, struct netlink_ext_ack *extack); int phy_start_cable_test_tdr(struct phy_device *phydev, - struct netlink_ext_ack *extack); + struct netlink_ext_ack *extack, + u32 start, u32 last, u32 step, s8 pair); #else static inline int phy_start_cable_test(struct phy_device *phydev, @@ -1271,7 +1274,8 @@ int phy_start_cable_test(struct phy_device *phydev, } static inline int phy_start_cable_test_tdr(struct phy_device *phydev, - struct netlink_ext_ack *extack) + struct netlink_ext_ack *extack, + u32 start, u32 last, u32 step, s8 pair) { NL_SET_ERR_MSG(extack, "Kernel not compiled with PHYLIB support"); return -EOPNOTSUPP; diff --git a/include/uapi/linux/ethtool_netlink.h b/include/uapi/linux/ethtool_netlink.h index 4f223edcefda..e41b38567cbe 100644 --- a/include/uapi/linux/ethtool_netlink.h +++ b/include/uapi/linux/ethtool_netlink.h @@ -483,6 +483,10 @@ enum { enum { ETHTOOL_A_CABLE_TEST_TDR_UNSPEC, ETHTOOL_A_CABLE_TEST_TDR_HEADER, /* nest - _A_HEADER_* */ + ETHTOOL_A_CABLE_TEST_TDR_FIRST, /* u32 */ + ETHTOOL_A_CABLE_TEST_TDR_LAST, /* u32 */ + ETHTOOL_A_CABLE_TEST_TDR_STEP, /* u32 */ + ETHTOOL_A_CABLE_TEST_TDR_PAIR, /* u8 */ /* add new constants above here */ __ETHTOOL_A_CABLE_TEST_TDR_CNT, diff --git a/net/ethtool/cabletest.c b/net/ethtool/cabletest.c index c02575e26336..e9b1c811ed41 100644 --- a/net/ethtool/cabletest.c +++ b/net/ethtool/cabletest.c @@ -5,7 +5,11 @@ #include "netlink.h" #include "common.h" -/* CABLE_TEST_ACT */ +/* 802.3 standard allows 100 meters for BaseT cables. However longer + * cables might work, depending on the quality of the cables and the + * PHY. So allow testing for up to 150 meters. + */ +#define MAX_CABLE_LENGTH 150 static const struct nla_policy cable_test_act_policy[ETHTOOL_A_CABLE_TEST_MAX + 1] = { @@ -203,18 +207,30 @@ int ethnl_cable_test_fault_length(struct phy_device *phydev, u8 pair, u32 cm) } EXPORT_SYMBOL_GPL(ethnl_cable_test_fault_length); +struct cable_test_tdr_req_info { + struct ethnl_req_info base; +}; + static const struct nla_policy cable_test_tdr_act_policy[ETHTOOL_A_CABLE_TEST_TDR_MAX + 1] = { [ETHTOOL_A_CABLE_TEST_TDR_UNSPEC] = { .type = NLA_REJECT }, [ETHTOOL_A_CABLE_TEST_TDR_HEADER] = { .type = NLA_NESTED }, + [ETHTOOL_A_CABLE_TEST_TDR_FIRST] = { .type = NLA_U32 }, + [ETHTOOL_A_CABLE_TEST_TDR_LAST] = { .type = NLA_U32 }, + [ETHTOOL_A_CABLE_TEST_TDR_STEP] = { .type = NLA_U32 }, + [ETHTOOL_A_CABLE_TEST_TDR_PAIR] = { .type = NLA_U8 }, }; +/* CABLE_TEST_TDR_ACT */ + int ethnl_act_cable_test_tdr(struct sk_buff *skb, struct genl_info *info) { struct nlattr *tb[ETHTOOL_A_CABLE_TEST_TDR_MAX + 1]; struct ethnl_req_info req_info = {}; struct net_device *dev; + u32 first, last, step; int ret; + u8 pair; ret = nlmsg_parse(info->nlhdr, GENL_HDRLEN, tb, ETHTOOL_A_CABLE_TEST_TDR_MAX, @@ -235,12 +251,59 @@ int ethnl_act_cable_test_tdr(struct sk_buff *skb, struct genl_info *info) goto out_dev_put; } + if (tb[ETHTOOL_A_CABLE_TEST_TDR_FIRST]) + first = nla_get_u32(tb[ETHTOOL_A_CABLE_TEST_TDR_FIRST]); + else + first = 1; + if (tb[ETHTOOL_A_CABLE_TEST_TDR_LAST]) + last = nla_get_u32(tb[ETHTOOL_A_CABLE_TEST_TDR_LAST]); + else + last = MAX_CABLE_LENGTH; + + if (tb[ETHTOOL_A_CABLE_TEST_TDR_STEP]) + step = nla_get_u32(tb[ETHTOOL_A_CABLE_TEST_TDR_STEP]); + else + step = 1; + + if (tb[ETHTOOL_A_CABLE_TEST_TDR_PAIR]) { + pair = nla_get_u8(tb[ETHTOOL_A_CABLE_TEST_TDR_PAIR]); + if (pair < ETHTOOL_A_CABLE_PAIR_A || + pair > ETHTOOL_A_CABLE_PAIR_D) { + NL_SET_ERR_MSG(info->extack, + "invalid pair parameter"); + return -EINVAL; + } + } else { + pair = PHY_PAIR_ALL; + } + + if (first > MAX_CABLE_LENGTH) { + NL_SET_ERR_MSG(info->extack, "invalid first parameter"); + return -EINVAL; + } + + if (last > MAX_CABLE_LENGTH) { + NL_SET_ERR_MSG(info->extack, "invalid last parameter"); + return -EINVAL; + } + + if (first > last) { + NL_SET_ERR_MSG(info->extack, "invalid first/last parameter"); + return -EINVAL; + } + + if (!step) { + NL_SET_ERR_MSG(info->extack, "invalid step parameter"); + return -EINVAL; + } + rtnl_lock(); ret = ethnl_ops_begin(dev); if (ret < 0) goto out_rtnl; - ret = phy_start_cable_test_tdr(dev->phydev, info->extack); + ret = phy_start_cable_test_tdr(dev->phydev, info->extack, + first, last, step, pair); ethnl_ops_complete(dev);