[PATCH 2/2] ARM: dts: r8a7743: Add IIC cores to dtsi

[Biju Das]

> -----Original Message-----
> From: Sergei Shtylyov [mailto:sergei.shtylyov at cogentembedded.com]
> Sent: 09 August 2017 10:17
> To: Biju Das <biju.das at bp.renesas.com>; Rob Herring <robh+dt at kernel.org>;
> Mark Rutland <mark.rutland at arm.com>
> Cc: Simon Horman <horms at verge.net.au>; Wolfram Sang <wsa at the-
> dreams.de>; Magnus Damm <magnus.damm at gmail.com>; Russell King
> <linux at armlinux.org.uk>; Chris Paterson <Chris.Paterson2 at renesas.com>;
> devicetree at vger.kernel.org; linux-renesas-soc at vger.kernel.org; linux-arm-
> kernel at 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 <biju.das at bp.renesas.com>
> > ---
> > 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 at 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 = <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.
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.