1. Patchew
  2. linux
  3. [PATCH 0/4] ARM: dts: imx: {bx50v3,b850v3}: Update device trees
  4. View patch

[PATCH 2/4] ARM: dts: imx: bx50v3: Configure phy-mode

Ian Ray posted 4 patches 1 month, 2 weeks ago
There is a newer version of this series
[PATCH 2/4] ARM: dts: imx: bx50v3: Configure phy-mode
Posted by Ian Ray 1 month, 2 weeks ago 
Set `phy-mode' on network switch CPU ports to eliminate a warning.

Signed-off-by: Ian Ray <ian.ray@gehealthcare.com>
---
 arch/arm/boot/dts/nxp/imx/imx6q-b450v3.dts | 1 +
 arch/arm/boot/dts/nxp/imx/imx6q-b650v3.dts | 1 +
 arch/arm/boot/dts/nxp/imx/imx6q-b850v3.dts | 1 +
 3 files changed, 3 insertions(+)

diff --git a/arch/arm/boot/dts/nxp/imx/imx6q-b450v3.dts b/arch/arm/boot/dts/nxp/imx/imx6q-b450v3.dts
index d994b32ad825..7326b8ad08cb 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6q-b450v3.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6q-b450v3.dts
@@ -140,6 +140,7 @@ port@3 {
 	port@4 {
 		reg = <4>;
 		label = "cpu";
+		phy-mode = "gmii";
 		ethernet = <&switch_nic>;
 		phy-handle = <&switchphy4>;
 	};
diff --git a/arch/arm/boot/dts/nxp/imx/imx6q-b650v3.dts b/arch/arm/boot/dts/nxp/imx/imx6q-b650v3.dts
index b0d345f5d071..9a43935be11c 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6q-b650v3.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6q-b650v3.dts
@@ -139,6 +139,7 @@ port@3 {
 	port@4 {
 		reg = <4>;
 		label = "cpu";
+		phy-mode = "gmii";
 		ethernet = <&switch_nic>;
 		phy-handle = <&switchphy4>;
 	};
diff --git a/arch/arm/boot/dts/nxp/imx/imx6q-b850v3.dts b/arch/arm/boot/dts/nxp/imx/imx6q-b850v3.dts
index cad112e05475..e7ccdcba2d5c 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6q-b850v3.dts
+++ b/arch/arm/boot/dts/nxp/imx/imx6q-b850v3.dts
@@ -286,6 +286,7 @@ port@3 {
 	port@4 {
 		reg = <4>;
 		label = "cpu";
+		phy-mode = "gmii";
 		ethernet = <&switch_nic>;
 		phy-handle = <&switchphy4>;
 	};
-- 
2.49.0
Re: [PATCH 2/4] ARM: dts: imx: bx50v3: Configure phy-mode
Posted by Daniel Baluta 1 month, 2 weeks ago 
On 2/17/26 14:59, Ian Ray wrote:
> Set `phy-mode' on network switch CPU ports to eliminate a warning.

Can you mention in the commit message what is the warning

