linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: lager: use generic pinctrl properties
@ 2016-03-16  1:03 Simon Horman
  2016-03-16  8:56 ` Geert Uytterhoeven
  2016-03-17  6:20 ` Laurent Pinchart
  0 siblings, 2 replies; 4+ messages in thread
From: Simon Horman @ 2016-03-16  1:03 UTC (permalink / raw)
  To: linux-arm-kernel

Since 16ccaf5bb5a5 ("pinctrl: sh-pfc: Accept standard function, pins and
groups properties") renesas pfc drivers accept generic "function", "pins"
and "groups" properties.

This patch updates the lager device tree to use the generic
rather than renesas-specific properties.

Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
Based on renesas-devel-20160315-v4.5
---
 arch/arm/boot/dts/r8a7790-lager.dts | 92 ++++++++++++++++++-------------------
 1 file changed, 46 insertions(+), 46 deletions(-)

diff --git a/arch/arm/boot/dts/r8a7790-lager.dts b/arch/arm/boot/dts/r8a7790-lager.dts
index aa6ca92a9485..a5828721ab7c 100644
--- a/arch/arm/boot/dts/r8a7790-lager.dts
+++ b/arch/arm/boot/dts/r8a7790-lager.dts
@@ -314,119 +314,119 @@
 	pinctrl-names = "default";
 
 	du_pins: du {
-		renesas,groups = "du_rgb666", "du_sync_1", "du_clk_out_0";
-		renesas,function = "du";
+		groups = "du_rgb666", "du_sync_1", "du_clk_out_0";
+		function = "du";
 	};
 
 	scif0_pins: serial0 {
-		renesas,groups = "scif0_data";
-		renesas,function = "scif0";
+		groups = "scif0_data";
+		function = "scif0";
 	};
 
 	scif_clk_pins: scif_clk {
-		renesas,groups = "scif_clk";
-		renesas,function = "scif_clk";
+		groups = "scif_clk";
+		function = "scif_clk";
 	};
 
 	ether_pins: ether {
-		renesas,groups = "eth_link", "eth_mdio", "eth_rmii";
-		renesas,function = "eth";
+		groups = "eth_link", "eth_mdio", "eth_rmii";
+		function = "eth";
 	};
 
 	phy1_pins: phy1 {
-		renesas,groups = "intc_irq0";
-		renesas,function = "intc";
+		groups = "intc_irq0";
+		function = "intc";
 	};
 
 	scifa1_pins: serial1 {
-		renesas,groups = "scifa1_data";
-		renesas,function = "scifa1";
+		groups = "scifa1_data";
+		function = "scifa1";
 	};
 
 	sdhi0_pins: sd0 {
-		renesas,groups = "sdhi0_data4", "sdhi0_ctrl";
-		renesas,function = "sdhi0";
+		groups = "sdhi0_data4", "sdhi0_ctrl";
+		function = "sdhi0";
 	};
 
 	sdhi2_pins: sd2 {
-		renesas,groups = "sdhi2_data4", "sdhi2_ctrl";
-		renesas,function = "sdhi2";
+		groups = "sdhi2_data4", "sdhi2_ctrl";
+		function = "sdhi2";
 	};
 
 	mmc1_pins: mmc1 {
-		renesas,groups = "mmc1_data8", "mmc1_ctrl";
-		renesas,function = "mmc1";
+		groups = "mmc1_data8", "mmc1_ctrl";
+		function = "mmc1";
 	};
 
 	qspi_pins: spi0 {
-		renesas,groups = "qspi_ctrl", "qspi_data4";
-		renesas,function = "qspi";
+		groups = "qspi_ctrl", "qspi_data4";
+		function = "qspi";
 	};
 
 	msiof1_pins: spi2 {
-		renesas,groups = "msiof1_clk", "msiof1_sync", "msiof1_rx",
+		groups = "msiof1_clk", "msiof1_sync", "msiof1_rx",
 				 "msiof1_tx";
-		renesas,function = "msiof1";
+		function = "msiof1";
 	};
 
 	i2c0_pins: i2c0 {
-		renesas,groups = "i2c0";
-		renesas,function = "i2c0";
+		groups = "i2c0";
+		function = "i2c0";
 	};
 
 	iic0_pins: iic0 {
-		renesas,groups = "iic0";
-		renesas,function = "iic0";
+		groups = "iic0";
+		function = "iic0";
 	};
 
 	iic1_pins: iic1 {
-		renesas,groups = "iic1";
-		renesas,function = "iic1";
+		groups = "iic1";
+		function = "iic1";
 	};
 
 	iic2_pins: iic2 {
-		renesas,groups = "iic2";
-		renesas,function = "iic2";
+		groups = "iic2";
+		function = "iic2";
 	};
 
 	iic3_pins: iic3 {
-		renesas,groups = "iic3";
-		renesas,function = "iic3";
+		groups = "iic3";
+		function = "iic3";
 	};
 
 	hsusb_pins: hsusb {
-		renesas,groups = "usb0_ovc_vbus";
-		renesas,function = "usb0";
+		groups = "usb0_ovc_vbus";
+		function = "usb0";
 	};
 
 	usb0_pins: usb0 {
-		renesas,groups = "usb0";
-		renesas,function = "usb0";
+		groups = "usb0";
+		function = "usb0";
 	};
 
 	usb1_pins: usb1 {
-		renesas,groups = "usb1";
-		renesas,function = "usb1";
+		groups = "usb1";
+		function = "usb1";
 	};
 
 	usb2_pins: usb2 {
-		renesas,groups = "usb2";
-		renesas,function = "usb2";
+		groups = "usb2";
+		function = "usb2";
 	};
 
 	vin1_pins: vin {
-		renesas,groups = "vin1_data8", "vin1_clk";
-		renesas,function = "vin1";
+		groups = "vin1_data8", "vin1_clk";
+		function = "vin1";
 	};
 
 	sound_pins: sound {
-		renesas,groups = "ssi0129_ctrl", "ssi0_data", "ssi1_data";
-		renesas,function = "ssi";
+		groups = "ssi0129_ctrl", "ssi0_data", "ssi1_data";
+		function = "ssi";
 	};
 
 	sound_clk_pins: sound_clk {
-		renesas,groups = "audio_clk_a";
-		renesas,function = "audio_clk";
+		groups = "audio_clk_a";
+		function = "audio_clk";
 	};
 };
 
