* [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
@ 2020-03-27 12:40 ` Russell King
0 siblings, 0 replies; 18+ messages in thread
From: Russell King @ 2020-03-27 12:40 UTC (permalink / raw)
To: Gregory Clement
Cc: Mark Rutland, Andrew Lunn, Jason Cooper, devicetree,
Madalin Bucur (OSS),
Rob Herring, linux-arm-kernel, Sebastian Hesselbarth
Update the SolidRun Armada 8040 platforms phy interface types from the
old 10gbase-kr to the newer and more correct 10gbase-r.
Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
---
arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts | 2 +-
arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts | 4 ++--
arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts | 4 ++--
3 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts b/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
index 1e7b47affe26..006666272489 100644
--- a/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
+++ b/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
@@ -401,7 +401,7 @@
/* SFP */
&cp0_eth0 {
status = "okay";
- phy-mode = "10gbase-kr";
+ phy-mode = "10gbase-r";
managed = "in-band-status";
phys = <&cp0_comphy2 0>;
sfp = <&sfp_cp0_eth0>;
diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts b/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
index 26114d96d637..b44f4c029b78 100644
--- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
+++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
@@ -16,14 +16,14 @@
&cp0_eth0 {
status = "okay";
- phy-mode = "10gbase-kr";
+ phy-mode = "10gbase-r";
managed = "in-band-status";
sfp = <&sfp_eth0>;
};
&cp1_eth0 {
status = "okay";
- phy-mode = "10gbase-kr";
+ phy-mode = "10gbase-r";
managed = "in-band-status";
sfp = <&sfp_eth1>;
};
diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
index 087a5502631e..42741adf867b 100644
--- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
+++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
@@ -63,14 +63,14 @@
status = "okay";
/* Network PHY */
phy = <&phy0>;
- phy-mode = "10gbase-kr";
+ phy-mode = "10gbase-r";
};
&cp1_eth0 {
status = "okay";
/* Network PHY */
phy = <&phy8>;
- phy-mode = "10gbase-kr";
+ phy-mode = "10gbase-r";
};
&cp1_pinctrl {
--
2.20.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 18+ messages in thread* RE: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
2020-03-27 12:40 ` Russell King
@ 2020-03-27 12:42 ` Madalin Bucur (OSS)
-1 siblings, 0 replies; 18+ messages in thread
From: Madalin Bucur (OSS) @ 2020-03-27 12:42 UTC (permalink / raw)
To: Russell King, Gregory Clement
Cc: Madalin Bucur (OSS),
Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Rob Herring,
Mark Rutland, linux-arm-kernel, devicetree
> -----Original Message-----
> From: Russell King,,, <rmk@armlinux.org.uk> On Behalf Of Russell King
> Sent: Friday, March 27, 2020 2:41 PM
> To: Gregory Clement <gregory.clement@bootlin.com>
> Cc: Madalin Bucur (OSS) <madalin.bucur@oss.nxp.com>; Jason Cooper
> <jason@lakedaemon.net>; Andrew Lunn <andrew@lunn.ch>; Sebastian
> Hesselbarth <sebastian.hesselbarth@gmail.com>; Rob Herring
> <robh+dt@kernel.org>; Mark Rutland <mark.rutland@arm.com>; linux-arm-
> kernel@lists.infradead.org; devicetree@vger.kernel.org
> Subject: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface
> types
>
> Update the SolidRun Armada 8040 platforms phy interface types from the
> old 10gbase-kr to the newer and more correct 10gbase-r.
>
> Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Reviewed-by: Madalin Bucur <madalin.bucur@oss.nxp.com>
> ---
> arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts | 2 +-
> arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts | 4 ++--
> arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts | 4 ++--
> 3 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> b/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> index 1e7b47affe26..006666272489 100644
> --- a/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> @@ -401,7 +401,7 @@
> /* SFP */
> &cp0_eth0 {
> status = "okay";
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> managed = "in-band-status";
> phys = <&cp0_comphy2 0>;
> sfp = <&sfp_cp0_eth0>;
> diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
> b/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
> index 26114d96d637..b44f4c029b78 100644
> --- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
> @@ -16,14 +16,14 @@
>
> &cp0_eth0 {
> status = "okay";
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> managed = "in-band-status";
> sfp = <&sfp_eth0>;
> };
>
> &cp1_eth0 {
> status = "okay";
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> managed = "in-band-status";
> sfp = <&sfp_eth1>;
> };
> diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
> b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
> index 087a5502631e..42741adf867b 100644
> --- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
> @@ -63,14 +63,14 @@
> status = "okay";
> /* Network PHY */
> phy = <&phy0>;
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> };
>
> &cp1_eth0 {
> status = "okay";
> /* Network PHY */
> phy = <&phy8>;
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> };
>
> &cp1_pinctrl {
> --
> 2.20.1
^ permalink raw reply [flat|nested] 18+ messages in thread* RE: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
@ 2020-03-27 12:42 ` Madalin Bucur (OSS)
0 siblings, 0 replies; 18+ messages in thread
From: Madalin Bucur (OSS) @ 2020-03-27 12:42 UTC (permalink / raw)
To: Russell King, Gregory Clement
Cc: Mark Rutland, Andrew Lunn, Jason Cooper, devicetree,
Madalin Bucur (OSS),
Rob Herring, linux-arm-kernel, Sebastian Hesselbarth
> -----Original Message-----
> From: Russell King,,, <rmk@armlinux.org.uk> On Behalf Of Russell King
> Sent: Friday, March 27, 2020 2:41 PM
> To: Gregory Clement <gregory.clement@bootlin.com>
> Cc: Madalin Bucur (OSS) <madalin.bucur@oss.nxp.com>; Jason Cooper
> <jason@lakedaemon.net>; Andrew Lunn <andrew@lunn.ch>; Sebastian
> Hesselbarth <sebastian.hesselbarth@gmail.com>; Rob Herring
> <robh+dt@kernel.org>; Mark Rutland <mark.rutland@arm.com>; linux-arm-
> kernel@lists.infradead.org; devicetree@vger.kernel.org
> Subject: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface
> types
>
> Update the SolidRun Armada 8040 platforms phy interface types from the
> old 10gbase-kr to the newer and more correct 10gbase-r.
>
> Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Reviewed-by: Madalin Bucur <madalin.bucur@oss.nxp.com>
> ---
> arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts | 2 +-
> arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts | 4 ++--
> arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts | 4 ++--
> 3 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> b/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> index 1e7b47affe26..006666272489 100644
> --- a/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> @@ -401,7 +401,7 @@
> /* SFP */
> &cp0_eth0 {
> status = "okay";
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> managed = "in-band-status";
> phys = <&cp0_comphy2 0>;
> sfp = <&sfp_cp0_eth0>;
> diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
> b/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
> index 26114d96d637..b44f4c029b78 100644
> --- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
> @@ -16,14 +16,14 @@
>
> &cp0_eth0 {
> status = "okay";
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> managed = "in-band-status";
> sfp = <&sfp_eth0>;
> };
>
> &cp1_eth0 {
> status = "okay";
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> managed = "in-band-status";
> sfp = <&sfp_eth1>;
> };
> diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
> b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
> index 087a5502631e..42741adf867b 100644
> --- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
> @@ -63,14 +63,14 @@
> status = "okay";
> /* Network PHY */
> phy = <&phy0>;
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> };
>
> &cp1_eth0 {
> status = "okay";
> /* Network PHY */
> phy = <&phy8>;
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> };
>
> &cp1_pinctrl {
> --
> 2.20.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
2020-03-27 12:40 ` Russell King
@ 2020-03-27 12:45 ` Russell King - ARM Linux admin
-1 siblings, 0 replies; 18+ messages in thread
From: Russell King - ARM Linux admin @ 2020-03-27 12:45 UTC (permalink / raw)
To: Gregory Clement
Cc: Madalin Bucur (OSS),
Jason Cooper, Andrew Lunn, Sebastian Hesselbarth, Rob Herring,
Mark Rutland, linux-arm-kernel, devicetree
A note for Gregorii / bootlin people:
After this patch, these still remain:
arch/arm64/boot/dts/marvell/cn9131-db.dts: phy-mode = "10gbase-kr";
arch/arm64/boot/dts/marvell/armada-7040-db.dts: phy-mode = "10gbase-kr";
arch/arm64/boot/dts/marvell/armada-8040-db.dts: phy-mode = "10gbase-kr";
arch/arm64/boot/dts/marvell/armada-8040-db.dts: phy-mode = "10gbase-kr";
arch/arm64/boot/dts/marvell/cn9132-db.dts: phy-mode = "10gbase-kr";
arch/arm64/boot/dts/marvell/cn9130-db.dts: phy-mode = "10gbase-kr";
Please can you check whether these should be updated, and if so also
update them? 10GBASE-KR is the backplane variant of 10GBASE-R which
involves some negotiation.
Thanks.
On Fri, Mar 27, 2020 at 12:40:42PM +0000, Russell King wrote:
> Update the SolidRun Armada 8040 platforms phy interface types from the
> old 10gbase-kr to the newer and more correct 10gbase-r.
>
> Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
> ---
> arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts | 2 +-
> arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts | 4 ++--
> arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts | 4 ++--
> 3 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts b/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> index 1e7b47affe26..006666272489 100644
> --- a/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> @@ -401,7 +401,7 @@
> /* SFP */
> &cp0_eth0 {
> status = "okay";
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> managed = "in-band-status";
> phys = <&cp0_comphy2 0>;
> sfp = <&sfp_cp0_eth0>;
> diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts b/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
> index 26114d96d637..b44f4c029b78 100644
> --- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
> @@ -16,14 +16,14 @@
>
> &cp0_eth0 {
> status = "okay";
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> managed = "in-band-status";
> sfp = <&sfp_eth0>;
> };
>
> &cp1_eth0 {
> status = "okay";
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> managed = "in-band-status";
> sfp = <&sfp_eth1>;
> };
> diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
> index 087a5502631e..42741adf867b 100644
> --- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
> @@ -63,14 +63,14 @@
> status = "okay";
> /* Network PHY */
> phy = <&phy0>;
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> };
>
> &cp1_eth0 {
> status = "okay";
> /* Network PHY */
> phy = <&phy8>;
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> };
>
> &cp1_pinctrl {
> --
> 2.20.1
>
>
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 10.2Mbps down 587kbps up
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
@ 2020-03-27 12:45 ` Russell King - ARM Linux admin
0 siblings, 0 replies; 18+ messages in thread
From: Russell King - ARM Linux admin @ 2020-03-27 12:45 UTC (permalink / raw)
To: Gregory Clement
Cc: Mark Rutland, Andrew Lunn, Jason Cooper, devicetree,
Madalin Bucur (OSS),
Rob Herring, linux-arm-kernel, Sebastian Hesselbarth
A note for Gregorii / bootlin people:
After this patch, these still remain:
arch/arm64/boot/dts/marvell/cn9131-db.dts: phy-mode = "10gbase-kr";
arch/arm64/boot/dts/marvell/armada-7040-db.dts: phy-mode = "10gbase-kr";
arch/arm64/boot/dts/marvell/armada-8040-db.dts: phy-mode = "10gbase-kr";
arch/arm64/boot/dts/marvell/armada-8040-db.dts: phy-mode = "10gbase-kr";
arch/arm64/boot/dts/marvell/cn9132-db.dts: phy-mode = "10gbase-kr";
arch/arm64/boot/dts/marvell/cn9130-db.dts: phy-mode = "10gbase-kr";
Please can you check whether these should be updated, and if so also
update them? 10GBASE-KR is the backplane variant of 10GBASE-R which
involves some negotiation.
Thanks.
On Fri, Mar 27, 2020 at 12:40:42PM +0000, Russell King wrote:
> Update the SolidRun Armada 8040 platforms phy interface types from the
> old 10gbase-kr to the newer and more correct 10gbase-r.
>
> Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
> ---
> arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts | 2 +-
> arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts | 4 ++--
> arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts | 4 ++--
> 3 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts b/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> index 1e7b47affe26..006666272489 100644
> --- a/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-8040-clearfog-gt-8k.dts
> @@ -401,7 +401,7 @@
> /* SFP */
> &cp0_eth0 {
> status = "okay";
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> managed = "in-band-status";
> phys = <&cp0_comphy2 0>;
> sfp = <&sfp_cp0_eth0>;
> diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts b/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
> index 26114d96d637..b44f4c029b78 100644
> --- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin-singleshot.dts
> @@ -16,14 +16,14 @@
>
> &cp0_eth0 {
> status = "okay";
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> managed = "in-band-status";
> sfp = <&sfp_eth0>;
> };
>
> &cp1_eth0 {
> status = "okay";
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> managed = "in-band-status";
> sfp = <&sfp_eth1>;
> };
> diff --git a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
> index 087a5502631e..42741adf867b 100644
> --- a/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
> +++ b/arch/arm64/boot/dts/marvell/armada-8040-mcbin.dts
> @@ -63,14 +63,14 @@
> status = "okay";
> /* Network PHY */
> phy = <&phy0>;
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> };
>
> &cp1_eth0 {
> status = "okay";
> /* Network PHY */
> phy = <&phy8>;
> - phy-mode = "10gbase-kr";
> + phy-mode = "10gbase-r";
> };
>
> &cp1_pinctrl {
> --
> 2.20.1
>
>
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 10.2Mbps down 587kbps up
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
2020-03-27 12:40 ` Russell King
@ 2020-03-27 13:43 ` Andrew Lunn
-1 siblings, 0 replies; 18+ messages in thread
From: Andrew Lunn @ 2020-03-27 13:43 UTC (permalink / raw)
To: Russell King
Cc: Gregory Clement, Madalin Bucur (OSS),
Jason Cooper, Sebastian Hesselbarth, Rob Herring, Mark Rutland,
linux-arm-kernel, devicetree
On Fri, Mar 27, 2020 at 12:40:42PM +0000, Russell King wrote:
> Update the SolidRun Armada 8040 platforms phy interface types from the
> old 10gbase-kr to the newer and more correct 10gbase-r.
>
> Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Can we add a fixes: tag to this? So it gets backported to the point
10gbase-r was added.
Andrew
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
@ 2020-03-27 13:43 ` Andrew Lunn
0 siblings, 0 replies; 18+ messages in thread
From: Andrew Lunn @ 2020-03-27 13:43 UTC (permalink / raw)
To: Russell King
Cc: Mark Rutland, devicetree, Jason Cooper, Madalin Bucur (OSS),
Gregory Clement, Rob Herring, linux-arm-kernel,
Sebastian Hesselbarth
On Fri, Mar 27, 2020 at 12:40:42PM +0000, Russell King wrote:
> Update the SolidRun Armada 8040 platforms phy interface types from the
> old 10gbase-kr to the newer and more correct 10gbase-r.
>
> Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Can we add a fixes: tag to this? So it gets backported to the point
10gbase-r was added.
Andrew
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
2020-03-27 13:43 ` Andrew Lunn
@ 2020-03-27 13:46 ` Russell King - ARM Linux admin
-1 siblings, 0 replies; 18+ messages in thread
From: Russell King - ARM Linux admin @ 2020-03-27 13:46 UTC (permalink / raw)
To: Andrew Lunn
Cc: Gregory Clement, Madalin Bucur (OSS),
Jason Cooper, Sebastian Hesselbarth, Rob Herring, Mark Rutland,
linux-arm-kernel, devicetree
On Fri, Mar 27, 2020 at 02:43:19PM +0100, Andrew Lunn wrote:
> On Fri, Mar 27, 2020 at 12:40:42PM +0000, Russell King wrote:
> > Update the SolidRun Armada 8040 platforms phy interface types from the
> > old 10gbase-kr to the newer and more correct 10gbase-r.
> >
> > Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
>
> Reviewed-by: Andrew Lunn <andrew@lunn.ch>
>
> Can we add a fixes: tag to this? So it gets backported to the point
> 10gbase-r was added.
I think "carefully" is the answer. It's not just the point where
10gbase-r was added because there's a whole bunch of other patches
that are also required to the various ethernet and PHY drivers, SFP
and phylink. It's probably going to take a bit to work that out,
get it wrong and stuff will break.
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 10.2Mbps down 587kbps up
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
@ 2020-03-27 13:46 ` Russell King - ARM Linux admin
0 siblings, 0 replies; 18+ messages in thread
From: Russell King - ARM Linux admin @ 2020-03-27 13:46 UTC (permalink / raw)
To: Andrew Lunn
Cc: Mark Rutland, devicetree, Jason Cooper, Madalin Bucur (OSS),
Gregory Clement, Rob Herring, linux-arm-kernel,
Sebastian Hesselbarth
On Fri, Mar 27, 2020 at 02:43:19PM +0100, Andrew Lunn wrote:
> On Fri, Mar 27, 2020 at 12:40:42PM +0000, Russell King wrote:
> > Update the SolidRun Armada 8040 platforms phy interface types from the
> > old 10gbase-kr to the newer and more correct 10gbase-r.
> >
> > Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
>
> Reviewed-by: Andrew Lunn <andrew@lunn.ch>
>
> Can we add a fixes: tag to this? So it gets backported to the point
> 10gbase-r was added.
I think "carefully" is the answer. It's not just the point where
10gbase-r was added because there's a whole bunch of other patches
that are also required to the various ethernet and PHY drivers, SFP
and phylink. It's probably going to take a bit to work that out,
get it wrong and stuff will break.
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 10.2Mbps down 587kbps up
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
2020-03-27 13:46 ` Russell King - ARM Linux admin
@ 2020-03-27 13:50 ` Russell King - ARM Linux admin
-1 siblings, 0 replies; 18+ messages in thread
From: Russell King - ARM Linux admin @ 2020-03-27 13:50 UTC (permalink / raw)
To: Andrew Lunn
Cc: Mark Rutland, devicetree, Jason Cooper, Madalin Bucur (OSS),
Gregory Clement, Rob Herring, linux-arm-kernel,
Sebastian Hesselbarth
On Fri, Mar 27, 2020 at 01:46:52PM +0000, Russell King - ARM Linux admin wrote:
> On Fri, Mar 27, 2020 at 02:43:19PM +0100, Andrew Lunn wrote:
> > On Fri, Mar 27, 2020 at 12:40:42PM +0000, Russell King wrote:
> > > Update the SolidRun Armada 8040 platforms phy interface types from the
> > > old 10gbase-kr to the newer and more correct 10gbase-r.
> > >
> > > Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
> >
> > Reviewed-by: Andrew Lunn <andrew@lunn.ch>
> >
> > Can we add a fixes: tag to this? So it gets backported to the point
> > 10gbase-r was added.
>
> I think "carefully" is the answer. It's not just the point where
> 10gbase-r was added because there's a whole bunch of other patches
> that are also required to the various ethernet and PHY drivers, SFP
> and phylink. It's probably going to take a bit to work that out,
> get it wrong and stuff will break.
... and actually there is no point. It is _way_ too soon to even
start thinking of using 10GKR for its true purpose - the patches
to add 10GBASER were only merged for v5.6-rc1, so we're still waiting
for a kernel with that update to be released. In other words, there's
no stable kernels to backport it to yet.
Do we want it to go into -rc? If so, it also needs to be thoroughly
tested.
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 10.2Mbps down 587kbps up
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
@ 2020-03-27 13:50 ` Russell King - ARM Linux admin
0 siblings, 0 replies; 18+ messages in thread
From: Russell King - ARM Linux admin @ 2020-03-27 13:50 UTC (permalink / raw)
To: Andrew Lunn
Cc: Mark Rutland, devicetree, Jason Cooper, Madalin Bucur (OSS),
Gregory Clement, Rob Herring, linux-arm-kernel,
Sebastian Hesselbarth
On Fri, Mar 27, 2020 at 01:46:52PM +0000, Russell King - ARM Linux admin wrote:
> On Fri, Mar 27, 2020 at 02:43:19PM +0100, Andrew Lunn wrote:
> > On Fri, Mar 27, 2020 at 12:40:42PM +0000, Russell King wrote:
> > > Update the SolidRun Armada 8040 platforms phy interface types from the
> > > old 10gbase-kr to the newer and more correct 10gbase-r.
> > >
> > > Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>
> >
> > Reviewed-by: Andrew Lunn <andrew@lunn.ch>
> >
> > Can we add a fixes: tag to this? So it gets backported to the point
> > 10gbase-r was added.
>
> I think "carefully" is the answer. It's not just the point where
> 10gbase-r was added because there's a whole bunch of other patches
> that are also required to the various ethernet and PHY drivers, SFP
> and phylink. It's probably going to take a bit to work that out,
> get it wrong and stuff will break.
... and actually there is no point. It is _way_ too soon to even
start thinking of using 10GKR for its true purpose - the patches
to add 10GBASER were only merged for v5.6-rc1, so we're still waiting
for a kernel with that update to be released. In other words, there's
no stable kernels to backport it to yet.
Do we want it to go into -rc? If so, it also needs to be thoroughly
tested.
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 10.2Mbps down 587kbps up
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
2020-03-27 13:50 ` Russell King - ARM Linux admin
@ 2020-03-27 14:15 ` Andrew Lunn
-1 siblings, 0 replies; 18+ messages in thread
From: Andrew Lunn @ 2020-03-27 14:15 UTC (permalink / raw)
To: Russell King - ARM Linux admin
Cc: Mark Rutland, devicetree, Jason Cooper, Madalin Bucur (OSS),
Gregory Clement, Rob Herring, linux-arm-kernel,
Sebastian Hesselbarth
> ... and actually there is no point.
O.K. lets merge these as normal, not as fixes. It is probably too
late for ARM SoC, so i guess they will get merged next cycle.
Andrew
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
@ 2020-03-27 14:15 ` Andrew Lunn
0 siblings, 0 replies; 18+ messages in thread
From: Andrew Lunn @ 2020-03-27 14:15 UTC (permalink / raw)
To: Russell King - ARM Linux admin
Cc: Mark Rutland, devicetree, Jason Cooper, Madalin Bucur (OSS),
Gregory Clement, Rob Herring, linux-arm-kernel,
Sebastian Hesselbarth
> ... and actually there is no point.
O.K. lets merge these as normal, not as fixes. It is probably too
late for ARM SoC, so i guess they will get merged next cycle.
Andrew
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread* Re: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
2020-03-27 14:15 ` Andrew Lunn
@ 2020-03-27 15:17 ` Russell King - ARM Linux admin
-1 siblings, 0 replies; 18+ messages in thread
From: Russell King - ARM Linux admin @ 2020-03-27 15:17 UTC (permalink / raw)
To: Andrew Lunn
Cc: Mark Rutland, devicetree, Jason Cooper, Madalin Bucur (OSS),
Gregory Clement, Rob Herring, linux-arm-kernel,
Sebastian Hesselbarth
On Fri, Mar 27, 2020 at 03:15:38PM +0100, Andrew Lunn wrote:
> > ... and actually there is no point.
>
> O.K. lets merge these as normal, not as fixes. It is probably too
> late for ARM SoC, so i guess they will get merged next cycle.
Yep, which means that the backplane patches from NXP will have to be
delayed about 20 weeks (about 10 weeks in the arm64 tree before being
merged into 5.8-rc1, and then another 10 weeks before 5.8 is released.)
That's probably not a bad thing, as it'll let us hash out the conflicts
between my PCS patch set and their backplane patches.
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 10.2Mbps down 587kbps up
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
@ 2020-03-27 15:17 ` Russell King - ARM Linux admin
0 siblings, 0 replies; 18+ messages in thread
From: Russell King - ARM Linux admin @ 2020-03-27 15:17 UTC (permalink / raw)
To: Andrew Lunn
Cc: Mark Rutland, devicetree, Jason Cooper, Madalin Bucur (OSS),
Gregory Clement, Rob Herring, linux-arm-kernel,
Sebastian Hesselbarth
On Fri, Mar 27, 2020 at 03:15:38PM +0100, Andrew Lunn wrote:
> > ... and actually there is no point.
>
> O.K. lets merge these as normal, not as fixes. It is probably too
> late for ARM SoC, so i guess they will get merged next cycle.
Yep, which means that the backplane patches from NXP will have to be
delayed about 20 weeks (about 10 weeks in the arm64 tree before being
merged into 5.8-rc1, and then another 10 weeks before 5.8 is released.)
That's probably not a bad thing, as it'll let us hash out the conflicts
between my PCS patch set and their backplane patches.
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line in suburbia: sync at 10.2Mbps down 587kbps up
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
2020-03-27 15:17 ` Russell King - ARM Linux admin
@ 2020-03-27 17:16 ` Gregory CLEMENT
-1 siblings, 0 replies; 18+ messages in thread
From: Gregory CLEMENT @ 2020-03-27 17:16 UTC (permalink / raw)
To: Russell King - ARM Linux admin, Andrew Lunn
Cc: Mark Rutland, devicetree, Jason Cooper, Madalin Bucur (OSS),
Rob Herring, linux-arm-kernel, Sebastian Hesselbarth
Hello,
> On Fri, Mar 27, 2020 at 03:15:38PM +0100, Andrew Lunn wrote:
>> > ... and actually there is no point.
>>
>> O.K. lets merge these as normal, not as fixes. It is probably too
>> late for ARM SoC, so i guess they will get merged next cycle.
>
> Yep, which means that the backplane patches from NXP will have to be
> delayed about 20 weeks (about 10 weeks in the arm64 tree before being
> merged into 5.8-rc1, and then another 10 weeks before 5.8 is released.)
> That's probably not a bad thing, as it'll let us hash out the conflicts
> between my PCS patch set and their backplane patches.
None of my pull request has been merged yet, so I can make an other one
ontop of the existing ones with this patch, and I will let decide the
ARM SoC maintainer if they can take it or not.
Gregory
>
> --
> RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
> FTTC broadband for 0.8mile line in suburbia: sync at 10.2Mbps down 587kbps up
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
^ permalink raw reply [flat|nested] 18+ messages in thread
* Re: [PATCH] arm64: dts: update SolidRun Armada 8040 phy interface types
@ 2020-03-27 17:16 ` Gregory CLEMENT
0 siblings, 0 replies; 18+ messages in thread
From: Gregory CLEMENT @ 2020-03-27 17:16 UTC (permalink / raw)
To: Russell King - ARM Linux admin, Andrew Lunn
Cc: Mark Rutland, devicetree, Jason Cooper, Madalin Bucur (OSS),
Rob Herring, linux-arm-kernel, Sebastian Hesselbarth
Hello,
> On Fri, Mar 27, 2020 at 03:15:38PM +0100, Andrew Lunn wrote:
>> > ... and actually there is no point.
>>
>> O.K. lets merge these as normal, not as fixes. It is probably too
>> late for ARM SoC, so i guess they will get merged next cycle.
>
> Yep, which means that the backplane patches from NXP will have to be
> delayed about 20 weeks (about 10 weeks in the arm64 tree before being
> merged into 5.8-rc1, and then another 10 weeks before 5.8 is released.)
> That's probably not a bad thing, as it'll let us hash out the conflicts
> between my PCS patch set and their backplane patches.
None of my pull request has been merged yet, so I can make an other one
ontop of the existing ones with this patch, and I will let decide the
ARM SoC maintainer if they can take it or not.
Gregory
>
> --
> RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
> FTTC broadband for 0.8mile line in suburbia: sync at 10.2Mbps down 587kbps up
--
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 18+ messages in thread