[PATCH v3 4/7] arm: dts: r7s72100: Add pin controller node

[Jacopo Mondi]

Add pin controller node with 12 gpio controller sub-nodes to
r7s72100 dtsi.

Signed-off-by: Jacopo Mondi 
Reviewed-by: Geert Uytterhoeven 
---
 arch/arm/boot/dts/r7s72100.dtsi | 80 +
 1 file changed, 80 insertions(+)

diff --git a/arch/arm/boot/dts/r7s72100.dtsi b/arch/arm/boot/dts/r7s72100.dtsi
index b8aa256..83e8f27 100644
--- a/arch/arm/boot/dts/r7s72100.dtsi
+++ b/arch/arm/boot/dts/r7s72100.dtsi
@@ -180,6 +180,86 @@
};
};
 
+   pinctrl: pinctrl@fcfe3000 {
+   compatible = "renesas,r7s72100-ports";
+
+   #pinctrl-cells = <1>;
+
+   reg = <0xfcfe3000 0x42C0>;
+
+   port0: gpio-0 {
+   gpio-controller;
+   #gpio-cells = <2>;
+   gpio-ranges = <&pinctrl 0 0 6>;
+   };
+
+   port1: gpio-1 {
+   gpio-controller;
+   #gpio-cells = <2>;
+   gpio-ranges = <&pinctrl 0 16 16>;
+   };
+
+   port2: gpio-2 {
+   gpio-controller;
+   #gpio-cells = <2>;
+   gpio-ranges = <&pinctrl 0 32 16>;
+   };
+
+   port3: gpio-3 {
+   gpio-controller;
+   #gpio-cells = <2>;
+   gpio-ranges = <&pinctrl 0 48 16>;
+   };
+
+   port4: gpio-4 {
+   gpio-controller;
+   #gpio-cells = <2>;
+   gpio-ranges = <&pinctrl 0 64 16>;
+   };
+
+   port5: gpio-5 {
+   gpio-controller;
+   #gpio-cells = <2>;
+   gpio-ranges = <&pinctrl 0 80 11>;
+   };
+
+   port6: gpio-6 {
+   gpio-controller;
+   #gpio-cells = <2>;
+   gpio-ranges = <&pinctrl 0 96 16>;
+   };
+
+   port7: gpio-7 {
+   gpio-controller;
+   #gpio-cells = <2>;
+   gpio-ranges = <&pinctrl 0 112 16>;
+   };
+
+   port8: gpio-8 {
+   gpio-controller;
+   #gpio-cells = <2>;
+   gpio-ranges = <&pinctrl 0 128 16>;
+   };
+
+   port9: gpio-9 {
+   gpio-controller;
+   #gpio-cells = <2>;
+   gpio-ranges = <&pinctrl 0 144 8>;
+   };
+
+   port10: gpio-10 {
+   gpio-controller;
+   #gpio-cells = <2>;
+   gpio-ranges = <&pinctrl 0 160 16>;
+   };
+
+   port11: gpio-11 {
+   gpio-controller;
+   #gpio-cells = <2>;
+   gpio-ranges = <&pinctrl 0 176 16>;
+   };
+   };
+
scif0: serial@e8007000 {
compatible = "renesas,scif-r7s72100", "renesas,scif";
reg = <0xe8007000 64>;
-- 
2.7.4

Re: [PATCH v3 4/7] arm: dts: r7s72100: Add pin controller node

[Sergei Shtylyov]

Hello!

On 03/24/2017 06:22 PM, Jacopo Mondi wrote:


Add pin controller node with 12 gpio controller sub-nodes to
r7s72100 dtsi.

Signed-off-by: Jacopo Mondi 
Reviewed-by: Geert Uytterhoeven 
---
 arch/arm/boot/dts/r7s72100.dtsi | 80 +
 1 file changed, 80 insertions(+)

diff --git a/arch/arm/boot/dts/r7s72100.dtsi b/arch/arm/boot/dts/r7s72100.dtsi
index b8aa256..83e8f27 100644
--- a/arch/arm/boot/dts/r7s72100.dtsi
+++ b/arch/arm/boot/dts/r7s72100.dtsi
@@ -180,6 +180,86 @@
};
};

+   pinctrl: pinctrl@fcfe3000 {


   I suggest to call it "pin-controller@...", no need to reduce.

[...]

MBR, Sergei

RE: [PATCH v3 4/7] arm: dts: r7s72100: Add pin controller node

[Chris Brandt]

Hi Jacopo,


On Friday, March 24, 2017, Jacopo Mondi
> + pinctrl: pinctrl@fcfe3000 {
> + compatible = "renesas,r7s72100-ports";
> +
> + #pinctrl-cells = <1>;
> +
> + reg = <0xfcfe3000 0x42C0>;

Out of curiosity, why did you change this from 0x4248 to 0x42C0?

In your update for renesas,rza1-pinctrl.txt, for the 'Example', you changed it 
from 0x4248 to 0x4230 (which Geert pointed out makes more sense), but then in 
the actual DT file you changed it to 0x42C0. Typo?


Chris

Re: [PATCH v3 4/7] arm: dts: r7s72100: Add pin controller node

[jacopo]

Hi Chris,

On Mon, Mar 27, 2017 at 05:12:04PM +, Chris Brandt wrote:
> Hi Jacopo,
> 
> 
> On Friday, March 24, 2017, Jacopo Mondi
> > +   pinctrl: pinctrl@fcfe3000 {
> > +   compatible = "renesas,r7s72100-ports";
> > +
> > +   #pinctrl-cells = <1>;
> > +
> > +   reg = <0xfcfe3000 0x42C0>;
> 
> Out of curiosity, why did you change this from 0x4248 to 0x42C0?
> 
> In your update for renesas,rza1-pinctrl.txt, for the 'Example', you changed 
> it from 0x4248 to 0x4230 (which Geert pointed out makes more sense), but then 
> in the actual DT file you changed it to 0x42C0. Typo?
>

Yes, type.
Or inability to perform basic mathematical operations.
Not sure yet.

Thanks for spotting this :)
   j


> 
> Chris
>