All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
@ 2022-12-16 17:29 ` Sean Anderson
  0 siblings, 0 replies; 14+ messages in thread
From: Sean Anderson @ 2022-12-16 17:29 UTC (permalink / raw)
  To: David S . Miller, netdev
  Cc: devicetree, Rob Herring, Christophe Leroy, Nicholas Piggin,
	Michael Ellerman, linuxppc-dev, Krzysztof Kozlowski,
	linux-kernel, Camelia Alexandra Groza, Sean Anderson

There aren't enough resources to run these ports at 10G speeds. Disable
10G for these ports, reverting to the previous speed.

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>
---

Changes in v2:
- Remove the 10g properties, instead of removing the MAC dtsis.

 arch/powerpc/boot/dts/fsl/t2081si-post.dtsi | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

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 = <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


^ permalink raw reply related	[flat|nested] 14+ messages in thread
* [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
@ 2022-12-16 17:29 ` Sean Anderson
  0 siblings, 0 replies; 14+ messages in thread
From: Sean Anderson @ 2022-12-16 17:29 UTC (permalink / raw)
  To: David S . Miller, netdev
  Cc: devicetree, Sean Anderson, linux-kernel, Nicholas Piggin,
	Rob Herring, Krzysztof Kozlowski, linuxppc-dev,
	Camelia Alexandra Groza

There aren't enough resources to run these ports at 10G speeds. Disable
10G for these ports, reverting to the previous speed.

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>
---

Changes in v2:
- Remove the 10g properties, instead of removing the MAC dtsis.

 arch/powerpc/boot/dts/fsl/t2081si-post.dtsi | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

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 = <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


^ permalink raw reply related	[flat|nested] 14+ messages in thread
* RE: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
  2022-12-16 17:29 ` Sean Anderson
@ 2022-12-19 16:23   ` Camelia Alexandra Groza
  -1 siblings, 0 replies; 14+ messages in thread
From: Camelia Alexandra Groza @ 2022-12-19 16:23 UTC (permalink / raw)
  To: Sean Anderson, David S . Miller, netdev
  Cc: devicetree, Rob Herring, Christophe Leroy, Nicholas Piggin,
	Michael Ellerman, linuxppc-dev, Krzysztof Kozlowski,
	linux-kernel, Sean Anderson

> -----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; Camelia
> 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
> 
> There aren't enough resources to run these ports at 10G speeds. Disable
> 10G for these ports, reverting to the previous speed.
> 
> 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.
> 
>  arch/powerpc/boot/dts/fsl/t2081si-post.dtsi | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
> 
> 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 = <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


^ permalink raw reply	[flat|nested] 14+ messages in thread
* RE: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
@ 2022-12-19 16:23   ` Camelia Alexandra Groza
  0 siblings, 0 replies; 14+ messages in thread
From: Camelia Alexandra Groza @ 2022-12-19 16:23 UTC (permalink / raw)
  To: Sean Anderson, David S . Miller, netdev
  Cc: devicetree, Sean Anderson, linux-kernel, Nicholas Piggin,
	Rob Herring, Krzysztof Kozlowski, linuxppc-dev

> -----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; Camelia
> 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
> 
> There aren't enough resources to run these ports at 10G speeds. Disable
> 10G for these ports, reverting to the previous speed.
> 
> 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.
> 
>  arch/powerpc/boot/dts/fsl/t2081si-post.dtsi | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
> 
> 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 = <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


^ permalink raw reply	[flat|nested] 14+ messages in thread
* RE: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
  2022-12-19 16:23   ` Camelia Alexandra Groza
