From mboxrd@z Thu Jan 1 00:00:00 1970 From: ezequiel.garcia@free-electrons.com (Ezequiel Garcia) Date: Fri, 8 Aug 2014 15:21:56 -0300 Subject: [PATCH 2/6] ARM: mvebu: Add proper pin muxing on Globalscale Mirabox board In-Reply-To: <20140808171208.GC26751@lunn.ch> References: <1407511136-26477-1-git-send-email-ezequiel.garcia@free-electrons.com> <1407511136-26477-3-git-send-email-ezequiel.garcia@free-electrons.com> <20140808171208.GC26751@lunn.ch> Message-ID: <20140808182156.GA16889@arch.cereza> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 08 Aug 07:12 PM, Andrew Lunn wrote: > On Fri, Aug 08, 2014 at 12:18:52PM -0300, Ezequiel Garcia wrote: > > This commit adds the required pin muxing for the network interfaces and > > the MDIO interface to be properly initialized. For instance, this makes > > it possible for a bootloader to initialize and access the network interfaces. > > > > Signed-off-by: Ezequiel Garcia > > --- > > arch/arm/boot/dts/armada-370-mirabox.dts | 6 ++++++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/arch/arm/boot/dts/armada-370-mirabox.dts b/arch/arm/boot/dts/armada-370-mirabox.dts > > index 097df7d8..c745256 100644 > > --- a/arch/arm/boot/dts/armada-370-mirabox.dts > > +++ b/arch/arm/boot/dts/armada-370-mirabox.dts > > @@ -91,6 +91,8 @@ > > }; > > > > mdio { > > + pinctrl-0 = <&mdio_pins>; > > + pinctrl-names = "default"; > > phy0: ethernet-phy at 0 { > > reg = <0>; > > }; > > @@ -100,11 +102,15 @@ > > }; > > }; > > ethernet at 70000 { > > + pinctrl-0 = <&ge0_pins>; > > + pinctrl-names = "default"; > > Hi Ezequiel > > Could you put these one level higher in the DT hierarchy? i.e. once in > armada-370.dtsi and not in every board .dts file? > Are you sure that's the correct approach? As explained on the cover letter, these MAC MPP mux are not required in the SGMII case. Moreover, this is not the only way to wire the MAC0 and MAC1 controllers, so I even considered pushing down the pinmux to the dts files! The devicetree location of the pinmux is something that still confuses me, so I'm more than open to directions here. -- Ezequiel Garc?a, Free Electrons Embedded Linux, Kernel and Android Engineering http://free-electrons.com