Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Camelia Alexandra Groza <camelia.groza@nxp.com>
To:     Sean Anderson <sean.anderson@seco.com>,
        "David S . Miller" <davem@davemloft.net>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>
CC:     "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Christophe Leroy <christophe.leroy@csgroup.eu>,
        Nicholas Piggin <npiggin@gmail.com>,
        Michael Ellerman <mpe@ellerman.id.au>,
        "linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Sean Anderson <sean.anderson@seco.com>
Subject: RE: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
Thread-Topic: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
Thread-Index: AQHZEXQBPotIqLiZVkqXNZXBHISHtK51Z8xA
Date:   Mon, 19 Dec 2022 16:23:09 +0000
Message-ID: <VI1PR04MB5807014739D89583FF87D43EF2E59@VI1PR04MB5807.eurprd04.prod.outlook.com>
References: <20221216172937.2960054-1-sean.anderson@seco.com>
In-Reply-To: <20221216172937.2960054-1-sean.anderson@seco.com>
Accept-Language: en-GB, ro-RO, en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?32RBhJ1PbppLjxNqnMl+0o2gXo9bRyhpTvZ3RKN72UeYGUeqSvC0ZQ38rG5d?=
 =?us-ascii?Q?kcURpNBi4Ufp/YY1T+yrpibRd3j6Wa9ecL8Jsmu6eq73juf20aew8kT/jLOj?=
 =?us-ascii?Q?Do/rh4ePU3tO51sgrFvLJmJPsu6wup8nS0gbklkT2tOOlw6Ur4faZSgQHT41?=
 =?us-ascii?Q?5vWDnSwVLl9ztK7UM8llwCAfxD/dqQnZIRVnn217KV9OfuFukZevE9grVu5b?=
 =?us-ascii?Q?j4YDhjgpCdPTODrSdqFj26HRSftrx9lTEKwXKxoQ6oyYeNHKHwSWLvVKS4kg?=
 =?us-ascii?Q?CvuxviMMi33KWUaiTtXa1zFJIauyRS/3nz/K6UuyyJJ/+g+RwBgxpePWZU33?=
 =?us-ascii?Q?XbPFwLTib+X1b1LZTl9wAk9G29Fp7YCxDboCxQLa3yOpp2MRaShnyWHLgsD1?=
 =?us-ascii?Q?2BE5ihjSHMFYiW6HITWsSm8CrJVZQPWkvZizLsR6iYwHFRAWURFyfFMgs2Ph?=
 =?us-ascii?Q?fCsxpAlSOVxl6k8dlFf3iQt8Et4KAKtBzgpi2lcB31CnW3XspT2fFtwR6Mj8?=
 =?us-ascii?Q?SSaI+pGxV3rVK/kriMvdwUv74k2p5dSeQPaOw+IzAOi8O2oDQMkj3NG5W7nn?=
 =?us-ascii?Q?GJOGiWscrdsiRPAA43oHMPGjjs8XGMdRTpO3eZgEB+P8ida8h/ebrrdet6Mg?=
 =?us-ascii?Q?16dh73D1ONx0o61JGyCYD10Y5xFFSRzEpCEAUcp4leVmCV/q4sK97pIWNSoC?=
 =?us-ascii?Q?h180t3LifcD5jI16FjtIHa+uMWKYN6Bhy9YYYrD6eT55/xRahT5YEYnT8UQy?=
 =?us-ascii?Q?ZwfWD8BiSs6JdktqeI0JkfOWKd3uBY6DRzu0u0vlezrbMXq4womLrGS9OMuz?=
 =?us-ascii?Q?y2RRo7fr10KFOYgUmgE4hvnYt5x/eoncJ/Gwh/mL6GK9gmmmaIEs8vsc9hdC?=
 =?us-ascii?Q?EAc9bsBCrM+tuCMPD2iRk7cxzgJccXrGO18LhjVzMnTCAqOgP+VZSI2s5HG+?=
 =?us-ascii?Q?f1n9Pu6abKegnPPrr1Tc5OF/5nN4dzu1VewlYIMhqcZFKYfwRN/HA9Mysjn3?=
 =?us-ascii?Q?LnEkfJai/aADShdlhI0nFKGgBu1XcKrxAijQBsVeAfv6Y30gbXAuuCQMj3DK?=
 =?us-ascii?Q?BCGKNmIvSwYEBxENfDKwZ11M/Ta/s7jxm8K45LZ219qARg6y7HzsFp4ksIJX?=
 =?us-ascii?Q?dOaMewAgGssLsJrC6sUYUJVRMkn9g+jGJjMFWHJH6rYG+Df5S9lTfKEFyYH2?=
 =?us-ascii?Q?IdCD5opTmt87wMlhudToQa0N0Eg/MdT+R01JaLKmTvE6fIuh6VzNHF7Vi6W5?=
 =?us-ascii?Q?q+h6sUeaB89OWkICi9OgppeWaf1SLob5vbKk+mZPNIPQmW51WjVLrxqeFo+6?=
 =?us-ascii?Q?GZUgYHRENT44M7OFwe6GhR7iCRCu9tEnvPAb6NU1tMpINqEp8csaoee/SWS2?=
 =?us-ascii?Q?cvqzmGAxzOEiJ7VvuoTq8yy8StuuMojZZRsBh4nV0DZLDvQDu6US55ThX6xR?=
 =?us-ascii?Q?juvIeCQw0ouivZhnjAigs+IX0rZF1nTJJIiFE6XypdmffHITB/qdxlhUUzc5?=
 =?us-ascii?Q?3/4gQn4G6kUI2rJm+0XvO4qYXwPz1sZtSc+0pldWLaBYKEQOT1sK32VwTvlk?=
 =?us-ascii?Q?kHszc8FsSjO92w32CaazGzc7+2pxbAAc1U7IMLfU?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5807.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3d9b69bc-f3e2-4c07-9eb5-08dae1dd5133
