[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <b8343fc3-3ea6-12e2-576f-2c5a39897a1d@microchip.com>
Date: Mon, 9 May 2022 06:27:17 +0000
From: <Claudiu.Beznea@...rochip.com>
To: <michael@...le.cc>, <Kavyasree.Kotagiri@...rochip.com>,
<Nicolas.Ferre@...rochip.com>
CC: <arnd@...db.de>, <olof@...om.net>, <soc@...nel.org>,
<linux-arm-kernel@...ts.infradead.org>,
<devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
<robh+dt@...nel.org>, <krzysztof.kozlowski@...onical.com>,
<alexandre.belloni@...tlin.com>, <Tudor.Ambarus@...rochip.com>,
<Horatiu.Vultur@...rochip.com>
Subject: Re: [PATCH v4 13/13] ARM: dts: kswitch-d10: enable networking
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@...le.cc>
Reviewed-by: Claudiu Beznea <claudiu.beznea@...rochip.com>
> ---
> ...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
>
Powered by blists - more mailing lists