Re: [PATCH 2/2] ARM: dts: r8a7743: Add I2C DT support

2017-08-13 Thread Simon Horman 
On Wed, Aug 09, 2017 at 09:23:05AM +, Biju Das wrote:
> 
> 
> > -Original Message-
> > From: Simon Horman [mailto:ho...@verge.net.au]
> > Sent: 09 August 2017 09:53
> > To: Biju Das <biju@bp.renesas.com>
> > Cc: Rob Herring <robh...@kernel.org>; Mark Rutland
> > <mark.rutl...@arm.com>; Wolfram Sang <w...@the-dreams.de>; Magnus
> > Damm <magnus.d...@gmail.com>; Russell King <li...@armlinux.org.uk>;
> > Chris Paterson <chris.paters...@renesas.com>; devicet...@vger.kernel.org;
> > linux-renesas-soc@vger.kernel.org; linux-arm-ker...@lists.infradead.org
> > Subject: Re: [PATCH 2/2] ARM: dts: r8a7743: Add I2C DT support
> >
> > On Tue, Aug 08, 2017 at 12:24:09PM +0100, Biju Das wrote:
> > > Add the I2C[0-5] devices to the r8a7743 device tree.
> > >
> > > Signed-off-by: Biju Das <biju@bp.renesas.com>
> > > ---
> > > This patch has been tested against Linux-next tag 20170727 and renesas-dev
> > branch.
> > > This patch depends on
> > > https://www.mail-archive.com/linux-renesas-soc@vger.kernel.org/msg1700
> > > 8.html
> >
> > As per the patch at the link above it seems that these patches are targeted 
> > at
> > the renesas tree.  The best practice in that case is to base patches on the 
> > latest
> > devel branch. Please consider doing so in future.
> >
> > I would also slightly prefer if dependencies were referenced by name, in 
> > this
> > case:
> >
> > *  [PATCH 2/3] ARM: dts: r8a7743: Add APMU node and second CPU core
> >
> > or
> >
> > * [PATCH  0/3] Add SMP support
> >
> > Also including a link is fine by me.
> >
> >
> > It would also be useful if you described the dependency as a conflict 
> > (merge-
> > time), compile-time or run-time.
> 
> 
> Thanks Simon. I will take care this next time.
> 
> > As for the patch itself, it looks good to me.

As I believe the dependencies are apply-time and seemed easy to resolve
I have applied this patch for v4.14.

RE: [PATCH 2/2] ARM: dts: r8a7743: Add I2C DT support

2017-08-09 Thread Biju Das 


> -Original Message-
> From: Simon Horman [mailto:ho...@verge.net.au]
> Sent: 09 August 2017 09:53
> To: Biju Das <biju@bp.renesas.com>
> Cc: Rob Herring <robh...@kernel.org>; Mark Rutland
> <mark.rutl...@arm.com>; Wolfram Sang <w...@the-dreams.de>; Magnus
> Damm <magnus.d...@gmail.com>; Russell King <li...@armlinux.org.uk>;
> Chris Paterson <chris.paters...@renesas.com>; devicet...@vger.kernel.org;
> linux-renesas-soc@vger.kernel.org; linux-arm-ker...@lists.infradead.org
> Subject: Re: [PATCH 2/2] ARM: dts: r8a7743: Add I2C DT support
>
> On Tue, Aug 08, 2017 at 12:24:09PM +0100, Biju Das wrote:
> > Add the I2C[0-5] devices to the r8a7743 device tree.
> >
> > Signed-off-by: Biju Das <biju@bp.renesas.com>
> > ---
> > This patch has been tested against Linux-next tag 20170727 and renesas-dev
> branch.
> > This patch depends on
> > https://www.mail-archive.com/linux-renesas-soc@vger.kernel.org/msg1700
> > 8.html
>
> As per the patch at the link above it seems that these patches are targeted at
> the renesas tree.  The best practice in that case is to base patches on the 
> latest
> devel branch. Please consider doing so in future.
>
> I would also slightly prefer if dependencies were referenced by name, in this
> case:
>
> *  [PATCH 2/3] ARM: dts: r8a7743: Add APMU node and second CPU core
>
> or
>
> * [PATCH  0/3] Add SMP support
>
> Also including a link is fine by me.
>
>
> It would also be useful if you described the dependency as a conflict (merge-
> time), compile-time or run-time.


