On Monday 09 November 2015 17:08:34 Russell King - ARM Linux wrote: > They are "optional" because when you're using a DSA switch, you don't > specify a PHY (because, there isn't one). For example, this is what > I'm using with an Armada 388 board with a Marvell DSA switch. The > DSA does not appear as a PHY, and no node in the DSA stanza can be > referenced for a phy entry in the ethernet device's stanza. > > eth1: ethernet@30000 { > compatible = "marvell,armada-370-neta"; > reg = <0x30000 0x4000>; > interrupts-extended = <&mpic 10>; > clocks = <&gateclk 3>; > managed = "in-band-status"; > phy-mode = "sgmii"; > status = "okay"; > }; > >
Ok, then it would be nice to change the binding to reflect that, and also document the "managed" property there. Arnd -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html