diff mbox

[v2,3/3] ARM: dts: blanche: add Ethernet pins

Message ID 3279336.lifIduxqUW@wasted.cogentembedded.com (mailing list archive)
State Superseded
Delegated to: Simon Horman
Headers show

Commit Message

Sergei Shtylyov July 4, 2016, 8:55 p.m. UTC
Add the (previously omitted) pin data for the SMSC LAN89218 Ethernet chip
to the Blanche board's device tree: the chip's IRQ output is connected to
the SoC's IRQ0 pin and its nCS input is connected to the SoC's EX_CS0# pin.

Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

---
 arch/arm/boot/dts/r8a7792-blanche.dts |   15 +++++++++++++++
 1 file changed, 15 insertions(+)

Comments

Geert Uytterhoeven July 5, 2016, 7:29 a.m. UTC | #1
Hi Sergei,

On Mon, Jul 4, 2016 at 10:55 PM, Sergei Shtylyov
<sergei.shtylyov@cogentembedded.com> wrote:
> Add the (previously omitted) pin data for the SMSC LAN89218 Ethernet chip
> to the Blanche board's device tree: the chip's IRQ output is connected to
> the SoC's IRQ0 pin and its nCS input is connected to the SoC's EX_CS0# pin.
>
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>
> ---
>  arch/arm/boot/dts/r8a7792-blanche.dts |   15 +++++++++++++++
>  1 file changed, 15 insertions(+)
>
> Index: renesas/arch/arm/boot/dts/r8a7792-blanche.dts
> ===================================================================
> --- renesas.orig/arch/arm/boot/dts/r8a7792-blanche.dts
> +++ renesas/arch/arm/boot/dts/r8a7792-blanche.dts
> @@ -50,6 +50,9 @@
>                 reg-io-width = <4>;
>                 vddvario-supply = <&d3_3v>;
>                 vdd33a-supply = <&d3_3v>;
> +
> +               pinctrl-0 = <&lan89218_pins>;
> +               pinctrl-names = "default";
>         };
>  };
>
> @@ -67,6 +70,18 @@
>                 groups = "scif3_data";
>                 function = "scif3";
>         };
> +
> +       lan89218_pins: lan89218 {
> +               intc {
> +                       groups = "intc_irq0";
> +                       function = "intc";
> +               };
> +               lbsc {
> +                       groups = "lbsc_ex_cs0";
> +                       function = "lbsc";

Don't you need pinctrl for the other LBSC pins, like the address and data lines
(Ax and Dy)?

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Sergei Shtylyov July 5, 2016, 1:43 p.m. UTC | #2
On 07/05/2016 10:29 AM, Geert Uytterhoeven wrote:

>> Add the (previously omitted) pin data for the SMSC LAN89218 Ethernet chip
>> to the Blanche board's device tree: the chip's IRQ output is connected to
>> the SoC's IRQ0 pin and its nCS input is connected to the SoC's EX_CS0# pin.
>>
>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>>
>> ---
>>  arch/arm/boot/dts/r8a7792-blanche.dts |   15 +++++++++++++++
>>  1 file changed, 15 insertions(+)
>>
>> Index: renesas/arch/arm/boot/dts/r8a7792-blanche.dts
>> ===================================================================
>> --- renesas.orig/arch/arm/boot/dts/r8a7792-blanche.dts
>> +++ renesas/arch/arm/boot/dts/r8a7792-blanche.dts
[...]
>> @@ -67,6 +70,18 @@
>>                 groups = "scif3_data";
>>                 function = "scif3";
>>         };
>> +
>> +       lan89218_pins: lan89218 {
>> +               intc {
>> +                       groups = "intc_irq0";
>> +                       function = "intc";
>> +               };
>> +               lbsc {
>> +                       groups = "lbsc_ex_cs0";
>> +                       function = "lbsc";
>
> Don't you need pinctrl for the other LBSC pins, like the address and data lines
> (Ax and Dy)?
>

    Unlike the chip selects, the address/data lines are not "monopolized" by a 
single device, so I'm not sure what to do with them...

> Gr{oetje,eeting}s,
>
>                         Geert

MBR, Sergei
Sergei Shtylyov July 6, 2016, 7:12 p.m. UTC | #3
On 07/04/2016 11:55 PM, Sergei Shtylyov wrote:

> Add the (previously omitted) pin data for the SMSC LAN89218 Ethernet chip
> to the Blanche board's device tree: the chip's IRQ output is connected to
> the SoC's IRQ0 pin and its nCS input is connected to the SoC's EX_CS0# pin.
>
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>
> ---
>  arch/arm/boot/dts/r8a7792-blanche.dts |   15 +++++++++++++++
>  1 file changed, 15 insertions(+)
>
> Index: renesas/arch/arm/boot/dts/r8a7792-blanche.dts
> ===================================================================
> --- renesas.orig/arch/arm/boot/dts/r8a7792-blanche.dts
> +++ renesas/arch/arm/boot/dts/r8a7792-blanche.dts
[...]
> @@ -67,6 +70,18 @@
>  		groups = "scif3_data";
>  		function = "scif3";
>  	};
> +
> +	lan89218_pins: lan89218 {
> +		intc {
> +			groups = "intc_irq0";
> +			function = "intc";
> +		};
> +		lbsc {
> +			groups = "lbsc_ex_cs0";
> +			function = "lbsc";
> +		};
> +	};
> +

    Oops, stray newline... will fix!

>  };
>
>  &scif0 {

MBR, Sergei
diff mbox

Patch

Index: renesas/arch/arm/boot/dts/r8a7792-blanche.dts
===================================================================
--- renesas.orig/arch/arm/boot/dts/r8a7792-blanche.dts
+++ renesas/arch/arm/boot/dts/r8a7792-blanche.dts
@@ -50,6 +50,9 @@ 
 		reg-io-width = <4>;
 		vddvario-supply = <&d3_3v>;
 		vdd33a-supply = <&d3_3v>;
+
+		pinctrl-0 = <&lan89218_pins>;
+		pinctrl-names = "default";
 	};
 };
 
@@ -67,6 +70,18 @@ 
 		groups = "scif3_data";
 		function = "scif3";
 	};
+
+	lan89218_pins: lan89218 {
+		intc {
+			groups = "intc_irq0";
+			function = "intc";
+		};
+		lbsc {
+			groups = "lbsc_ex_cs0";
+			function = "lbsc";
+		};
+	};
+
 };
 
 &scif0 {