diff mbox

[2/2] ARM: dts: iwg23s-sbc: Add EtherAVB support

Message ID 1524584253-52936-3-git-send-email-biju.das@bp.renesas.com (mailing list archive)
State Accepted
Commit b6ef074bcadf9f89295bc7ee19424714b6ffc4c8
Delegated to: Simon Horman
Headers show

Commit Message

Biju Das April 24, 2018, 3:37 p.m. UTC 
Define the iW-RainboW-G23S board dependent part of the
EtherAVB device node.

Signed-off-by: Biju Das <biju.das@bp.renesas.com>
Reviewed-by: Chris Paterson <chris.paterson2@renesas.com>
---
* currently u-boot sets the AVB pin configuration.

 arch/arm/boot/dts/r8a77470-iwg23s-sbc.dts | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

Comments

Simon Horman April 30, 2018, 8 a.m. UTC | #1 
On Tue, Apr 24, 2018 at 04:37:33PM +0100, Biju Das wrote:
> Define the iW-RainboW-G23S board dependent part of the
> EtherAVB device node.
> 
> Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> Reviewed-by: Chris Paterson <chris.paterson2@renesas.com>

Thanks, applied.

> ---
> * currently u-boot sets the AVB pin configuration.

Understood, I assume at some point the pins will be described in DT.
Biju Das April 30, 2018, 8:14 a.m. UTC | #2 
Hi Simon,

> -----Original Message-----
> From: Simon Horman [mailto:horms@verge.net.au]
> Sent: 30 April 2018 09:00
> To: Biju Das <biju.das@bp.renesas.com>
> Cc: Rob Herring <robh+dt@kernel.org>; Mark Rutland
> <mark.rutland@arm.com>; Magnus Damm <magnus.damm@gmail.com>;
> Geert Uytterhoeven <geert+renesas@glider.be>; Chris Paterson
> <Chris.Paterson2@renesas.com>; Fabrizio Castro
> <fabrizio.castro@bp.renesas.com>; devicetree@vger.kernel.org; linux-
> renesas-soc@vger.kernel.org
> Subject: Re: [PATCH 2/2] ARM: dts: iwg23s-sbc: Add EtherAVB support
>
> On Tue, Apr 24, 2018 at 04:37:33PM +0100, Biju Das wrote:
> > Define the iW-RainboW-G23S board dependent part of the EtherAVB
> device
> > node.
> >
> > Signed-off-by: Biju Das <biju.das@bp.renesas.com>
> > Reviewed-by: Chris Paterson <chris.paterson2@renesas.com>
>
> Thanks, applied.
>
> > ---
> > * currently u-boot sets the AVB pin configuration.
>
> Understood, I assume at some point the pins will be described in DT.

Yes that is correct.  Will submit after the r8a77470 PFC driver acceptance.



Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
diff mbox

Patch

diff --git a/arch/arm/boot/dts/r8a77470-iwg23s-sbc.dts b/arch/arm/boot/dts/r8a77470-iwg23s-sbc.dts
index d21baad..e3585da 100644
--- a/arch/arm/boot/dts/r8a77470-iwg23s-sbc.dts
+++ b/arch/arm/boot/dts/r8a77470-iwg23s-sbc.dts
@@ -12,11 +12,12 @@ 
 	compatible = "iwave,g23s", "renesas,r8a77470";
 
 	aliases {
+		ethernet0 = &avb;
 		serial1 = &scif1;
 	};
 
 	chosen {
-		bootargs = "ignore_loglevel";
+		bootargs = "ignore_loglevel rw root=/dev/nfs ip=dhcp";
 		stdout-path = "serial1:115200n8";
 	};
 
@@ -26,6 +27,18 @@ 
 	};
 };
 
+&avb {
+	phy-handle = <&phy3>;
+	phy-mode = "gmii";
+	renesas,no-ether-link;
+	status = "okay";
+
+	phy3: ethernet-phy@3 {
+		reg = <3>;
+		micrel,led-mode = <1>;
+	};
+};
+
 &extal_clk {
 	clock-frequency = <20000000>;
 };