-- 
2.1.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH] ARM: dts: lager: use generic pinctrl properties
  2016-03-16  1:03 [PATCH] ARM: dts: lager: use generic pinctrl properties Simon Horman
@ 2016-03-16  8:56 ` Geert Uytterhoeven
  2016-03-17  2:14   ` Simon Horman
  2016-03-17  6:20 ` Laurent Pinchart
  1 sibling, 1 reply; 4+ messages in thread
From: Geert Uytterhoeven @ 2016-03-16  8:56 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Mar 16, 2016 at 2:03 AM, Simon Horman
<horms+renesas@verge.net.au> wrote:
> Since 16ccaf5bb5a5 ("pinctrl: sh-pfc: Accept standard function, pins and
> groups properties") renesas pfc drivers accept generic "function", "pins"
> and "groups" properties.
>
> This patch updates the lager device tree to use the generic
> rather than renesas-specific properties.
>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>

Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] ARM: dts: lager: use generic pinctrl properties
  2016-03-16  8:56 ` Geert Uytterhoeven
@ 2016-03-17  2:14   ` Simon Horman
  0 siblings, 0 replies; 4+ messages in thread
From: Simon Horman @ 2016-03-17  2:14 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Mar 16, 2016 at 09:56:24AM +0100, Geert Uytterhoeven wrote:
> On Wed, Mar 16, 2016 at 2:03 AM, Simon Horman
> <horms+renesas@verge.net.au> wrote:
> > Since 16ccaf5bb5a5 ("pinctrl: sh-pfc: Accept standard function, pins and
> > groups properties") renesas pfc drivers accept generic "function", "pins"
> > and "groups" properties.
> >
> > This patch updates the lager device tree to use the generic
> > rather than renesas-specific properties.
> >
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> 
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>

Thanks, I have queued this up as a cleanup for v4.7.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] ARM: dts: lager: use generic pinctrl properties
  2016-03-16  1:03 [PATCH] ARM: dts: lager: use generic pinctrl properties Simon Horman
  2016-03-16  8:56 ` Geert Uytterhoeven
