netdev - Re: [PATCH v3 0/8] arm64: rockchip: Initial GeekBox enablement

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: <56DD8176.2080603@st.com>
Date:	Mon, 7 Mar 2016 14:26:14 +0100
From:	Giuseppe CAVALLARO <peppe.cavallaro@...com>
To:	Andreas Färber <afaerber@...e.de>,
	Heiko Stübner <heiko@...ech.de>
CC:	<linux-rockchip@...ts.infradead.org>, <netdev@...r.kernel.org>,
	<devicetree@...r.kernel.org>,
	LAKML <linux-arm-kernel@...ts.infradead.org>,
	Gabriel Fernandez <gabriel.fernandez@...aro.org>
Subject: Re: [PATCH v3 0/8] arm64: rockchip: Initial GeekBox enablement

Hello Andreas, Heiko,

On 3/7/2016 1:35 PM, Andreas Färber wrote:
> Hi Heiko,
>
> Am 07.03.2016 um 13:24 schrieb Heiko Stübner:
>> Am Montag, 7. März 2016, 13:17:54 schrieb Andreas Färber:
>>> Am 06.03.2016 um 20:53 schrieb Andreas Färber:
>>>> On next-20160304 the GMAC seems to have regressed, it no longer finds the
>>>> PHY:
>>>>
>>>> libphy: PHY stmmac-0:ffffffff not found

snps,phy-addr is not correct. Can you check it?

In the stmmac_mdio if this is not passed as parameter the bus should
be scanned and the first IDs found should select the right address.
I suppose in your box there is a real transceiver, so maybe the patch
from Gabriel is not actually necessary.

>>>> eth0: Could not attach to PHY
>>>> stmmac_open: Cannot attach to PHY (error: -19)
>>>
>>> Still with us in next-20160307. CC'ing stmmac maintainers.

peppe

>>
>> do you remember from what revision you rebased away from, when it was still
>> working (as a known-good state)?
>
> Hm, next-20160111 possibly... leaving all the 5 days and 4 days ago
> changes as candidates ("stmmac: share reset function between dwmac100
> and dwmac1000" .. "stmmac: Fix 'eth0: No PHY found' regression").
>
> Regards,
> Andreas
>