[PATCH net-next] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
Camelia Alexandra Groza
camelia.groza at nxp.com
Fri Dec 16 03:12:02 AEDT 2022
> -----Original Message-----
> From: Sean Anderson <sean.anderson at seco.com>
> Sent: Thursday, November 3, 2022 23:29
> To: David S . Miller <davem at davemloft.net>; netdev at vger.kernel.org
> Cc: devicetree at vger.kernel.org; Michael Ellerman <mpe at ellerman.id.au>;
> linux-kernel at vger.kernel.org; Rob Herring <robh+dt at kernel.org>;
> Christophe Leroy <christophe.leroy at csgroup.eu>; linuxppc-
> dev at lists.ozlabs.org; Nicholas Piggin <npiggin at gmail.com>; Krzysztof
> Kozlowski <krzysztof.kozlowski+dt at linaro.org>; Sean Anderson
> <sean.anderson at seco.com>; Camelia Alexandra Groza
> <camelia.groza at nxp.com>
> Subject: [PATCH net-next] powerpc: dts: t208x: Disable 10G on MAC1 and
> MAC2
>
> There aren't enough resources to run these ports at 10G speeds. Just
> keep the pcs changes, and revert the rest. This is not really correct,
> since the hardware could support 10g in some other configuration...
>
> Fixes: 36926a7d70c2 ("powerpc: dts: t208x: Mark MAC1 and MAC2 as 10G")
> Reported-by: Camelia Alexandra Groza <camelia.groza at nxp.com>
> Signed-off-by: Sean Anderson <sean.anderson at seco.com>
> ---
>
Hi Sean,
I know I'm late, but there are a couple of issues with this patch. Do you intend
on sending a v2 or should I pick it up?
> .../boot/dts/fsl/qoriq-fman3-0-10g-2.dtsi | 45 -------------------
> .../boot/dts/fsl/qoriq-fman3-0-10g-3.dtsi | 45 -------------------
> arch/powerpc/boot/dts/fsl/t2081si-post.dtsi | 6 ++-
> 3 files changed, 4 insertions(+), 92 deletions(-)
> delete mode 100644 arch/powerpc/boot/dts/fsl/qoriq-fman3-0-10g-2.dtsi
> delete mode 100644 arch/powerpc/boot/dts/fsl/qoriq-fman3-0-10g-3.dtsi
>
> diff --git a/arch/powerpc/boot/dts/fsl/qoriq-fman3-0-10g-2.dtsi
> b/arch/powerpc/boot/dts/fsl/qoriq-fman3-0-10g-2.dtsi
> deleted file mode 100644
> index 6b3609574b0f..000000000000
> --- a/arch/powerpc/boot/dts/fsl/qoriq-fman3-0-10g-2.dtsi
> +++ /dev/null
> @@ -1,45 +0,0 @@
> -// SPDX-License-Identifier: BSD-3-Clause OR GPL-2.0-or-later
> -/*
> - * QorIQ FMan v3 10g port #2 device tree stub [ controller @ offset
> 0x400000 ]
> - *
> - * Copyright 2022 Sean Anderson <sean.anderson at seco.com>
> - * Copyright 2012 - 2015 Freescale Semiconductor Inc.
> - */
> -
> -fman at 400000 {
> - fman0_rx_0x08: port at 88000 {
> - cell-index = <0x8>;
> - compatible = "fsl,fman-v3-port-rx";
> - reg = <0x88000 0x1000>;
> - fsl,fman-10g-port;
> - };
> -
> - fman0_tx_0x28: port at a8000 {
> - cell-index = <0x28>;
> - compatible = "fsl,fman-v3-port-tx";
> - reg = <0xa8000 0x1000>;
> - fsl,fman-10g-port;
> - };
> -
> - ethernet at e0000 {
> - cell-index = <0>;
> - compatible = "fsl,fman-memac";
> - reg = <0xe0000 0x1000>;
> - fsl,fman-ports = <&fman0_rx_0x08 &fman0_tx_0x28>;
> - ptp-timer = <&ptp_timer0>;
> - pcsphy-handle = <&pcsphy0>, <&pcsphy0>;
> - pcs-handle-names = "sgmii", "xfi";
> - };
> -
> - mdio at e1000 {
> - #address-cells = <1>;
> - #size-cells = <0>;
> - compatible = "fsl,fman-memac-mdio", "fsl,fman-xmdio";
> - reg = <0xe1000 0x1000>;
> - fsl,erratum-a011043; /* must ignore read errors */
> -
> - pcsphy0: ethernet-phy at 0 {
> - reg = <0x0>;
> - };
> - };
> -};
> diff --git a/arch/powerpc/boot/dts/fsl/qoriq-fman3-0-10g-3.dtsi
> b/arch/powerpc/boot/dts/fsl/qoriq-fman3-0-10g-3.dtsi
> deleted file mode 100644
> index 28ed1a85a436..000000000000
> --- a/arch/powerpc/boot/dts/fsl/qoriq-fman3-0-10g-3.dtsi
> +++ /dev/null
> @@ -1,45 +0,0 @@
> -// SPDX-License-Identifier: BSD-3-Clause OR GPL-2.0-or-later
> -/*
> - * QorIQ FMan v3 10g port #3 device tree stub [ controller @ offset
> 0x400000 ]
> - *
> - * Copyright 2022 Sean Anderson <sean.anderson at seco.com>
> - * Copyright 2012 - 2015 Freescale Semiconductor Inc.
> - */
> -
> -fman at 400000 {
> - fman0_rx_0x09: port at 89000 {
> - cell-index = <0x9>;
> - compatible = "fsl,fman-v3-port-rx";
> - reg = <0x89000 0x1000>;
> - fsl,fman-10g-port;
> - };
> -
> - fman0_tx_0x29: port at a9000 {
> - cell-index = <0x29>;
> - compatible = "fsl,fman-v3-port-tx";
> - reg = <0xa9000 0x1000>;
> - fsl,fman-10g-port;
> - };
> -
> - ethernet at e2000 {
> - cell-index = <1>;
> - compatible = "fsl,fman-memac";
> - reg = <0xe2000 0x1000>;
> - fsl,fman-ports = <&fman0_rx_0x09 &fman0_tx_0x29>;
> - ptp-timer = <&ptp_timer0>;
> - pcsphy-handle = <&pcsphy1>, <&pcsphy1>;
> - pcs-handle-names = "sgmii", "xfi";
> - };
> -
> - mdio at e3000 {
> - #address-cells = <1>;
> - #size-cells = <0>;
> - compatible = "fsl,fman-memac-mdio", "fsl,fman-xmdio";
> - reg = <0xe3000 0x1000>;
> - fsl,erratum-a011043; /* must ignore read errors */
> -
> - pcsphy1: ethernet-phy at 0 {
> - reg = <0x0>;
> - };
> - };
> -};
> diff --git a/arch/powerpc/boot/dts/fsl/t2081si-post.dtsi
> b/arch/powerpc/boot/dts/fsl/t2081si-post.dtsi
> index 74e17e134387..fed3879fa0aa 100644
> --- a/arch/powerpc/boot/dts/fsl/t2081si-post.dtsi
> +++ b/arch/powerpc/boot/dts/fsl/t2081si-post.dtsi
> @@ -609,8 +609,8 @@ usb1: usb at 211000 {
> /include/ "qoriq-bman1.dtsi"
>
> /include/ "qoriq-fman3-0.dtsi"
> -/include/ "qoriq-fman3-0-10g-2.dtsi"
> -/include/ "qoriq-fman3-0-10g-3.dtsi"
> +/include/ "qoriq-fman3-0-1g-2.dtsi"
> +/include/ "qoriq-fman3-0-1g-3.dtsi"
These two should be qoriq-fman3-0-1g-0.dtsi and qoriq-fman3-0-1g-1.dtsi.
You are including 1g-2.dtsi and 1g-3.dtsi twice.
> /include/ "qoriq-fman3-0-1g-2.dtsi"
> /include/ "qoriq-fman3-0-1g-3.dtsi"
> /include/ "qoriq-fman3-0-1g-4.dtsi"
> @@ -619,9 +619,11 @@ usb1: usb at 211000 {
> /include/ "qoriq-fman3-0-10g-1.dtsi"
> fman at 400000 {
> enet0: ethernet at e0000 {
> + pcs-handle-names = "sgmii", "xfi";
> };
>
> enet1: ethernet at e2000 {
> + pcs-handle-names = "sgmii", "xfi";
The second pcsphy for this port is still qsgmiia_pcs1 as described in
qoriq-fman3-0-1g-1.dtsi. It should also be overwritten, not only the name
property:
pcsphy-handle = <&pcsphy1>, <&pcsphy1>;
> };
>
> enet2: ethernet at e4000 {
> --
> 2.35.1.1320.gc452695387.dirty
More information about the Linuxppc-dev
mailing list