@ 2022-12-22 15:41     ` Camelia Alexandra Groza
  -1 siblings, 0 replies; 14+ messages in thread
From: Camelia Alexandra Groza @ 2022-12-22 15:41 UTC (permalink / raw)
  To: Sean Anderson, David S . Miller, netdev
  Cc: devicetree, Rob Herring, Christophe Leroy, Nicholas Piggin,
	Michael Ellerman, linuxppc-dev, Krzysztof Kozlowski,
	linux-kernel, Sean Anderson

> -----Original Message-----
> From: Camelia Alexandra Groza <camelia.groza@nxp.com>
> Sent: Monday, December 19, 2022 18:23
> To: Sean Anderson <sean.anderson@seco.com>; 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; Sean
> Anderson <sean.anderson@seco.com>
> Subject: RE: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and
> MAC2
> 
> > -----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;
> Camelia
> > 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
> >
> > There aren't enough resources to run these ports at 10G speeds. Disable
> > 10G for these ports, reverting to the previous speed.
> >
> > 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>

I see the patch marked Not Applicable in the netdev patchwork.
What tree will it go through?

> > Changes in v2:
> > - Remove the 10g properties, instead of removing the MAC dtsis.
> >
> >  arch/powerpc/boot/dts/fsl/t2081si-post.dtsi | 16 ++++++++++++++++
> >  1 file changed, 16 insertions(+)
> >
> > 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 = <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


^ permalink raw reply	[flat|nested] 14+ messages in thread
* RE: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
@ 2022-12-22 15:41     ` Camelia Alexandra Groza
  0 siblings, 0 replies; 14+ messages in thread
From: Camelia Alexandra Groza @ 2022-12-22 15:41 UTC (permalink / raw)
  To: Sean Anderson, David S . Miller, netdev
  Cc: devicetree, Sean Anderson, linux-kernel, Nicholas Piggin,
	Rob Herring, Krzysztof Kozlowski, linuxppc-dev

> -----Original Message-----
> From: Camelia Alexandra Groza <camelia.groza@nxp.com>
> Sent: Monday, December 19, 2022 18:23
> To: Sean Anderson <sean.anderson@seco.com>; 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; Sean
> Anderson <sean.anderson@seco.com>
> Subject: RE: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and
> MAC2
> 
> > -----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;
> Camelia
> > 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
> >
> > There aren't enough resources to run these ports at 10G speeds. Disable
> > 10G for these ports, reverting to the previous speed.
> >
> > 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>

I see the patch marked Not Applicable in the netdev patchwork.
What tree will it go through?

> > Changes in v2:
> > - Remove the 10g properties, instead of removing the MAC dtsis.
> >
> >  arch/powerpc/boot/dts/fsl/t2081si-post.dtsi | 16 ++++++++++++++++
> >  1 file changed, 16 insertions(+)
> >
> > 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 = <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


^ permalink raw reply	[flat|nested] 14+ messages in thread
* Re: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
  2022-12-22 15:41     ` Camelia Alexandra Groza
@ 2022-12-22 19:08       ` Jakub Kicinski
  -1 siblings, 0 replies; 14+ messages in thread
From: Jakub Kicinski @ 2022-12-22 19:08 UTC (permalink / raw)
  To: Camelia Alexandra Groza
  Cc: Sean Anderson, David S . Miller, netdev, devicetree, Rob Herring,
	Christophe Leroy, Nicholas Piggin, Michael Ellerman,
	linuxppc-dev, Krzysztof Kozlowski, linux-kernel

On Thu, 22 Dec 2022 15:41:00 +0000 Camelia Alexandra Groza wrote:
> > Reviewed-by: Camelia Groza <camelia.groza@nxp.com>
> > Tested-by: Camelia Groza <camelia.groza@nxp.com>  
> 
> I see the patch marked Not Applicable in the netdev patchwork.
> What tree will it go through?

I could be wrong but I think DTS patches are supposed to go via the
platform / arch trees. We mostly take bindings via the networking trees
(and DTS changes if they are part of a larger code+binding+dts set).
But we can obviously apply this patch if that's the preference of
the PowerPC maintainers..

