From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH 1/4] dt-bindings: net: ravb: Document optional reset-gpios property Date: Thu, 5 Oct 2017 18:24:51 -0500 Message-ID: <20171005232451.loogqlkmar6pdrdd@rob-hp-laptop> References: <1506614014-4398-1-git-send-email-geert+renesas@glider.be> <1506614014-4398-2-git-send-email-geert+renesas@glider.be> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Geert Uytterhoeven , "David S . Miller" , Simon Horman , Magnus Damm , Andrew Lunn , Florian Fainelli , Niklas =?iso-8859-1?Q?S=F6derlund?= , netdev@vger.kernel.org, linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org To: Sergei Shtylyov Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-renesas-soc-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Thu, Sep 28, 2017 at 11:07:38PM +0300, Sergei Shtylyov wrote: > Hello! > > On 09/28/2017 06:53 PM, Geert Uytterhoeven wrote: > > > The optional "reset-gpios" property (part of the generic MDIO bus > > properties) lets us describe the GPIO used for resetting the Ethernet > > PHY. > > > > Signed-off-by: Geert Uytterhoeven > > --- > > Documentation/devicetree/bindings/net/renesas,ravb.txt | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/net/renesas,ravb.txt b/Documentation/devicetree/bindings/net/renesas,ravb.txt > > index c902261893b913f5..4a6ec1ba32d0bf16 100644 > > --- a/Documentation/devicetree/bindings/net/renesas,ravb.txt > > +++ b/Documentation/devicetree/bindings/net/renesas,ravb.txt > > @@ -52,6 +52,7 @@ Optional properties: > > AVB_LINK signal. > > - renesas,ether-link-active-low: boolean, specify when the AVB_LINK signal is > > active-low instead of normal active-high. > > +- reset-gpios: see mdio.txt in the same directory. > > Sigh, I can only repeat that was a terrible prop name choice -- when > applied to a MAC node... what reset does it mean? MAC? Agreed. This should be in the phy node. Or MDIO at least.