Re: [PATCH v4 13/13] ARM: dts: kswitch-d10: enable networking

[Claudiu.Beznea]

On 03.05.2022 01:41, Michael Walle wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> Enable all the necessary network related nodes, wire the pinctrl
> configurations, add the PHYs and connect them to the corresponding
> network ports.
> 
> Signed-off-by: Michael Walle <michael@xxxxxxxx>

Reviewed-by: Claudiu Beznea <claudiu.beznea@xxxxxxxxxxxxx>


> ---
>  ...lan966x-kontron-kswitch-d10-mmt-6g-2gs.dts | 16 +++
>  .../lan966x-kontron-kswitch-d10-mmt-8g.dts    | 26 +++++
>  .../dts/lan966x-kontron-kswitch-d10-mmt.dtsi  | 97 +++++++++++++++++++
>  3 files changed, 139 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-6g-2gs.dts b/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-6g-2gs.dts
> index 7b12cbe11c58..0f555eb45bda 100644
> --- a/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-6g-2gs.dts
> +++ b/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-6g-2gs.dts
> @@ -76,3 +76,19 @@ fc4_b_pins: fc4-b-i2c-pins {
>                 function = "fc4_b";
>         };
>  };
> +
> +&port2 {
> +       phys = <&serdes 2 SERDES6G(0)>;
> +       sfp = <&sfp0>;
> +       managed = "in-band-status";
> +       phy-mode = "sgmii";
> +       status = "okay";
> +};
> +
> +&port3 {
> +       phys = <&serdes 3 SERDES6G(1)>;
> +       sfp = <&sfp1>;
> +       managed = "in-band-status";
> +       phy-mode = "sgmii";
> +       status = "okay";
> +};
> diff --git a/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-8g.dts b/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-8g.dts
> index 4b35f6c46e7f..5feef9a59a79 100644
> --- a/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-8g.dts
> +++ b/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt-8g.dts
> @@ -11,3 +11,29 @@ / {
>         compatible = "kontron,kswitch-d10-mmt-8g", "kontron,s1921",
>                      "microchip,lan9668", "microchip,lan966";
>  };
> +
> +&mdio0 {
> +       phy2: ethernet-phy@3 {
> +               reg = <3>;
> +       };
> +
> +       phy3: ethernet-phy@4 {
> +               reg = <4>;
> +       };
> +};
> +
> +&port2 {
> +       phys = <&serdes 2 SERDES6G(0)>;
> +       phy-handle = <&phy2>;
> +       phy-mode = "sgmii";
> +       managed = "in-band-status";
> +       status = "okay";
> +};
> +
> +&port3 {
> +       phys = <&serdes 3 SERDES6G(1)>;
> +       phy-handle = <&phy3>;
> +       phy-mode = "sgmii";
> +       managed = "in-band-status";
> +       status = "okay";
> +};
> diff --git a/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi b/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi
> index 4c1ebb4aa5b0..4cab1b3b3b29 100644
> --- a/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi
> +++ b/arch/arm/boot/dts/lan966x-kontron-kswitch-d10-mmt.dtsi
> @@ -5,6 +5,7 @@
> 
>  /dts-v1/;
>  #include "lan966x.dtsi"
> +#include "dt-bindings/phy/phy-lan966x-serdes.h"
> 
>  / {
>         aliases {
> @@ -52,6 +53,12 @@ fc3_b_pins: fc3-b-pins {
>                 function = "fc3_b";
>         };
> 
> +       miim_c_pins: miim-c-pins {
> +               /* MDC, MDIO */
> +               pins = "GPIO_59", "GPIO_60";
> +               function = "miim_c";
> +       };
> +
>         sgpio_a_pins: sgpio-a-pins {
>                 /* SCK, D0, D1 */
>                 pins = "GPIO_32", "GPIO_33", "GPIO_34";
> @@ -71,6 +78,92 @@ usart0_pins: usart0-pins {
>         };
>  };
> 
> +&mdio0 {
> +       pinctrl-0 = <&miim_c_pins>;
> +       pinctrl-names = "default";
> +       reset-gpios = <&gpio 29 GPIO_ACTIVE_LOW>;
> +       clock-frequency = <2500000>;
> +       status = "okay";
> +
> +       phy4: ethernet-phy@5 {
> +               reg = <5>;
> +               coma-mode-gpios = <&gpio 37 GPIO_ACTIVE_HIGH>;
> +       };
> +
> +       phy5: ethernet-phy@6 {
> +               reg = <6>;
> +               coma-mode-gpios = <&gpio 37 GPIO_ACTIVE_HIGH>;
> +       };
> +
> +       phy6: ethernet-phy@7 {
> +               reg = <7>;
> +               coma-mode-gpios = <&gpio 37 GPIO_ACTIVE_HIGH>;
> +       };
> +
> +       phy7: ethernet-phy@8 {
> +               reg = <8>;
> +               coma-mode-gpios = <&gpio 37 GPIO_ACTIVE_HIGH>;
> +       };
> +};
> +
> +&mdio1 {
> +       status = "okay";
> +};
> +
> +&phy0 {
> +       status = "okay";
> +};
> +
> +&phy1 {
> +       status = "okay";
> +};
> +
> +&port0 {
> +       phys = <&serdes 0 CU(0)>;
> +       phy-handle = <&phy0>;
> +       phy-mode = "gmii";
> +       status = "okay";
> +};
> +
> +&port1 {
> +       phys = <&serdes 1 CU(1)>;
> +       phy-handle = <&phy1>;
> +       phy-mode = "gmii";
> +       status = "okay";
> +};
> +
> +&port4 {
> +       phys = <&serdes 4 SERDES6G(2)>;
> +       phy-handle = <&phy4>;
> +       phy-mode = "qsgmii";
> +       status = "okay";
> +};
> +
> +&port5 {
> +       phys = <&serdes 5 SERDES6G(2)>;
> +       phy-handle = <&phy5>;
> +       phy-mode = "qsgmii";
> +       status = "okay";
> +};
> +
> +&port6 {
> +       phys = <&serdes 6 SERDES6G(2)>;
> +       phy-handle = <&phy6>;
> +       phy-mode = "qsgmii";
> +       status = "okay";
> +};
> +
> +&port7 {
> +       phys = <&serdes 7 SERDES6G(2)>;
> +       phy-handle = <&phy7>;
> +       phy-mode = "qsgmii";
> +       status = "okay";
> +};
> +
> +&serdes {
> +       status = "okay";
> +};
> +
>  &sgpio {
>         pinctrl-0 = <&sgpio_a_pins>, <&sgpio_b_pins>;
>         pinctrl-names = "default";
> @@ -88,6 +181,10 @@ sgpio_out: gpio@1 {
>         };
>  };
> 
> +&switch {
> +       status = "okay";
> +};
> +
>  &watchdog {
>         status = "okay";
>  };
> --
> 2.30.2
>