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@xxxxxxxxxxxx> Acked-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > 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