[PATCH/RFC 07/10] arm64: dts: r8a7795: enable EthernetAVB on Salvator-X

linux-sh.vger.kernel.org archive mirror
 help / color / mirror / Atom feed

* [PATCH/RFC 07/10] arm64: dts: r8a7795: enable EthernetAVB on Salvator-X
@ 2015-08-27  9:24 Simon Horman
  2015-08-27 10:40 ` Sergei Shtylyov
                   ` (5 more replies)
  0 siblings, 6 replies; 7+ messages in thread
From: Simon Horman @ 2015-08-27  9:24 UTC (permalink / raw)
  To: linux-sh

From: Kazuya Mizuguchi <kazuya.mizuguchi.ks@renesas.com>

Signed-off-by: Kazuya Mizuguchi <kazuya.mizuguchi.ks@renesas.com>
[horms: updated filename to salvator-x; updated changelog]
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
 arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 34 ++++++++++++++++++++++
 arch/arm64/boot/dts/renesas/r8a7795.dtsi           |  1 +
 2 files changed, 35 insertions(+)

diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
index 88cc34819e6a..e8d940cc831a 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
@@ -53,6 +53,11 @@
 		renesas,groups = "usb2";
 		renesas,function = "usb2";
 	};
+
+	avb_pins: avb {
+		renesas,groups = "avb_mdc";
+		renesas,function = "avb";
+	};
 };
 
 &scif2 {
@@ -103,3 +108,32 @@
 &ohci2 {
 	status = "okay";
 };
+
+&avb {
+	pinctrl-0 = <&avb_pins>;
+	pinctrl-names = "default";
+	renesas,no-ether-link;
+	renesas,phy = <0>;
+	phy-reset-gpio = <&gpio2 10 0>;
+	phy-handle = <&phy0>;
+	status = "okay";
+
+	phy0: ethernet-phy@0 {
+		rxc-skew-ps = <900>;
+		rxdv-skew-ps = <0>;
+		rxd0-skew-ps = <0>;
+		rxd1-skew-ps = <0>;
+		rxd2-skew-ps = <0>;
+		rxd3-skew-ps = <0>;
+		txc-skew-ps = <900>;
+		txen-skew-ps = <0>;
+		txd0-skew-ps = <0>;
+		txd1-skew-ps = <0>;
+		txd2-skew-ps = <0>;
+		txd3-skew-ps = <0>;
+		reg = <0>;
+		interrupt-parent = <&gpio2>;
+		interrupts = <11 IRQ_TYPE_LEVEL_LOW>;
+	};
+
+};
diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
index 01502f4d2f5f..42c2eaa8167a 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
@@ -372,4 +372,5 @@
 			status = "disabled";
 		};
 	};
+
 };
-- 
2.1.4


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH/RFC 07/10] arm64: dts: r8a7795: enable EthernetAVB on Salvator-X
  2015-08-27  9:24 [PATCH/RFC 07/10] arm64: dts: r8a7795: enable EthernetAVB on Salvator-X Simon Horman
@ 2015-08-27 10:40 ` Sergei Shtylyov
  2015-08-27 11:14 ` Geert Uytterhoeven
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Sergei Shtylyov @ 2015-08-27 10:40 UTC (permalink / raw)
  To: linux-sh

Hello.

On 8/27/2015 12:24 PM, Simon Horman wrote:

> From: Kazuya Mizuguchi <kazuya.mizuguchi.ks@renesas.com>
>
> Signed-off-by: Kazuya Mizuguchi <kazuya.mizuguchi.ks@renesas.com>
> [horms: updated filename to salvator-x; updated changelog]
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
>   arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 34 ++++++++++++++++++++++
>   arch/arm64/boot/dts/renesas/r8a7795.dtsi           |  1 +
>   2 files changed, 35 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> index 88cc34819e6a..e8d940cc831a 100644
> --- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
[...]
> @@ -103,3 +108,32 @@
>   &ohci2 {
>   	status = "okay";
>   };
> +
> +&avb {
> +	pinctrl-0 = <&avb_pins>;
> +	pinctrl-names = "default";
> +	renesas,no-ether-link;
> +	renesas,phy = <0>;

    What's that for? :-)

> +	phy-reset-gpio = <&gpio2 10 0>;
> +	phy-handle = <&phy0>;
> +	status = "okay";
> +
> +	phy0: ethernet-phy@0 {
> +		rxc-skew-ps = <900>;
> +		rxdv-skew-ps = <0>;
> +		rxd0-skew-ps = <0>;
> +		rxd1-skew-ps = <0>;
> +		rxd2-skew-ps = <0>;
> +		rxd3-skew-ps = <0>;
> +		txc-skew-ps = <900>;
> +		txen-skew-ps = <0>;
> +		txd0-skew-ps = <0>;
> +		txd1-skew-ps = <0>;
> +		txd2-skew-ps = <0>;
> +		txd3-skew-ps = <0>;

    Hm, and the PHY driver can parse all this?

> +		reg = <0>;
> +		interrupt-parent = <&gpio2>;
> +		interrupts = <11 IRQ_TYPE_LEVEL_LOW>;
> +	};
> +
> +};
> diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> index 01502f4d2f5f..42c2eaa8167a 100644
> --- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
> @@ -372,4 +372,5 @@
>   			status = "disabled";
>   		};
>   	};
> +
>   };

    Stray newline?

