All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: clearfog: keep dts alphabetically ordered
@ 2017-03-25 12:53 ` Russell King
  0 siblings, 0 replies; 6+ messages in thread
From: Russell King @ 2017-03-25 12:53 UTC (permalink / raw)
  To: Gregory Clement
  Cc: Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Rob Herring,
	Mark Rutland, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA

Keep the clearfog DTS file ordered alphabetically - Florian placed the
MDIO entry after pinctrl, which mis-orders the file.

Signed-off-by: Russell King <rmk+kernel-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org>
---
 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

--
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

^ permalink raw reply related	[flat|nested] 6+ messages in thread
* [PATCH] ARM: dts: clearfog: keep dts alphabetically ordered
@ 2017-03-25 12:53 ` Russell King
  0 siblings, 0 replies; 6+ messages in thread
From: Russell King @ 2017-03-25 12:53 UTC (permalink / raw)
  To: linux-arm-kernel

Keep the clearfog DTS file ordered alphabetically - Florian placed the
MDIO entry after pinctrl, which mis-orders the file.

Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
---
 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

^ permalink raw reply related	[flat|nested] 6+ messages in thread
* Re: [PATCH] ARM: dts: clearfog: keep dts alphabetically ordered
  2017-03-25 12:53 ` Russell King
@ 2017-03-26 14:02   ` Andrew Lunn
  -1 siblings, 0 replies; 6+ messages in thread
From: Andrew Lunn @ 2017-03-26 14:02 UTC (permalink / raw)
  To: Russell King
  Cc: Mark Rutland, devicetree, Jason Cooper, Rob Herring,
	Gregory Clement, linux-arm-kernel, Sebastian Hesselbarth

On Sat, Mar 25, 2017 at 12:53:10PM +0000, 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 <rmk+kernel@armlinux.org.uk>

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew

^ permalink raw reply	[flat|nested] 6+ messages in thread
* [PATCH] ARM: dts: clearfog: keep dts alphabetically ordered
@ 2017-03-26 14:02   ` Andrew Lunn
  0 siblings, 0 replies; 6+ messages in thread
From: Andrew Lunn @ 2017-03-26 14:02 UTC (permalink / raw)
  To: linux-arm-kernel

On Sat, Mar 25, 2017 at 12:53:10PM +0000, 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 <rmk+kernel@armlinux.org.uk>

Reviewed-by: Andrew Lunn <andrew@lunn.ch>

    Andrew

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [PATCH] ARM: dts: clearfog: keep dts alphabetically ordered
  2017-03-25 12:53 ` Russell King
@ 2017-03-30 15:34     ` Gregory CLEMENT
  -1 siblings, 0 replies; 6+ messages in thread
From: Gregory CLEMENT @ 2017-03-30 15:34 UTC (permalink / raw)
  To: Russell King
  Cc: Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Rob Herring,
	Mark Rutland, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA

Hi Russell,
 
 On sam., mars 25 2017, Russell King <rmk+kernel-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org> 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 <rmk+kernel-I+IVW8TIWO2tmTQ+vhA3Yw@public.gmane.org>

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

^ permalink raw reply	[flat|nested] 6+ messages in thread
* [PATCH] ARM: dts: clearfog: keep dts alphabetically ordered
@ 2017-03-30 15:34     ` Gregory CLEMENT
  0 siblings, 0 replies; 6+ messages in thread
From: Gregory CLEMENT @ 2017-03-30 15:34 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Russell,
 
 On sam., mars 25 2017, Russell King <rmk+kernel@armlinux.org.uk> 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 <rmk+kernel@armlinux.org.uk>

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

^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-03-30 15:34 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-25 12:53 [PATCH] ARM: dts: clearfog: keep dts alphabetically ordered Russell King
2017-03-25 12:53 ` Russell King
2017-03-26 14:02 ` Andrew Lunn
2017-03-26 14:02   ` Andrew Lunn
     [not found] ` <E1crlBu-0002Gp-V4-eh5Bv4kxaXIk46pC+1QYvQNdhmdF6hFW@public.gmane.org>
2017-03-30 15:34   ` Gregory CLEMENT
2017-03-30 15:34     ` Gregory CLEMENT

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.