> -----Original Message----- > From: Sergei Shtylyov [mailto:sergei.shtylyov@cogentembedded.com] > Sent: 09 August 2017 10:17 > To: Biju Das ; Rob Herring ; > Mark Rutland > Cc: Simon Horman ; Wolfram Sang dreams.de>; Magnus Damm ; Russell King > ; Chris Paterson ; > devicetree@vger.kernel.org; linux-renesas-soc@vger.kernel.org; linux-arm- > kernel@lists.infradead.org > Subject: Re: [PATCH 2/2] ARM: dts: r8a7743: Add IIC cores to dtsi > > On 8/8/2017 3:46 PM, Biju Das wrote: > > > Signed-off-by: Biju Das > > --- > > 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. > > > +/* 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 = ; > > +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. Thanks Sergie, I will correct this. [>] > [...] > > MBR, Sergei Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709. NrybXǧv^)޺{.n+zzz)w*jgݢj.۰\Mgja' ޢ j:+vwjmzZ+ݢj"!i