X-MS-Exchange-CrossTenant-originalarrivaltime: 19 Dec 2022 16:23:09.2237
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: eRpfO/HeHZ8+WfZqQ3a1khcxGv2pRCeuDjhVKXpHDQvD+vkLBT2XA1RD5sV14pJM9HGNcvR7IcgBUufAFo5wPg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR04MB9628
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> -----Original Message-----
> From: Sean Anderson <sean.anderson@seco.com>
> Sent: Friday, December 16, 2022 19:30
> To: David S . Miller <davem@davemloft.net>; netdev@vger.kernel.org
> Cc: devicetree@vger.kernel.org; Rob Herring <robh+dt@kernel.org>;
> Christophe Leroy <christophe.leroy@csgroup.eu>; Nicholas Piggin
> <npiggin@gmail.com>; Michael Ellerman <mpe@ellerman.id.au>; linuxppc-
> dev@lists.ozlabs.org; Krzysztof Kozlowski
> <krzysztof.kozlowski+dt@linaro.org>; linux-kernel@vger.kernel.org; Cameli=
a
> Alexandra Groza <camelia.groza@nxp.com>; Sean Anderson
> <sean.anderson@seco.com>
> Subject: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and
> MAC2
>=20
> There aren't enough resources to run these ports at 10G speeds. Disable
> 10G for these ports, reverting to the previous speed.
>=20
> Fixes: 36926a7d70c2 ("powerpc: dts: t208x: Mark MAC1 and MAC2 as 10G")
> Reported-by: Camelia Alexandra Groza <camelia.groza@nxp.com>
> Signed-off-by: Sean Anderson <sean.anderson@seco.com>
> ---

Thank you.

Reviewed-by: Camelia Groza <camelia.groza@nxp.com>
Tested-by: Camelia Groza <camelia.groza@nxp.com>

> Changes in v2:
> - Remove the 10g properties, instead of removing the MAC dtsis.
>=20
>  arch/powerpc/boot/dts/fsl/t2081si-post.dtsi | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
>=20
> diff --git a/arch/powerpc/boot/dts/fsl/t2081si-post.dtsi
> b/arch/powerpc/boot/dts/fsl/t2081si-post.dtsi
> index 74e17e134387..27714dc2f04a 100644
> --- a/arch/powerpc/boot/dts/fsl/t2081si-post.dtsi
> +++ b/arch/powerpc/boot/dts/fsl/t2081si-post.dtsi
> @@ -659,3 +659,19 @@ L2_1: l2-cache-controller@c20000 {
>  		interrupts =3D <16 2 1 9>;
>  	};
>  };
> +
> +&fman0_rx_0x08 {
> +	/delete-property/ fsl,fman-10g-port;
> +};
> +
> +&fman0_tx_0x28 {
> +	/delete-property/ fsl,fman-10g-port;
> +};
> +
> +&fman0_rx_0x09 {
> +	/delete-property/ fsl,fman-10g-port;
> +};
> +
> +&fman0_tx_0x29 {
> +	/delete-property/ fsl,fman-10g-port;
> +};
> --
> 2.35.1.1320.gc452695387.dirty