[PATCH] ARM: dts: imx: add Gateworks Ventana GW5904 support

On Fri, Mar 10, 2017 at 7:04 PM, Andrew Lunn [off-list ref] wrote:

On Fri, Mar 10, 2017 at 05:20:11PM -0800, Florian Fainelli wrote:

quoted

On 03/10/2017 05:07 PM, Tim Harvey wrote:

quoted

On Fri, Mar 10, 2017 at 12:43 PM, Florian Fainelli [off-list ref] wrote:

quoted

quoted

+     dsa {
+             compatible = "marvell,dsa";
+             #address-cells = <2>;
+             #size-cells = <0>;
+
+             dsa,ethernet = <&fec>;
+             dsa,mii-bus = <&mdio>;
+

Please consider using the new binding for DSA switches, see:

https://patchwork.kernel.org/patch/9493037/

Hi Florian,

I tried the new binding first, but haven't gotten it to work yet. Let
me make sure I understand what I should be doing.

This is a MV88E6176 so I first need to add dt support for that to the
mv88e6xxx driver (which supports the device, just not via dt):

Hi Tim

Nope. The 6176 is compatible with the 6085.

quoted

quoted

Then I remove the old binding and add the new binding as such:

&fec {
        pinctrl-names = "default";
        pinctrl-0 = <&pinctrl_enet>;
        phy-mode = "rgmii-id";
        status = "okay";

You most likely need to declare a fixed PHY for the FEC to be setting up
the link, speed and duplex?

Take a look at arch/arm/boot/dts/vf610-zii-dev* as an example. This
uses a vf610, so is somewhat similar to the imx6. Same fec driver, but
the mdio is more complex due to there being 3 switches.

     Andrew

Andrew,

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.

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.

Thanks,

Tim