^ permalink raw reply	[flat|nested] 14+ messages in thread
* Re: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
@ 2022-12-22 19:08       ` Jakub Kicinski
  0 siblings, 0 replies; 14+ messages in thread
From: Jakub Kicinski @ 2022-12-22 19:08 UTC (permalink / raw)
  To: Camelia Alexandra Groza
  Cc: devicetree, Sean Anderson, netdev, linux-kernel, Rob Herring,
	Nicholas Piggin, Krzysztof Kozlowski, linuxppc-dev,
	David S . Miller

On Thu, 22 Dec 2022 15:41:00 +0000 Camelia Alexandra Groza wrote:
> > Reviewed-by: Camelia Groza <camelia.groza@nxp.com>
> > Tested-by: Camelia Groza <camelia.groza@nxp.com>  
> 
> I see the patch marked Not Applicable in the netdev patchwork.
> What tree will it go through?

I could be wrong but I think DTS patches are supposed to go via the
platform / arch trees. We mostly take bindings via the networking trees
(and DTS changes if they are part of a larger code+binding+dts set).
But we can obviously apply this patch if that's the preference of
the PowerPC maintainers..

^ permalink raw reply	[flat|nested] 14+ messages in thread
* Re: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
  2022-12-22 19:08       ` Jakub Kicinski
@ 2022-12-22 23:30         ` Michael Ellerman
  -1 siblings, 0 replies; 14+ messages in thread
From: Michael Ellerman @ 2022-12-22 23:30 UTC (permalink / raw)
  To: Jakub Kicinski, Camelia Alexandra Groza
  Cc: Sean Anderson, David S . Miller, netdev, devicetree, Rob Herring,
	Christophe Leroy, Nicholas Piggin, linuxppc-dev,
	Krzysztof Kozlowski, linux-kernel

Jakub Kicinski <kuba@kernel.org> writes:
> On Thu, 22 Dec 2022 15:41:00 +0000 Camelia Alexandra Groza wrote:
>> > Reviewed-by: Camelia Groza <camelia.groza@nxp.com>
>> > Tested-by: Camelia Groza <camelia.groza@nxp.com>  
>> 
>> I see the patch marked Not Applicable in the netdev patchwork.
>> What tree will it go through?
>
> I could be wrong but I think DTS patches are supposed to go via the
> platform / arch trees. We mostly take bindings via the networking trees
> (and DTS changes if they are part of a larger code+binding+dts set).
> But we can obviously apply this patch if that's the preference of
> the PowerPC maintainers..

The commit it Fixes went in via the networking tree, so I think it would
make sense for you to take this also via the networking tree.

cheers

^ permalink raw reply	[flat|nested] 14+ messages in thread
* Re: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
@ 2022-12-22 23:30         ` Michael Ellerman
  0 siblings, 0 replies; 14+ messages in thread
From: Michael Ellerman @ 2022-12-22 23:30 UTC (permalink / raw)
  To: Jakub Kicinski, Camelia Alexandra Groza
  Cc: devicetree, Sean Anderson, netdev, linux-kernel, Rob Herring,
	Nicholas Piggin, Krzysztof Kozlowski, linuxppc-dev,
	David S . Miller

Jakub Kicinski <kuba@kernel.org> writes:
> On Thu, 22 Dec 2022 15:41:00 +0000 Camelia Alexandra Groza wrote:
>> > Reviewed-by: Camelia Groza <camelia.groza@nxp.com>
>> > Tested-by: Camelia Groza <camelia.groza@nxp.com>  
>> 
>> I see the patch marked Not Applicable in the netdev patchwork.
>> What tree will it go through?
>
> I could be wrong but I think DTS patches are supposed to go via the
> platform / arch trees. We mostly take bindings via the networking trees
> (and DTS changes if they are part of a larger code+binding+dts set).
> But we can obviously apply this patch if that's the preference of
> the PowerPC maintainers..

The commit it Fixes went in via the networking tree, so I think it would
make sense for you to take this also via the networking tree.

cheers

^ permalink raw reply	[flat|nested] 14+ messages in thread
* Re: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
  2022-12-22 23:30         ` Michael Ellerman
@ 2022-12-23  1:56           ` Jakub Kicinski
  -1 siblings, 0 replies; 14+ messages in thread
From: Jakub Kicinski @ 2022-12-23  1:56 UTC (permalink / raw)
  To: Michael Ellerman
  Cc: Camelia Alexandra Groza, Sean Anderson, David S . Miller, netdev,
	devicetree, Rob Herring, Christophe Leroy, Nicholas Piggin,
	linuxppc-dev, Krzysztof Kozlowski, linux-kernel