and how did you get it?
Re: [PATCH 2/4] ARM: dts: imx: bx50v3: Configure phy-mode
Posted by Ian Ray 1 month, 2 weeks ago 
On Tue, Feb 17, 2026 at 03:08:17PM +0200, Daniel Baluta wrote:
> CAUTION: This email originated from outside of GE HealthCare. Only open links or attachments if you trust the sender. Report suspicious emails using Outlook’s “Report” button.
> 
> On 2/17/26 14:59, Ian Ray wrote:
> > Set `phy-mode' on network switch CPU ports to eliminate a warning.
> 
> Can you mention in the commit message what is the warning

Apologies for not being clearer. (Checks notes.) The warning was:

-- >8 --
mv88e6085 gpio-0:00: OF node /mdio-gpio/switch@0/ports/port@4 of CPU port 4 lacks the required "phy-mode" property
-- >8 --

I will prepare a V2.

> 
> and how did you get it?
>
Re: [PATCH 2/4] ARM: dts: imx: bx50v3: Configure phy-mode
Posted by Andrew Lunn 1 month, 2 weeks ago 
On Tue, Feb 17, 2026 at 02:59:49PM +0200, Ian Ray wrote:
> Set `phy-mode' on network switch CPU ports to eliminate a warning.
> 
> Signed-off-by: Ian Ray <ian.ray@gehealthcare.com>
> ---
>  arch/arm/boot/dts/nxp/imx/imx6q-b450v3.dts | 1 +
>  arch/arm/boot/dts/nxp/imx/imx6q-b650v3.dts | 1 +
>  arch/arm/boot/dts/nxp/imx/imx6q-b850v3.dts | 1 +
>  3 files changed, 3 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/nxp/imx/imx6q-b450v3.dts b/arch/arm/boot/dts/nxp/imx/imx6q-b450v3.dts
> index d994b32ad825..7326b8ad08cb 100644
> --- a/arch/arm/boot/dts/nxp/imx/imx6q-b450v3.dts
> +++ b/arch/arm/boot/dts/nxp/imx/imx6q-b450v3.dts
> @@ -140,6 +140,7 @@ port@3 {
>  	port@4 {
>  		reg = <4>;
>  		label = "cpu";
> +		phy-mode = "gmii";

Are the PHYs external to the switch?

>  	port@4 {
>  		reg = <4>;
>  		label = "cpu";
> +		phy-mode = "gmii";
>  		ethernet = <&switch_nic>;
>  		phy-handle = <&switchphy4>;

You have back to back PHYs on the CPU port? This is not wrong, but
unusual.

	Andrew
Re: [PATCH 2/4] ARM: dts: imx: bx50v3: Configure phy-mode
Posted by Ian Ray 1 month, 2 weeks ago 
On Tue, Feb 17, 2026 at 03:58:42PM +0100, Andrew Lunn wrote:
> CAUTION: This email originated from outside of GE HealthCare. Only open links or attachments if you trust the sender. Report suspicious emails using Outlook’s “Report” button.
> 
> On Tue, Feb 17, 2026 at 02:59:49PM +0200, Ian Ray wrote:
> > Set `phy-mode' on network switch CPU ports to eliminate a warning.
> >
> > Signed-off-by: Ian Ray <ian.ray@gehealthcare.com>
> > ---
> >  arch/arm/boot/dts/nxp/imx/imx6q-b450v3.dts | 1 +
> >  arch/arm/boot/dts/nxp/imx/imx6q-b650v3.dts | 1 +
> >  arch/arm/boot/dts/nxp/imx/imx6q-b850v3.dts | 1 +
> >  3 files changed, 3 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/nxp/imx/imx6q-b450v3.dts b/arch/arm/boot/dts/nxp/imx/imx6q-b450v3.dts
> > index d994b32ad825..7326b8ad08cb 100644
> > --- a/arch/arm/boot/dts/nxp/imx/imx6q-b450v3.dts
> > +++ b/arch/arm/boot/dts/nxp/imx/imx6q-b450v3.dts
> > @@ -140,6 +140,7 @@ port@3 {
> >       port@4 {
> >               reg = <4>;
> >               label = "cpu";
> > +             phy-mode = "gmii";
> 
> Are the PHYs external to the switch?

Yes, port 4 is connected to an I210.

> 
> >       port@4 {
> >               reg = <4>;
> >               label = "cpu";
> > +             phy-mode = "gmii";
> >               ethernet = <&switch_nic>;
> >               phy-handle = <&switchphy4>;
> 
> You have back to back PHYs on the CPU port? This is not wrong, but
> unusual.

No, the patch sets `phy-mode' on three different boards.  (If I
understand your question correctly.)

Thanks,
Ian

> 
>         Andrew

Patchew 2.1-devel-ea86937

© 2016 - 2026 Red Hat, Inc.