> + > +/ { > + model = "Thecus N2350"; > + compatible = "marvell,armada385"; If you look at all the other armarda-385 .dts files, they all have a board compatible. It might not be needed now, but it could be useful in the future and allow backwards compatibility. > +&mdio { > + phy0: ethernet-phy@0 { > + reg = <1>; > + }; > +}; Since you have ref = <1>, this should be ethernet-phy@1.