diff mbox

arm64: renesas: salvator-x: Setup ethernet0 alias for U-Boot

Message ID 1446197120-4640-1-git-send-email-geert+renesas@glider.be (mailing list archive)
State Superseded
Delegated to: Simon Horman
Headers show

Commit Message

Geert Uytterhoeven Oct. 30, 2015, 9:25 a.m. UTC
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@glider.be>
---
 arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 1 +
 1 file changed, 1 insertion(+)

Comments

Laurent Pinchart Oct. 30, 2015, 1:06 p.m. UTC | #1
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 {
Geert Uytterhoeven Oct. 30, 2015, 1:16 p.m. UTC | #2
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
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Laurent Pinchart Oct. 30, 2015, 1:37 p.m. UTC | #3
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>
Simon Horman Nov. 2, 2015, 2:14 a.m. UTC | #4
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.
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

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 {