devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] arm64: renesas: salvator-x: Setup ethernet0 alias for U-Boot
@ 2015-10-30  9:25 Geert Uytterhoeven
  2015-10-30 13:06 ` Laurent Pinchart
  0 siblings, 1 reply; 5+ messages in thread
From: Geert Uytterhoeven @ 2015-10-30  9:25 UTC (permalink / raw)
  To: Simon Horman, Magnus Damm
  Cc: Laurent Pinchart, Wolfram Sang, linux-sh-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Geert Uytterhoeven

Add an ethernet0 alias for the wired network card so that U-Boot can
find the device-node and add an appropriate "local-mac-address"
property.

Signed-off-by: Geert Uytterhoeven <geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
---
 arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
index 86b70ce3d0599d6e..d16506338f020427 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
@@ -41,6 +41,7 @@
 	aliases {
 		serial0 = &scif2;
 		serial1 = &scif1;
+		ethernet0 = &avb;
 	};
 
 	chosen {
-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* Re: [PATCH] arm64: renesas: salvator-x: Setup ethernet0 alias for U-Boot
  2015-10-30  9:25 [PATCH] arm64: renesas: salvator-x: Setup ethernet0 alias for U-Boot Geert Uytterhoeven
@ 2015-10-30 13:06 ` Laurent Pinchart
  2015-10-30 13:16   ` Geert Uytterhoeven
  0 siblings, 1 reply; 5+ messages in thread
From: Laurent Pinchart @ 2015-10-30 13:06 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Simon Horman, Magnus Damm, Wolfram Sang, linux-sh, devicetree

Hi Geert,

Thank you for the patch.

On Friday 30 October 2015 10:25:20 Geert Uytterhoeven wrote:
> Add an ethernet0 alias for the wired network card so that U-Boot can
> find the device-node and add an appropriate "local-mac-address"
> property.

I like that :-) Can U-Boot add the property ? I thought it needed a 
placeholder.

> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
>  arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts index
> 86b70ce3d0599d6e..d16506338f020427 100644
> --- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> +++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
> @@ -41,6 +41,7 @@
>  	aliases {
>  		serial0 = &scif2;
>  		serial1 = &scif1;
> +		ethernet0 = &avb;
>  	};
> 
>  	chosen {

-- 
Regards,

Laurent Pinchart


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

* Re: [PATCH] arm64: renesas: salvator-x: Setup ethernet0 alias for U-Boot
  2015-10-30 13:06 ` Laurent Pinchart
@ 2015-10-30 13:16   ` Geert Uytterhoeven
  2015-10-30 13:37     ` Laurent Pinchart
  0 siblings, 1 reply; 5+ messages in thread
From: Geert Uytterhoeven @ 2015-10-30 13:16 UTC (permalink / raw)
  To: Laurent Pinchart
  Cc: Geert Uytterhoeven, Simon Horman, Magnus Damm, Wolfram Sang,
	Linux-sh list, devicetree@vger.kernel.org

On Fri, Oct 30, 2015 at 2:06 PM, Laurent Pinchart
<laurent.pinchart@ideasonboard.com> wrote:
> On Friday 30 October 2015 10:25:20 Geert Uytterhoeven wrote:
>> Add an ethernet0 alias for the wired network card so that U-Boot can
>> find the device-node and add an appropriate "local-mac-address"
>> property.
>
> I like that :-) Can U-Boot add the property ? I thought it needed a
> placeholder.

At least for Salvator-X it does. See also the thread at
http://www.spinics.net/lists/arm-kernel/msg456841.html

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] 5+ messages in thread

* Re: [PATCH] arm64: renesas: salvator-x: Setup ethernet0 alias for U-Boot
  2015-10-30 13:16   ` Geert Uytterhoeven
@ 2015-10-30 13:37     ` Laurent Pinchart
  2015-11-02  2:14       ` Simon Horman
  0 siblings, 1 reply; 5+ messages in thread
From: Laurent Pinchart @ 2015-10-30 13:37 UTC (permalink / raw)
  To: Geert Uytterhoeven
  Cc: Geert Uytterhoeven, Simon Horman, Magnus Damm, Wolfram Sang,
	Linux-sh list, devicetree@vger.kernel.org

Hi Geert,

On Friday 30 October 2015 14:16:03 Geert Uytterhoeven wrote:
> On Fri, Oct 30, 2015 at 2:06 PM, Laurent Pinchart wrote:
> > On Friday 30 October 2015 10:25:20 Geert Uytterhoeven wrote:
> >> Add an ethernet0 alias for the wired network card so that U-Boot can
> >> find the device-node and add an appropriate "local-mac-address"
> >> property.
> > 
> > I like that :-) Can U-Boot add the property ? I thought it needed a
> > placeholder.
> 
> At least for Salvator-X it does. See also the thread at
> http://www.spinics.net/lists/arm-kernel/msg456841.html

Cool :-)

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

-- 
Regards,

Laurent Pinchart


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

* Re: [PATCH] arm64: renesas: salvator-x: Setup ethernet0 alias for U-Boot
  2015-10-30 13:37     ` Laurent Pinchart
@ 2015-11-02  2:14       ` Simon Horman
  0 siblings, 0 replies; 5+ messages in thread
From: Simon Horman @ 2015-11-02  2:14 UTC (permalink / raw)
  To: Laurent Pinchart
  Cc: Geert Uytterhoeven, Geert Uytterhoeven, Magnus Damm, Wolfram Sang,
	Linux-sh list, devicetree@vger.kernel.org

On Fri, Oct 30, 2015 at 03:37:26PM +0200, Laurent Pinchart wrote:
> Hi Geert,
> 
> On Friday 30 October 2015 14:16:03 Geert Uytterhoeven wrote:
> > On Fri, Oct 30, 2015 at 2:06 PM, Laurent Pinchart wrote:
> > > On Friday 30 October 2015 10:25:20 Geert Uytterhoeven wrote:
> > >> Add an ethernet0 alias for the wired network card so that U-Boot can
> > >> find the device-node and add an appropriate "local-mac-address"
> > >> property.
> > > 
> > > I like that :-) Can U-Boot add the property ? I thought it needed a
> > > placeholder.
> > 
> > At least for Salvator-X it does. See also the thread at
> > http://www.spinics.net/lists/arm-kernel/msg456841.html
> 
> Cool :-)
> 
> Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

Thanks, I will include this in the next spin of the ravb integration
patches for r8a7795/salvator-x.

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

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

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-30  9:25 [PATCH] arm64: renesas: salvator-x: Setup ethernet0 alias for U-Boot Geert Uytterhoeven
2015-10-30 13:06 ` Laurent Pinchart
2015-10-30 13:16   ` Geert Uytterhoeven
2015-10-30 13:37     ` Laurent Pinchart
2015-11-02  2:14       ` 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).