On Wed, Aug 09, 2017 at 12:17:06PM +0300, Sergei Shtylyov wrote: > On 8/8/2017 3:46 PM, Biju Das wrote: > > >Signed-off-by: Biju Das <biju.das@xxxxxxxxxxxxxx> > >--- > >This patch has been tested against Linux-next tag 20170727 and renesas-dev branch. > >This patch depends on https://www.spinics.net/lists/arm-kernel/msg599220.html > > > > arch/arm/boot/dts/r8a7743.dtsi | 55 ++++++++++++++++++++++++++++++++++++++++++ > > 1 file changed, 55 insertions(+) > > > >diff --git a/arch/arm/boot/dts/r8a7743.dtsi b/arch/arm/boot/dts/r8a7743.dtsi > >index 14222c72..5c2b140 100644 > >--- a/arch/arm/boot/dts/r8a7743.dtsi > >+++ b/arch/arm/boot/dts/r8a7743.dtsi > [...] > >@@ -436,6 +439,58 @@ > > status = "disabled"; > > }; > >+ i2c6: i2c@e60b0000 { > > I'd use iic0 as the label. I have no preference here other than that we try to be consistent in DT for different R-Car SoCs. Wolfram, do you have an opinion on this? > > >+ /* doesn't need pinmux */ > >+ #address-cells = <1>; > >+ #size-cells = <0>; > >+ compatible = "renesas,iic-r8a7743", > >+ "renesas,rcar-gen2-iic", > >+ "renesas,rmobile-iic"; > >+ reg = <0 0xe60b0000 0 0x425>; > >+ interrupts = <GIC_SPI 173 IRQ_TYPE_LEVEL_HIGH>; > >+ clocks = <&cpg CPG_MOD 926>; > >+ dmas = <&dmac0 0x77>, <&dmac0 0x78>, > >+ <&dmac1 0x77>, <&dmac1 0x78>; > >+ dma-names = "tx", "rx", "tx", "rx"; > >+ power-domains = <&sysc R8A7743_PD_ALWAYS_ON>; > >+ resets = <&cpg CPG_MOD 926>; > > CPG_MOD not needed here, it's not a "clocks" prop. > > [...] > > MBR, Sergei > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
