Feedback on Device tree for Marvell 6176 using 'new' binding

Please give feed back on device tree description for

This node in the device tree with CONFIG_NET_DSA_LEGACY enabled in kernel correctly enumerates all interfaces on the Marvell 6176 switch.

dsa@0 {
                compatible = "marvell,mv88e6085";
                #address-cells = <2>;
                #size-cells = <0>;
                interrupt-parent = <&gpio2>;
                interrupts = <&gpio2 31 IRQ_TYPE_LEVEL_LOW>;
                dsa,ethernet = <&eth0>;

                switch@0 {
                        #address-cells = <1>;
                        #size-cells = <0>;
                        reg = <0 0>;

                        port@0 {
                                reg = <0>;
                                label = "port0";
                        };

                        port@1 {
                                reg = <1>;
                                label = "port1";
                        };

                        port@2 {
                                reg = <2>;
                                label = "port2";
                        };

                        port@5 {
                                reg = <5>;
                                label = "cpu";
                        };
                };
        };

BUT this attempt with the new binding causes 4.19 kernel to NOT boot.

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

        mdio: mdio {
                #address-cells = <1>;
                #size-cells = <0>;
                status = "okay";
        };
};

mdio {
                  #address-cells = <1>;
                  #size-cells = <0>;
                  interrupt-parent = <&gpio2>;
                  interrupts = <&gpio2 31 IRQ_TYPE_LEVEL_LOW>;
                  

                  switch: switch@0 {
                        compatible = "marvell,mv88e6085";                       
                        reg = <0>;
                        dsa,member = <0 0>;
                        
                        ports{
                                #address-cells = <1>;
                                #size-cells = <0>;
                                port@0 {
                                        reg = <0>;
                                        label = "port0";
                                };

                                port@1 {
                                        reg = <1>;
                                        label = "port1";
                                };

                                port@2 {
                                        reg = <2>;
                                        label = "port2";
                                };

                                port@5 {
                                        reg = <5>;
                                        label = "cpu"; 
                                        ethernet =<&fec>; 
                              };
                           };
                    };
                 };