From: Russell King - ARM Linux admin <linux@armlinux.org.uk>
To: Andrew Lunn <andrew@lunn.ch>
Cc: Oleksij Rempel <o.rempel@pengutronix.de>,
Florian Fainelli <f.fainelli@gmail.com>,
Heiner Kallweit <hkallweit1@gmail.com>,
"David S. Miller" <davem@davemloft.net>,
kernel@pengutronix.de, linux-kernel@vger.kernel.org,
netdev@vger.kernel.org
Subject: Re: [PATCH v1] net: phy: at803x: add cable test support
Date: Wed, 13 May 2020 14:50:04 +0100 [thread overview]
Message-ID: <20200513135004.GK1551@shell.armlinux.org.uk> (raw)
In-Reply-To: <20200513133209.GC499265@lunn.ch>
On Wed, May 13, 2020 at 03:32:09PM +0200, Andrew Lunn wrote:
> On Wed, May 13, 2020 at 02:06:48PM +0200, Oleksij Rempel wrote:
> > The cable test seems to be support by all of currently support Atherso
> > PHYs, so add support for all of them. This patch was tested only on
> > AR9331 PHY with following results:
> > - No cable is detected as short
> > - A 15m long cable connected only on one side is detected as 9m open.
>
> That sounds wrong. What about a shorted 15m cable? Is it also 9m? Do
> you have any other long cables you can test with? Is it always 1/2 the
> cable length?
I had similar inaccuracies with my recent faulty cable when testing
with a Marvell PHY as I mentioned.
"Using the VCT in the Marvell PHY points to it being pair 3, at a
distance of 0x190 or 0x50 depending on which way round the cable is
connected. That's in cm. The cable isn't 480cm long, it's 278cm
long, and the problem is up by one of the connectors."
0x190 = 400cm, 0x50 = 80cm.
Given that the issue was at one of the connectors on the cable, and
I tried VCT with it plugged into the same port, you can't even say
"well, if we define the start of the cable at 80cm, then that works
for the cable connected the other way around" - it gets us closer
but it's still about 30cm wrong.
It doesn't even work if you think maybe the figures have forgotten
to take into account the fact that the TDR pulse has to go out and
then return (so travel twice the distance, so maybe the figures are
doubled.)
So, it seems we have more than one PHY that produces only wildly
inaccurate guesses at the distance to the fault.
I'd say this technology is a "it would be nice if we could" but the
results can not be relied upon. It may be grounded in hard physics,
but there's clearly something causing incorrect results.
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 10.2Mbps down 587kbps up
next prev parent reply other threads:[~2020-05-13 13:50 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-13 12:06 [PATCH v1] net: phy: at803x: add cable test support Oleksij Rempel
2020-05-13 13:32 ` Andrew Lunn
2020-05-13 13:50 ` Russell King - ARM Linux admin [this message]
2020-05-13 15:23 ` Florian Fainelli
2020-05-13 15:45 ` Oleksij Rempel
2020-05-13 15:49 ` Andrew Lunn
2020-05-13 16:00 ` Oleksij Rempel
2020-05-13 16:23 ` Andrew Lunn
2020-05-13 16:26 ` Michael Walle
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=20200513135004.GK1551@shell.armlinux.org.uk \
--to=linux@armlinux.org.uk \
--cc=andrew@lunn.ch \
--cc=davem@davemloft.net \
--cc=f.fainelli@gmail.com \
--cc=hkallweit1@gmail.com \
--cc=kernel@pengutronix.de \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=o.rempel@pengutronix.de \
/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).