On Fri, 23 Dec 2022 10:30:36 +1100 Michael Ellerman wrote:
> The commit it Fixes went in via the networking tree, so I think it would
> make sense for you to take this also via the networking tree.

Roger that, thanks for confirming.

^ permalink raw reply	[flat|nested] 14+ messages in thread
* Re: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
@ 2022-12-23  1:56           ` Jakub Kicinski
  0 siblings, 0 replies; 14+ messages in thread
From: Jakub Kicinski @ 2022-12-23  1:56 UTC (permalink / raw)
  To: Michael Ellerman
  Cc: devicetree, Sean Anderson, netdev, Nicholas Piggin, linux-kernel,
	Rob Herring, Camelia Alexandra Groza, Krzysztof Kozlowski,
	linuxppc-dev, David S . Miller

On Fri, 23 Dec 2022 10:30:36 +1100 Michael Ellerman wrote:
> The commit it Fixes went in via the networking tree, so I think it would
> make sense for you to take this also via the networking tree.

Roger that, thanks for confirming.

^ permalink raw reply	[flat|nested] 14+ messages in thread
* Re: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
  2022-12-16 17:29 ` Sean Anderson
@ 2022-12-23  2:00   ` patchwork-bot+netdevbpf
  -1 siblings, 0 replies; 14+ messages in thread
From: patchwork-bot+netdevbpf @ 2022-12-23  2:00 UTC (permalink / raw)
  To: Sean Anderson
  Cc: davem, netdev, devicetree, robh+dt, christophe.leroy, npiggin,
	mpe, linuxppc-dev, krzysztof.kozlowski+dt, linux-kernel,
	camelia.groza

Hello:

This patch was applied to netdev/net.git (master)
by Jakub Kicinski <kuba@kernel.org>:

On Fri, 16 Dec 2022 12:29:37 -0500 you wrote:
> There aren't enough resources to run these ports at 10G speeds. Disable
> 10G for these ports, reverting to the previous speed.
> 
> 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>
> 
> [...]

Here is the summary with links:
  - [net,v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
    https://git.kernel.org/netdev/net/c/8d8bee13ae9e

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 14+ messages in thread
* Re: [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
@ 2022-12-23  2:00   ` patchwork-bot+netdevbpf
  0 siblings, 0 replies; 14+ messages in thread
From: patchwork-bot+netdevbpf @ 2022-12-23  2:00 UTC (permalink / raw)
  To: Sean Anderson
  Cc: devicetree, netdev, linux-kernel, npiggin, robh+dt,
	krzysztof.kozlowski+dt, linuxppc-dev, davem, camelia.groza

Hello:

This patch was applied to netdev/net.git (master)
by Jakub Kicinski <kuba@kernel.org>:

On Fri, 16 Dec 2022 12:29:37 -0500 you wrote:
> There aren't enough resources to run these ports at 10G speeds. Disable
> 10G for these ports, reverting to the previous speed.
> 
> 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>
> 
> [...]

Here is the summary with links:
  - [net,v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2
    https://git.kernel.org/netdev/net/c/8d8bee13ae9e

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 14+ messages in thread
end of thread, other threads:[~2022-12-23  2:01 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-12-16 17:29 [PATCH net v2] powerpc: dts: t208x: Disable 10G on MAC1 and MAC2 Sean Anderson
2022-12-16 17:29 ` Sean Anderson
2022-12-19 16:23 ` Camelia Alexandra Groza
2022-12-19 16:23   ` Camelia Alexandra Groza
2022-12-22 15:41   ` Camelia Alexandra Groza
2022-12-22 15:41     ` Camelia Alexandra Groza
2022-12-22 19:08     ` Jakub Kicinski
2022-12-22 19:08       ` Jakub Kicinski
2022-12-22 23:30       ` Michael Ellerman
2022-12-22 23:30         ` Michael Ellerman
2022-12-23  1:56         ` Jakub Kicinski
2022-12-23  1:56           ` Jakub Kicinski
2022-12-23  2:00 ` patchwork-bot+netdevbpf
2022-12-23  2:00   ` patchwork-bot+netdevbpf

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.