From: andrew@lunn.ch (Andrew Lunn)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: dts: imx: add Gateworks Ventana GW5904 support
Date: Mon, 13 Mar 2017 14:36:41 +0100 [thread overview]
Message-ID: <20170313133641.GC748@lunn.ch> (raw)
In-Reply-To: <CAJ+vNU0JNQuQEmG_oCu86cbG_wisVFRETNtCO1bC2ytMM+N2Vg@mail.gmail.com>
> Thanks for the hint on the compatibility. I wonder if
> Documentation/devicetree/bindings/net/dsa/marvell.txt should have some
> notes added about switch device compatibility? It wasn't clear to me
> that these would be compatible.
Hi Tim
This keeps coming up, so i plan on submitting a patch. I thought it
would be obvious. The driver supports over 20 Marvell switches, yet
only has two compatible strings. It is not too big a leap to figure
out they must all be compatible, or there would be 20 compatible
strings....
> What is the convention, if any, of the naming of the 'lan' ports (0
> based vs 1 based, eth vs lan)? Is it strictly up to the board vendor?
> The board I'm supporting has a silkscreen that shows 'Port 1' through
> 'Port 4' so I was leaning towards 1 based.
I recommend following what the label on the housing says. If you don't
have a housing, just a board, follow the silk screen, and hope the
designer of the housing also follows the silk screen.
Andrew
WARNING: multiple messages have this Message-ID (diff)
From: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>
To: Tim Harvey <tharvey-UMMOYl/HMS+akBO8gow8eQ@public.gmane.org>
Cc: Florian Fainelli
<f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
Shawn Guo <shawnguo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org"
<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>
Subject: Re: [PATCH] ARM: dts: imx: add Gateworks Ventana GW5904 support
Date: Mon, 13 Mar 2017 14:36:41 +0100 [thread overview]
Message-ID: <20170313133641.GC748@lunn.ch> (raw)
In-Reply-To: <CAJ+vNU0JNQuQEmG_oCu86cbG_wisVFRETNtCO1bC2ytMM+N2Vg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
> Thanks for the hint on the compatibility. I wonder if
> Documentation/devicetree/bindings/net/dsa/marvell.txt should have some
> notes added about switch device compatibility? It wasn't clear to me
> that these would be compatible.
Hi Tim
This keeps coming up, so i plan on submitting a patch. I thought it
would be obvious. The driver supports over 20 Marvell switches, yet
only has two compatible strings. It is not too big a leap to figure
out they must all be compatible, or there would be 20 compatible
strings....
> What is the convention, if any, of the naming of the 'lan' ports (0
> based vs 1 based, eth vs lan)? Is it strictly up to the board vendor?
> The board I'm supporting has a silkscreen that shows 'Port 1' through
> 'Port 4' so I was leaning towards 1 based.
I recommend following what the label on the housing says. If you don't
have a housing, just a board, follow the silk screen, and hope the
designer of the housing also follows the silk screen.
Andrew
--
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
next prev parent reply other threads:[~2017-03-13 13:36 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-10 20:40 [PATCH] ARM: dts: imx: add Gateworks Ventana GW5904 support Tim Harvey
2017-03-10 20:40 ` Tim Harvey
2017-03-10 20:43 ` Florian Fainelli
2017-03-10 20:43 ` Florian Fainelli
2017-03-11 1:07 ` Tim Harvey
2017-03-11 1:07 ` Tim Harvey
2017-03-11 1:20 ` Florian Fainelli
2017-03-11 1:20 ` Florian Fainelli
2017-03-11 3:04 ` Andrew Lunn
2017-03-11 3:04 ` Andrew Lunn
2017-03-13 13:27 ` Tim Harvey
2017-03-13 13:27 ` Tim Harvey
2017-03-13 13:36 ` Andrew Lunn [this message]
2017-03-13 13:36 ` Andrew Lunn
2017-03-13 13:20 ` Tim Harvey
2017-03-13 13:20 ` Tim Harvey
2017-03-13 13:28 ` Andrew Lunn
2017-03-13 13:28 ` Andrew Lunn
2017-03-14 10:55 ` Shawn Guo
2017-03-14 10:55 ` Shawn Guo
2017-03-14 15:05 ` Tim Harvey
2017-03-14 15:05 ` Tim Harvey
2017-03-15 6:31 ` Shawn Guo
2017-03-15 6:31 ` Shawn Guo
2017-03-15 15:13 ` [PATCH v2] " Tim Harvey
2017-03-15 15:13 ` Tim Harvey
2017-03-16 1:28 ` Shawn Guo
2017-03-16 1:28 ` Shawn Guo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170313133641.GC748@lunn.ch \
--to=andrew@lunn.ch \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.