netdev - Re: [PATCH net-next] net: dsa: Simplify dsa_slave_phy

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 for Android: free password hash cracker in your pocket

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

Message-ID: <5a0ff841-8431-97ca-36bf-3a5d67a52ab0@prevas.dk>
Date:   Fri, 27 Oct 2017 07:02:47 +0200
From:   Martin Hundebøll <mnhu@...vas.dk>
To:     Florian Fainelli <f.fainelli@...il.com>, <netdev@...r.kernel.org>
CC:     <davem@...emloft.net>, <andrew@...n.ch>,
        <vivien.didelot@...oirfairelinux.com>
Subject: Re: [PATCH net-next] net: dsa: Simplify dsa_slave_phy_setup()

On 2017-10-26 18:45, Florian Fainelli wrote:
> On 10/26/2017 04:07 AM, Martin Hundebøll wrote:
>> On 2017-10-26 02:32, Florian Fainelli wrote:
>>> Remove the code that tried to identify if a PHY designated by Device
>>> Tree required diversion through the DSA-created MDIO bus. This was
>>> created mainly for the bcm_sf2.c driver back when it did not have its
>>> own MDIO bus driver, which it now has since 461cd1b03e32 ("net: dsa:
>>> bcm_sf2: Register our slave MDIO bus").
>>>
>>> Signed-off-by: Florian Fainelli<f.fainelli@...il.com>
>> Tested-by: Martin Hundebøll<mnhu@...vas.dk>
> Thanks Martin, does that correctly fix the problem you reported a week
> ago on 639X?

It does indeed. Thanks for the work on this.

// Martin