MBR, Sergei


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH/RFC 07/10] arm64: dts: r8a7795: enable EthernetAVB on Salvator-X
  2015-08-27  9:24 [PATCH/RFC 07/10] arm64: dts: r8a7795: enable EthernetAVB on Salvator-X Simon Horman
  2015-08-27 10:40 ` Sergei Shtylyov
@ 2015-08-27 11:14 ` Geert Uytterhoeven
  2015-08-27 12:16 ` Sergei Shtylyov
                   ` (3 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Geert Uytterhoeven @ 2015-08-27 11:14 UTC (permalink / raw)
  To: linux-sh

On Thu, Aug 27, 2015 at 12:40 PM, Sergei Shtylyov
<sergei.shtylyov@cogentembedded.com> wrote:
>> +       phy0: ethernet-phy@0 {
>> +               rxc-skew-ps = <900>;
>> +               rxdv-skew-ps = <0>;
>> +               rxd0-skew-ps = <0>;
>> +               rxd1-skew-ps = <0>;
>> +               rxd2-skew-ps = <0>;
>> +               rxd3-skew-ps = <0>;
>> +               txc-skew-ps = <900>;
>> +               txen-skew-ps = <0>;
>> +               txd0-skew-ps = <0>;
>> +               txd1-skew-ps = <0>;
>> +               txd2-skew-ps = <0>;
>> +               txd3-skew-ps = <0>;
>
>
>    Hm, and the PHY driver can parse all this?

Please see drivers/net/phy/micrel.c:ksz9021_config_init().

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

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH/RFC 07/10] arm64: dts: r8a7795: enable EthernetAVB on Salvator-X
  2015-08-27  9:24 [PATCH/RFC 07/10] arm64: dts: r8a7795: enable EthernetAVB on Salvator-X Simon Horman
  2015-08-27 10:40 ` Sergei Shtylyov
  2015-08-27 11:14 ` Geert Uytterhoeven
@ 2015-08-27 12:16 ` Sergei Shtylyov
  2015-08-28  5:30 ` Simon Horman
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 7+ messages in thread
From: Sergei Shtylyov @ 2015-08-27 12:16 UTC (permalink / raw)
  To: linux-sh

On 8/27/2015 2:14 PM, Geert Uytterhoeven wrote:

>>> +       phy0: ethernet-phy@0 {
>>> +               rxc-skew-ps = <900>;
>>> +               rxdv-skew-ps = <0>;
>>> +               rxd0-skew-ps = <0>;
>>> +               rxd1-skew-ps = <0>;
>>> +               rxd2-skew-ps = <0>;
>>> +               rxd3-skew-ps = <0>;
>>> +               txc-skew-ps = <900>;
>>> +               txen-skew-ps = <0>;
>>> +               txd0-skew-ps = <0>;
>>> +               txd1-skew-ps = <0>;
>>> +               txd2-skew-ps = <0>;
>>> +               txd3-skew-ps = <0>;
>>
>>
>>     Hm, and the PHY driver can parse all this?

> Please see drivers/net/phy/micrel.c:ksz9021_config_init().

    OK, but don't we need to specify some kind of "compatible" prop?

> Gr{oetje,eeting}s,
>                          Geert

MBR, Sergei


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH/RFC 07/10] arm64: dts: r8a7795: enable EthernetAVB on Salvator-X
  2015-08-27  9:24 [PATCH/RFC 07/10] arm64: dts: r8a7795: enable EthernetAVB on Salvator-X Simon Horman
                   ` (2 preceding siblings ...)
  2015-08-27 12:16 ` Sergei Shtylyov
@ 2015-08-28  5:30 ` Simon Horman
  2015-08-28 16:50 ` Sergei Shtylyov
  2015-09-02  2:01 ` Simon Horman
  5 siblings, 0 replies; 7+ messages in thread
From: Simon Horman @ 2015-08-28  5:30 UTC (permalink / raw)
  To: linux-sh

On Thu, Aug 27, 2015 at 03:16:43PM +0300, Sergei Shtylyov wrote:
> On 8/27/2015 2:14 PM, Geert Uytterhoeven wrote:
> 
> >>>+       phy0: ethernet-phy@0 {
> >>>+               rxc-skew-ps = <900>;
> >>>+               rxdv-skew-ps = <0>;
> >>>+               rxd0-skew-ps = <0>;
> >>>+               rxd1-skew-ps = <0>;
> >>>+               rxd2-skew-ps = <0>;
> >>>+               rxd3-skew-ps = <0>;
> >>>+               txc-skew-ps = <900>;
> >>>+               txen-skew-ps = <0>;
> >>>+               txd0-skew-ps = <0>;
> >>>+               txd1-skew-ps = <0>;
> >>>+               txd2-skew-ps = <0>;
> >>>+               txd3-skew-ps = <0>;
> >>
> >>
> >>    Hm, and the PHY driver can parse all this?
> 
> >Please see drivers/net/phy/micrel.c:ksz9021_config_init().
> 
>    OK, but don't we need to specify some kind of "compatible" prop?

Not according to my reading of
Documentation/devicetree/bindings/net/micrel-ksz90x1.txt

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH/RFC 07/10] arm64: dts: r8a7795: enable EthernetAVB on Salvator-X
  2015-08-27  9:24 [PATCH/RFC 07/10] arm64: dts: r8a7795: enable EthernetAVB on Salvator-X Simon Horman
                   ` (3 preceding siblings ...)
  2015-08-28  5:30 ` Simon Horman
@ 2015-08-28 16:50 ` Sergei Shtylyov
  2015-09-02  2:01 ` Simon Horman
  5 siblings, 0 replies; 7+ messages in thread
From: Sergei Shtylyov @ 2015-08-28 16:50 UTC (permalink / raw)
  To: linux-sh

On 08/28/2015 08:30 AM, Simon Horman wrote:

>>>>> +       phy0: ethernet-phy@0 {
>>>>> +               rxc-skew-ps = <900>;
>>>>> +               rxdv-skew-ps = <0>;
>>>>> +               rxd0-skew-ps = <0>;
>>>>> +               rxd1-skew-ps = <0>;
>>>>> +               rxd2-skew-ps = <0>;
>>>>> +               rxd3-skew-ps = <0>;
>>>>> +               txc-skew-ps = <900>;
>>>>> +               txen-skew-ps = <0>;
>>>>> +               txd0-skew-ps = <0>;
>>>>> +               txd1-skew-ps = <0>;
>>>>> +               txd2-skew-ps = <0>;
>>>>> +               txd3-skew-ps = <0>;
>>>>
>>>>
>>>>     Hm, and the PHY driver can parse all this?
>>
>>> Please see drivers/net/phy/micrel.c:ksz9021_config_init().
>>
>>     OK, but don't we need to specify some kind of "compatible" prop?
>
> Not according to my reading of
> Documentation/devicetree/bindings/net/micrel-ksz90x1.txt

    Hm... I wonder why these props have generic names if the registers 
programmed with their values are clearly vendor-unique... These bindings don't 
look good to me... :-/

MBR, Sergei


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH/RFC 07/10] arm64: dts: r8a7795: enable EthernetAVB on Salvator-X
  2015-08-27  9:24 [PATCH/RFC 07/10] arm64: dts: r8a7795: enable EthernetAVB on Salvator-X Simon Horman
                   ` (4 preceding siblings ...)
  2015-08-28 16:50 ` Sergei Shtylyov
@ 2015-09-02  2:01 ` Simon Horman
  5 siblings, 0 replies; 7+ messages in thread
From: Simon Horman @ 2015-09-02  2:01 UTC (permalink / raw)
  To: linux-sh

On Thu, Aug 27, 2015 at 01:40:01PM +0300, Sergei Shtylyov wrote:
> Hello.
> 
> On 8/27/2015 12:24 PM, Simon Horman wrote:
> 
> >From: Kazuya Mizuguchi <kazuya.mizuguchi.ks@renesas.com>
> >
> >Signed-off-by: Kazuya Mizuguchi <kazuya.mizuguchi.ks@renesas.com>
> >[horms: updated filename to salvator-x; updated changelog]
> >Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> >---
> >  arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 34 ++++++++++++++++++++++
> >  arch/arm64/boot/dts/renesas/r8a7795.dtsi           |  1 +
> >  2 files changed, 35 insertions(+)
> >
> >diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> >index 88cc34819e6a..e8d940cc831a 100644
> >--- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> >+++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> [...]
> >@@ -103,3 +108,32 @@
> >  &ohci2 {
> >  	status = "okay";
> >  };
> >+
> >+&avb {
> >+	pinctrl-0 = <&avb_pins>;
> >+	pinctrl-names = "default";
> >+	renesas,no-ether-link;
> >+	renesas,phy = <0>;
> 
>    What's that for? :-)

Thanks for spotting that, I'll remove the renesas,phy property
as I can see anything that is using it.

[snip]

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2015-09-02  2:01 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-27  9:24 [PATCH/RFC 07/10] arm64: dts: r8a7795: enable EthernetAVB on Salvator-X Simon Horman
2015-08-27 10:40 ` Sergei Shtylyov
2015-08-27 11:14 ` Geert Uytterhoeven
2015-08-27 12:16 ` Sergei Shtylyov
2015-08-28  5:30 ` Simon Horman
2015-08-28 16:50 ` Sergei Shtylyov
2015-09-02  2:01 ` Simon Horman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).