linux-kernel - Re: [BUG] net: phy: genphy_loopback: add link speed configuration

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Ybm7jVwNfj01b7S4@lunn.ch>
Date:   Wed, 15 Dec 2021 10:55:25 +0100
From:   Andrew Lunn <andrew@...n.ch>
To:     "Ismail, Mohammad Athari" <mohammad.athari.ismail@...el.com>
Cc:     Oleksij Rempel <o.rempel@...gutronix.de>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "Voon, Weifeng" <weifeng.voon@...el.com>,
        "Wong, Vee Khee" <vee.khee.wong@...el.com>
Subject: Re: [BUG] net: phy: genphy_loopback: add link speed configuration

> > -----Original Message-----
> > From: Andrew Lunn <andrew@...n.ch>
> > Sent: Wednesday, December 15, 2021 5:23 PM
> > To: Ismail, Mohammad Athari <mohammad.athari.ismail@...el.com>
> > Cc: Oleksij Rempel <o.rempel@...gutronix.de>; netdev@...r.kernel.org;
> > linux-kernel@...r.kernel.org; Voon, Weifeng <weifeng.voon@...el.com>;
> > Wong, Vee Khee <vee.khee.wong@...el.com>
> > Subject: Re: [BUG] net: phy: genphy_loopback: add link speed configuration
> > 
> > > Thanks for the suggestion. The proposed solution also doesn't work. Still
> > get -110 error.
> > 
> > Please can you trace where this -110 comes from. Am i looking at the wrong
> > poll call?
> 
> I did read the ret value from genphy_soft_reset() and phy_read_poll_timeout().
> The -110 came from phy_read_poll_timeout().

O.K.

Does the PHY actually do loopback, despite the -110?

I'm wondering if we should ignore the return value from
phy_read_poll_timeout().

	Andrew