netdev - Re: [PATCH] [v9] net: emac: emac gigabit ethernet controller driver

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: <20160831211621.GB11775@lunn.ch>
Date:   Wed, 31 Aug 2016 23:16:21 +0200
From:   Andrew Lunn <andrew@...n.ch>
To:     Timur Tabi <timur@...eaurora.org>
Cc:     Rob Herring <robh@...nel.org>, netdev <netdev@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        linux-arm-msm <linux-arm-msm@...r.kernel.org>,
        Sagar Dharia <sdharia@...eaurora.org>,
        Shanker Donthineni <shankerd@...eaurora.org>,
        Vikram Sethi <vikrams@...eaurora.org>,
        Christopher Covington <cov@...eaurora.org>,
        Gilad Avidov <gavidov@...eaurora.org>,
        Bjorn Andersson <bjorn.andersson@...aro.org>,
        Mark Langsdorf <mlangsdo@...hat.com>,
        "jcm@...hat.com" <jcm@...hat.com>,
        Andy Gross <agross@...eaurora.org>,
        David Miller <davem@...emloft.net>,
        Florian Fainelli <f.fainelli@...il.com>,
        Lino Sanfilippo <LinoSanfilippo@....de>
Subject: Re: [PATCH] [v9] net: emac: emac gigabit ethernet controller driver

> >You can just as easily find the child node called ethernet-phy.
> 
> As Andrew pointed out, using phy-handle allows me to place the phy
> node anywhere.
> 
> I've already made changes to this design, and every change has
> raised objections.  I don't see anything wrong with phy-handle.  A
> lot of drivers use it.

Agreed. You should do the same as what all other driver do.

	Andrew