From mboxrd@z Thu Jan 1 00:00:00 1970 From: gregory.clement@free-electrons.com (Gregory CLEMENT) Date: Thu, 30 Mar 2017 17:34:21 +0200 Subject: [PATCH] ARM: dts: clearfog: keep dts alphabetically ordered In-Reply-To: (Russell King's message of "Sat, 25 Mar 2017 12:53:10 +0000") References: Message-ID: <871steaide.fsf@free-electrons.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Russell, On sam., mars 25 2017, Russell King wrote: > Keep the clearfog DTS file ordered alphabetically - Florian placed the > MDIO entry after pinctrl, which mis-orders the file. > > Signed-off-by: Russell King Applied on mvebu/dt with Reviewed-by flag from Andrew Lunn Thanks, Gregory > --- > arch/arm/boot/dts/armada-388-clearfog.dts | 38 +++++++++++++++---------------- > 1 file changed, 19 insertions(+), 19 deletions(-) > > diff --git a/arch/arm/boot/dts/armada-388-clearfog.dts b/arch/arm/boot/dts/armada-388-clearfog.dts > index b9ad1641bcf3..e74904bcd0a0 100644 > --- a/arch/arm/boot/dts/armada-388-clearfog.dts > +++ b/arch/arm/boot/dts/armada-388-clearfog.dts > @@ -183,25 +183,6 @@ > }; > }; > > -&pinctrl { > - clearfog_dsa0_clk_pins: clearfog-dsa0-clk-pins { > - marvell,pins = "mpp46"; > - marvell,function = "ref"; > - }; > - clearfog_dsa0_pins: clearfog-dsa0-pins { > - marvell,pins = "mpp23", "mpp41"; > - marvell,function = "gpio"; > - }; > - clearfog_spi1_cs_pins: spi1-cs-pins { > - marvell,pins = "mpp55"; > - marvell,function = "spi1"; > - }; > - rear_button_pins: rear-button-pins { > - marvell,pins = "mpp34"; > - marvell,function = "gpio"; > - }; > -}; > - > &mdio { > status = "okay"; > > @@ -261,6 +242,25 @@ > }; > }; > > +&pinctrl { > + clearfog_dsa0_clk_pins: clearfog-dsa0-clk-pins { > + marvell,pins = "mpp46"; > + marvell,function = "ref"; > + }; > + clearfog_dsa0_pins: clearfog-dsa0-pins { > + marvell,pins = "mpp23", "mpp41"; > + marvell,function = "gpio"; > + }; > + clearfog_spi1_cs_pins: spi1-cs-pins { > + marvell,pins = "mpp55"; > + marvell,function = "spi1"; > + }; > + rear_button_pins: rear-button-pins { > + marvell,pins = "mpp34"; > + marvell,function = "gpio"; > + }; > +}; > + > &spi1 { > /* > * Add SPI CS pins for clearfog: > -- > 2.7.4 > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gregory CLEMENT Subject: Re: [PATCH] ARM: dts: clearfog: keep dts alphabetically ordered Date: Thu, 30 Mar 2017 17:34:21 +0200 Message-ID: <871steaide.fsf@free-electrons.com> References: Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: (Russell King's message of "Sat, 25 Mar 2017 12:53:10 +0000") Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Russell King Cc: Jason Cooper , Andrew Lunn , Sebastian Hesselbarth , Rob Herring , Mark Rutland , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org Hi Russell, On sam., mars 25 2017, Russell King wrote: > Keep the clearfog DTS file ordered alphabetically - Florian placed the > MDIO entry after pinctrl, which mis-orders the file. > > Signed-off-by: Russell King Applied on mvebu/dt with Reviewed-by flag from Andrew Lunn Thanks, Gregory > --- > arch/arm/boot/dts/armada-388-clearfog.dts | 38 +++++++++++++++---------------- > 1 file changed, 19 insertions(+), 19 deletions(-) > > diff --git a/arch/arm/boot/dts/armada-388-clearfog.dts b/arch/arm/boot/dts/armada-388-clearfog.dts > index b9ad1641bcf3..e74904bcd0a0 100644 > --- a/arch/arm/boot/dts/armada-388-clearfog.dts > +++ b/arch/arm/boot/dts/armada-388-clearfog.dts > @@ -183,25 +183,6 @@ > }; > }; > > -&pinctrl { > - clearfog_dsa0_clk_pins: clearfog-dsa0-clk-pins { > - marvell,pins = "mpp46"; > - marvell,function = "ref"; > - }; > - clearfog_dsa0_pins: clearfog-dsa0-pins { > - marvell,pins = "mpp23", "mpp41"; > - marvell,function = "gpio"; > - }; > - clearfog_spi1_cs_pins: spi1-cs-pins { > - marvell,pins = "mpp55"; > - marvell,function = "spi1"; > - }; > - rear_button_pins: rear-button-pins { > - marvell,pins = "mpp34"; > - marvell,function = "gpio"; > - }; > -}; > - > &mdio { > status = "okay"; > > @@ -261,6 +242,25 @@ > }; > }; > > +&pinctrl { > + clearfog_dsa0_clk_pins: clearfog-dsa0-clk-pins { > + marvell,pins = "mpp46"; > + marvell,function = "ref"; > + }; > + clearfog_dsa0_pins: clearfog-dsa0-pins { > + marvell,pins = "mpp23", "mpp41"; > + marvell,function = "gpio"; > + }; > + clearfog_spi1_cs_pins: spi1-cs-pins { > + marvell,pins = "mpp55"; > + marvell,function = "spi1"; > + }; > + rear_button_pins: rear-button-pins { > + marvell,pins = "mpp34"; > + marvell,function = "gpio"; > + }; > +}; > + > &spi1 { > /* > * Add SPI CS pins for clearfog: > -- > 2.7.4 > -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html