Thanks Simon. I will take care this next time.

> As for the patch itself, it looks good to me.
>
> >  arch/arm/boot/dts/r8a7743.dtsi | 97
> > ++
> >  1 file changed, 97 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/r8a7743.dtsi
> > b/arch/arm/boot/dts/r8a7743.dtsi index 8c46f62..14222c72 100644
> > --- a/arch/arm/boot/dts/r8a7743.dtsi
> > +++ b/arch/arm/boot/dts/r8a7743.dtsi
> > @@ -18,6 +18,15 @@
> >  #address-cells = <2>;
> >  #size-cells = <2>;
> >
> > +aliases {
> > +i2c0 = 
> > +i2c1 = 
> > +i2c2 = 
> > +i2c3 = 
> > +i2c4 = 
> > +i2c5 = 
> > +};
> > +
> >  cpus {
> >  #address-cells = <1>;
> >  #size-cells = <0>;
> > @@ -339,6 +348,94 @@
> >  dma-channels = <15>;
> >  };
> >
> > +/* The memory map in the User's Manual maps the cores to
> bus
> > + *  numbers
> > + */
> > +i2c0: i2c@e6508000 {
> > +#address-cells = <1>;
> > +#size-cells = <0>;
> > +compatible = "renesas,i2c-r8a7743",
> > + "renesas,rcar-gen2-i2c";
> > +reg = <0 0xe6508000 0 0x40>;
> > +interrupts = ;
> > +clocks = < CPG_MOD 931>;
> > +power-domains = < R8A7743_PD_ALWAYS_ON>;
> > +resets = < 931>;
> > +i2c-scl-internal-delay-ns = <6>;
> > +status = "disabled";
> > +};
> > +
> > +i2c1: i2c@e6518000 {
> > +#address-cells = <1>;
> > +#size-cells = <0>;
> > +compatible = "renesas,i2c-r8a7743",
> > + "renesas,rcar-gen2-i2c";
> > +reg = <0 0xe6518000 0 0x40>;
> > +interrupts = ;
> > +clocks = < CPG_MOD 930>;
> > +power-domains = < R8A7743_PD_ALWAYS_ON>;
> > +resets = < 930>;
> > +i2c-scl-internal-delay-ns = <6>;
> > +status = "disabled";
> > +};
> > +
> > +i2c2: i2c@e653 {
> > +#address-cells = <1>;
> > +#size-cells = <0>;
> > +compatible = "renesas,i2c-r8a7743",
> > + "renesas,rcar-gen2-i2c";
> > +reg = <0 0xe653 0 0x40>;
> > +interrupts = ;
> > +clocks = < CPG_MOD 929>;
> > +power-domains = < R8A7743_PD_ALWAYS_ON>;
> > +resets = < 929>;
> > +i2c-scl-internal-delay-ns = <6>;
> > +status = "disabled";
> > +};
> > +
> > +i2c3: i2c@e654 {
> > +#address-cells = <1>;
> > +#size-cells = <0>;
> > +compatible = "renesas,i2c-r8a7743",
> > + "renesas,rcar-gen2-i2c";
> > +reg = <0 0xe654 0 0x40>;
> > +interrupts = ;
> > +clocks = < CPG_MOD 928>;
> > +power-domains = < R8A7743_PD_ALWAYS_ON>;
> > +resets = < 928>;
> > +i2c-scl-internal-delay-ns = <6>;
> > +status = "disabled";
> > +};
> > +
> > +i2c4: i2c@e652 {
> > +#address-cells

Re: [PATCH 2/2] ARM: dts: r8a7743: Add I2C DT support

2017-08-09 Thread Simon Horman 
On Tue, Aug 08, 2017 at 12:24:09PM +0100, Biju Das wrote:
> Add the I2C[0-5] devices to the r8a7743 device tree.
> 
> 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.mail-archive.com/linux-renesas-soc@vger.kernel.org/msg17008.html

As per the patch at the link above it seems that these patches are targeted
at the renesas tree.  The best practice in that case is to base patches on
the latest devel branch. Please consider doing so in future.

I would also slightly prefer if dependencies were referenced by name, in
this case:

*  [PATCH 2/3] ARM: dts: r8a7743: Add APMU node and second CPU core

or

* [PATCH  0/3] Add SMP support

Also including a link is fine by me.


It would also be useful if you described the dependency as a conflict
(merge-time), compile-time or run-time.

As for the patch itself, it looks good to me.

>  arch/arm/boot/dts/r8a7743.dtsi | 97 
> ++
>  1 file changed, 97 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/r8a7743.dtsi b/arch/arm/boot/dts/r8a7743.dtsi
> index 8c46f62..14222c72 100644
> --- a/arch/arm/boot/dts/r8a7743.dtsi
> +++ b/arch/arm/boot/dts/r8a7743.dtsi
> @@ -18,6 +18,15 @@
>   #address-cells = <2>;
>   #size-cells = <2>;
>  
> + aliases {
> + i2c0 = 
> + i2c1 = 
> + i2c2 = 
> + i2c3 = 
> + i2c4 = 
> + i2c5 = 
> + };
> +
>   cpus {
>   #address-cells = <1>;
>   #size-cells = <0>;
> @@ -339,6 +348,94 @@
>   dma-channels = <15>;
>   };
>  
> + /* The memory map in the User's Manual maps the cores to bus
> +  *  numbers
> +  */
> + i2c0: i2c@e6508000 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "renesas,i2c-r8a7743",
> +  "renesas,rcar-gen2-i2c";
> + reg = <0 0xe6508000 0 0x40>;
> + interrupts = ;
> + clocks = < CPG_MOD 931>;
> + power-domains = < R8A7743_PD_ALWAYS_ON>;
> + resets = < 931>;
> + i2c-scl-internal-delay-ns = <6>;
> + status = "disabled";
> + };
> +
> + i2c1: i2c@e6518000 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "renesas,i2c-r8a7743",
> +  "renesas,rcar-gen2-i2c";
> + reg = <0 0xe6518000 0 0x40>;
> + interrupts = ;
> + clocks = < CPG_MOD 930>;
> + power-domains = < R8A7743_PD_ALWAYS_ON>;
> + resets = < 930>;
> + i2c-scl-internal-delay-ns = <6>;
> + status = "disabled";
> + };
> +
> + i2c2: i2c@e653 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "renesas,i2c-r8a7743",
> +  "renesas,rcar-gen2-i2c";
> + reg = <0 0xe653 0 0x40>;
> + interrupts = ;
> + clocks = < CPG_MOD 929>;
> + power-domains = < R8A7743_PD_ALWAYS_ON>;
> + resets = < 929>;
> + i2c-scl-internal-delay-ns = <6>;
> + status = "disabled";
> + };
> +
> + i2c3: i2c@e654 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "renesas,i2c-r8a7743",
> +  "renesas,rcar-gen2-i2c";
> + reg = <0 0xe654 0 0x40>;
> + interrupts = ;
> + clocks = < CPG_MOD 928>;
> + power-domains = < R8A7743_PD_ALWAYS_ON>;
> + resets = < 928>;
> + i2c-scl-internal-delay-ns = <6>;
> + status = "disabled";
> + };
> +
> + i2c4: i2c@e652 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> + compatible = "renesas,i2c-r8a7743",
> +  "renesas,rcar-gen2-i2c";
> + reg = <0 0xe652 0 0x40>;
> + interrupts = ;
> + clocks = < CPG_MOD 927>;
> + power-domains = < R8A7743_PD_ALWAYS_ON>;
> + resets = < 927>;
> + i2c-scl-internal-delay-ns = <6>;
> + status = "disabled";
> + };
> +
> + i2c5: i2c@e6528000 {
> +

[PATCH 2/2] ARM: dts: r8a7743: Add I2C DT support

2017-08-08 Thread Biju Das 
Add the I2C[0-5] devices to the r8a7743 device tree.

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.mail-archive.com/linux-renesas-soc@vger.kernel.org/msg17008.html

 arch/arm/boot/dts/r8a7743.dtsi | 97 ++
 1 file changed, 97 insertions(+)

diff --git a/arch/arm/boot/dts/r8a7743.dtsi b/arch/arm/boot/dts/r8a7743.dtsi
index 8c46f62..14222c72 100644
--- a/arch/arm/boot/dts/r8a7743.dtsi
+++ b/arch/arm/boot/dts/r8a7743.dtsi
@@ -18,6 +18,15 @@
#address-cells = <2>;
#size-cells = <2>;
 
+   aliases {
+   i2c0 = 
+   i2c1 = 
+   i2c2 = 
+   i2c3 = 
+   i2c4 = 
+   i2c5 = 
+   };
+
cpus {
#address-cells = <1>;
#size-cells = <0>;
@@ -339,6 +348,94 @@
dma-channels = <15>;
};
 
+   /* The memory map in the User's Manual maps the cores to bus
+*  numbers
+*/
+   i2c0: i2c@e6508000 {
+   #address-cells = <1>;
+   #size-cells = <0>;
+   compatible = "renesas,i2c-r8a7743",
+"renesas,rcar-gen2-i2c";
+   reg = <0 0xe6508000 0 0x40>;
+   interrupts = ;
+   clocks = < CPG_MOD 931>;
+   power-domains = < R8A7743_PD_ALWAYS_ON>;
+   resets = < 931>;
+   i2c-scl-internal-delay-ns = <6>;
+   status = "disabled";
+   };
+
+   i2c1: i2c@e6518000 {
+   #address-cells = <1>;
+   #size-cells = <0>;
+   compatible = "renesas,i2c-r8a7743",
+"renesas,rcar-gen2-i2c";
+   reg = <0 0xe6518000 0 0x40>;
+   interrupts = ;
+   clocks = < CPG_MOD 930>;
+   power-domains = < R8A7743_PD_ALWAYS_ON>;
+   resets = < 930>;
+   i2c-scl-internal-delay-ns = <6>;
+   status = "disabled";
+   };
+
+   i2c2: i2c@e653 {
+   #address-cells = <1>;
+   #size-cells = <0>;
+   compatible = "renesas,i2c-r8a7743",
+"renesas,rcar-gen2-i2c";
+   reg = <0 0xe653 0 0x40>;
+   interrupts = ;
+   clocks = < CPG_MOD 929>;
+   power-domains = < R8A7743_PD_ALWAYS_ON>;
+   resets = < 929>;
+   i2c-scl-internal-delay-ns = <6>;
+   status = "disabled";
+   };
+
+   i2c3: i2c@e654 {
+   #address-cells = <1>;
+   #size-cells = <0>;
+   compatible = "renesas,i2c-r8a7743",
+"renesas,rcar-gen2-i2c";
+   reg = <0 0xe654 0 0x40>;
+   interrupts = ;
+   clocks = < CPG_MOD 928>;
+   power-domains = < R8A7743_PD_ALWAYS_ON>;
+   resets = < 928>;
+   i2c-scl-internal-delay-ns = <6>;
+   status = "disabled";
+   };
+
+   i2c4: i2c@e652 {
+   #address-cells = <1>;
+   #size-cells = <0>;
+   compatible = "renesas,i2c-r8a7743",
+"renesas,rcar-gen2-i2c";
+   reg = <0 0xe652 0 0x40>;
+   interrupts = ;
+   clocks = < CPG_MOD 927>;
+   power-domains = < R8A7743_PD_ALWAYS_ON>;
+   resets = < 927>;
+   i2c-scl-internal-delay-ns = <6>;
+   status = "disabled";
+   };
+
+   i2c5: i2c@e6528000 {
+   /* doesn't need pinmux */
+   #address-cells = <1>;
+   #size-cells = <0>;
+   compatible = "renesas,i2c-r8a7743",
+"renesas,rcar-gen2-i2c";
+   reg = <0 0xe6528000 0 0x40>;
+   interrupts = ;
+   clocks = < CPG_MOD 925>;
+   power-domains = < R8A7743_PD_ALWAYS_ON>;
+   resets = < 925>;
+   i2c-scl-internal-delay-ns = <110>;
+   status = "disabled";
+   };
+
scifa0: serial@e6c4 {
compatible =