@ 2016-03-17  6:20 ` Laurent Pinchart
  1 sibling, 0 replies; 4+ messages in thread
From: Laurent Pinchart @ 2016-03-17  6:20 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Simon,

Thank you for the patch.

On Wednesday 16 March 2016 10:03:09 Simon Horman wrote:
> Since 16ccaf5bb5a5 ("pinctrl: sh-pfc: Accept standard function, pins and
> groups properties") renesas pfc drivers accept generic "function", "pins"
> and "groups" properties.
> 
> This patch updates the lager device tree to use the generic
> rather than renesas-specific properties.
> 
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
> Based on renesas-devel-20160315-v4.5
> ---
>  arch/arm/boot/dts/r8a7790-lager.dts | 92 ++++++++++++++++-----------------
>  1 file changed, 46 insertions(+), 46 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/r8a7790-lager.dts
> b/arch/arm/boot/dts/r8a7790-lager.dts index aa6ca92a9485..a5828721ab7c
> 100644
> --- a/arch/arm/boot/dts/r8a7790-lager.dts
> +++ b/arch/arm/boot/dts/r8a7790-lager.dts
> @@ -314,119 +314,119 @@
>  	pinctrl-names = "default";
> 
>  	du_pins: du {
> -		renesas,groups = "du_rgb666", "du_sync_1", "du_clk_out_0";
> -		renesas,function = "du";
> +		groups = "du_rgb666", "du_sync_1", "du_clk_out_0";
> +		function = "du";
>  	};
> 
>  	scif0_pins: serial0 {
> -		renesas,groups = "scif0_data";
> -		renesas,function = "scif0";
> +		groups = "scif0_data";
> +		function = "scif0";
>  	};
> 
>  	scif_clk_pins: scif_clk {
> -		renesas,groups = "scif_clk";
> -		renesas,function = "scif_clk";
> +		groups = "scif_clk";
> +		function = "scif_clk";
>  	};
> 
>  	ether_pins: ether {
> -		renesas,groups = "eth_link", "eth_mdio", "eth_rmii";
> -		renesas,function = "eth";
> +		groups = "eth_link", "eth_mdio", "eth_rmii";
> +		function = "eth";
>  	};
> 
>  	phy1_pins: phy1 {
> -		renesas,groups = "intc_irq0";
> -		renesas,function = "intc";
> +		groups = "intc_irq0";
> +		function = "intc";
>  	};
> 
>  	scifa1_pins: serial1 {
> -		renesas,groups = "scifa1_data";
> -		renesas,function = "scifa1";
> +		groups = "scifa1_data";
> +		function = "scifa1";
>  	};
> 
>  	sdhi0_pins: sd0 {
> -		renesas,groups = "sdhi0_data4", "sdhi0_ctrl";
> -		renesas,function = "sdhi0";
> +		groups = "sdhi0_data4", "sdhi0_ctrl";
> +		function = "sdhi0";
>  	};
> 
>  	sdhi2_pins: sd2 {
> -		renesas,groups = "sdhi2_data4", "sdhi2_ctrl";
> -		renesas,function = "sdhi2";
> +		groups = "sdhi2_data4", "sdhi2_ctrl";
> +		function = "sdhi2";
>  	};
> 
>  	mmc1_pins: mmc1 {
> -		renesas,groups = "mmc1_data8", "mmc1_ctrl";
> -		renesas,function = "mmc1";
> +		groups = "mmc1_data8", "mmc1_ctrl";
> +		function = "mmc1";
>  	};
> 
>  	qspi_pins: spi0 {
> -		renesas,groups = "qspi_ctrl", "qspi_data4";
> -		renesas,function = "qspi";
> +		groups = "qspi_ctrl", "qspi_data4";
> +		function = "qspi";
>  	};
> 
>  	msiof1_pins: spi2 {
> -		renesas,groups = "msiof1_clk", "msiof1_sync", "msiof1_rx",
> +		groups = "msiof1_clk", "msiof1_sync", "msiof1_rx",
>  				 "msiof1_tx";
> -		renesas,function = "msiof1";
> +		function = "msiof1";
>  	};
> 
>  	i2c0_pins: i2c0 {
> -		renesas,groups = "i2c0";
> -		renesas,function = "i2c0";
> +		groups = "i2c0";
> +		function = "i2c0";
>  	};
> 
>  	iic0_pins: iic0 {
> -		renesas,groups = "iic0";
> -		renesas,function = "iic0";
> +		groups = "iic0";
> +		function = "iic0";
>  	};
> 
>  	iic1_pins: iic1 {
> -		renesas,groups = "iic1";
> -		renesas,function = "iic1";
> +		groups = "iic1";
> +		function = "iic1";
>  	};
> 
>  	iic2_pins: iic2 {
> -		renesas,groups = "iic2";
> -		renesas,function = "iic2";
> +		groups = "iic2";
> +		function = "iic2";
>  	};
> 
>  	iic3_pins: iic3 {
> -		renesas,groups = "iic3";
> -		renesas,function = "iic3";
> +		groups = "iic3";
> +		function = "iic3";
>  	};
> 
>  	hsusb_pins: hsusb {
> -		renesas,groups = "usb0_ovc_vbus";
> -		renesas,function = "usb0";
> +		groups = "usb0_ovc_vbus";
> +		function = "usb0";
>  	};
> 
>  	usb0_pins: usb0 {
> -		renesas,groups = "usb0";
> -		renesas,function = "usb0";
> +		groups = "usb0";
> +		function = "usb0";
>  	};
> 
>  	usb1_pins: usb1 {
> -		renesas,groups = "usb1";
> -		renesas,function = "usb1";
> +		groups = "usb1";
> +		function = "usb1";
>  	};
> 
>  	usb2_pins: usb2 {
> -		renesas,groups = "usb2";
> -		renesas,function = "usb2";
> +		groups = "usb2";
> +		function = "usb2";
>  	};
> 
>  	vin1_pins: vin {
> -		renesas,groups = "vin1_data8", "vin1_clk";
> -		renesas,function = "vin1";
> +		groups = "vin1_data8", "vin1_clk";
> +		function = "vin1";
>  	};
> 
>  	sound_pins: sound {
> -		renesas,groups = "ssi0129_ctrl", "ssi0_data", "ssi1_data";
> -		renesas,function = "ssi";
> +		groups = "ssi0129_ctrl", "ssi0_data", "ssi1_data";
> +		function = "ssi";
>  	};
> 
>  	sound_clk_pins: sound_clk {
> -		renesas,groups = "audio_clk_a";
> -		renesas,function = "audio_clk";
> +		groups = "audio_clk_a";
> +		function = "audio_clk";
>  	};
>  };

-- 
Regards,

Laurent Pinchart

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-03-17  6:20 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-03-16  1:03 [PATCH] ARM: dts: lager: use generic pinctrl properties Simon Horman
2016-03-16  8:56 ` Geert Uytterhoeven
2016-03-17  2:14   ` Simon Horman
2016-03-17  6:20 ` Laurent Pinchart

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).