From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Lunn Subject: Re: [PATCH v2 3/3] arm64: dts: marvell: Add ethernet switch definition for the ESPRESSObin Date: Tue, 20 Dec 2016 17:17:14 +0100 Message-ID: <20161220161714.GD30952@lunn.ch> References: <20161220085138.3998-1-romain.perier@free-electrons.com> <20161220085138.3998-4-romain.perier@free-electrons.com> <20161220154234.GC30952@lunn.ch> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org To: Romain Perier Cc: Vivien Didelot , Florian Fainelli , Jason Cooper , Sebastian Hesselbarth , Gregory Clement , netdev@vger.kernel.org, devicetree@vger.kernel.org, Rob Herring , Ian Campbell , Pawel Moll , Mark Rutland , Kumar Gala , linux-arm-kernel@lists.infradead.org, Thomas Petazzoni , Nadav Haklai List-Id: devicetree@vger.kernel.org > >>+ mdio { > >>+ #address-cells = <1>; > >>+ #size-cells = <0>; > >>+ reg = <1>; > > > >what is this reg value for? > > > > Andrew > > > > It was required to avoid a warning thrown by the mdio subsystem Do you remember what the warning was? This seems odd to me. I don't see why a reg is needed here. Thanks Andrew