1. Patchew
  2. linux
  3. [PATCH 00/13] arm64: dts: imx95: various updates
  4. View patch

[PATCH 08/13] arm64: dts: imx95-19x19-evk: Add phy supply for netc

Peng Fan posted 13 patches 1 month, 2 weeks ago
There is a newer version of this series
[PATCH 08/13] arm64: dts: imx95-19x19-evk: Add phy supply for netc
Posted by Peng Fan 1 month, 2 weeks ago 
Add phy supply for netc emdio on board.

Signed-off-by: Peng Fan <peng.fan@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts b/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts
index 39815b21d235d2f8cfa49720d3be49d056ea039a..feee6da65d37cae7413b09216014d6fe2b76b032 100644
--- a/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts
+++ b/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts
@@ -226,6 +226,25 @@ sound-wm8962 {
 				"IN3R", "AMIC",
 				"IN1R", "AMIC";
 	};
+
+	reg_aqr_stby: regulator-aqr-stby {
+		compatible = "regulator-fixed";
+		regulator-name = "aqr-stby";
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+		gpio = <&i2c5_pcal6408 1 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+	};
+
+	reg_aqr_en: regulator-aqr-en {
+		compatible = "regulator-fixed";
+		regulator-name = "aqr-en";
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+		vin-supply = <&reg_aqr_stby>;
+		gpio = <&i2c5_pcal6408 0 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+	};
 };
 
 &adc1 {
@@ -502,6 +521,7 @@ &netc_blk_ctrl {
 &netc_emdio {
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_emdio>;
+	phy-supply = <&reg_aqr_en>;
 	status = "okay";
 
 	ethphy0: ethernet-phy@1 {

-- 
2.37.1
Re: [PATCH 08/13] arm64: dts: imx95-19x19-evk: Add phy supply for netc
Posted by Tim Harvey 3 weeks, 4 days ago 
On Fri, Aug 15, 2025 at 4:50 AM Peng Fan <peng.fan@nxp.com> wrote:
>
> Add phy supply for netc emdio on board.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
>  arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts b/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts
> index 39815b21d235d2f8cfa49720d3be49d056ea039a..feee6da65d37cae7413b09216014d6fe2b76b032 100644
> --- a/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts
> +++ b/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts
> @@ -226,6 +226,25 @@ sound-wm8962 {
>                                 "IN3R", "AMIC",
>                                 "IN1R", "AMIC";
>         };
> +
> +       reg_aqr_stby: regulator-aqr-stby {
> +               compatible = "regulator-fixed";
> +               regulator-name = "aqr-stby";
> +               regulator-min-microvolt = <3300000>;
> +               regulator-max-microvolt = <3300000>;
> +               gpio = <&i2c5_pcal6408 1 GPIO_ACTIVE_HIGH>;
> +               enable-active-high;
> +       };
> +
> +       reg_aqr_en: regulator-aqr-en {
> +               compatible = "regulator-fixed";
> +               regulator-name = "aqr-en";
> +               regulator-min-microvolt = <3300000>;
> +               regulator-max-microvolt = <3300000>;
> +               vin-supply = <&reg_aqr_stby>;
> +               gpio = <&i2c5_pcal6408 0 GPIO_ACTIVE_HIGH>;
> +               enable-active-high;
> +       };
>  };
>
>  &adc1 {
> @@ -502,6 +521,7 @@ &netc_blk_ctrl {
>  &netc_emdio {
>         pinctrl-names = "default";
>         pinctrl-0 = <&pinctrl_emdio>;
> +       phy-supply = <&reg_aqr_en>;
>         status = "okay";
>
>         ethphy0: ethernet-phy@1 {
>
> --
> 2.37.1
>
>

Hi Peng,

I'm not sure where you are with regards to this patch but it indicates
to me that you may have the enetc_port2 10G port with the AQR113 PHY
working on the imx95-19x19-evk with upstream Linux. If so I would be
interested in knowing what other patches you have for that. I'm seeing
the interface fail to register with:
nxp_enetc4 0002:00:10.0 eth2: PHY has no common interfaces

I'm wondering if there is some support missing from enetc or the
aquantia phy required to work with 10gbase-r.

Best Regards,

Tim
Re: [PATCH 08/13] arm64: dts: imx95-19x19-evk: Add phy supply for netc
Posted by Peng Fan 3 weeks, 4 days ago 
Hi Tim,

On Mon, Sep 08, 2025 at 04:43:25PM -0700, Tim Harvey wrote:
>On Fri, Aug 15, 2025 at 4:50 AM Peng Fan <peng.fan@nxp.com> wrote:
>>
>> Add phy supply for netc emdio on board.
>>
>>
>
>Hi Peng,
>
>I'm not sure where you are with regards to this patch but it indicates
>to me that you may have the enetc_port2 10G port with the AQR113 PHY
>working on the imx95-19x19-evk with upstream Linux. If so I would be
>interested in knowing what other patches you have for that. I'm seeing
>the interface fail to register with:
>nxp_enetc4 0002:00:10.0 eth2: PHY has no common interfaces
>
>I'm wondering if there is some support missing from enetc or the
>aquantia phy required to work with 10gbase-r.

Thanks for raising this.

This is just to add the missing pieces of device tree.

For 10g port, it is still not ready for upstream.

You could only use the rgmii-id port. This change itself does not break
the rgmii-id port.

Thanks,
Peng

>
>Best Regards,
>
>Tim
Re: [PATCH 08/13] arm64: dts: imx95-19x19-evk: Add phy supply for netc
Posted by Tim Harvey 3 weeks, 3 days ago 
On Mon, Sep 8, 2025 at 8:22 PM Peng Fan <peng.fan@oss.nxp.com> wrote:
>
> Hi Tim,
>
> On Mon, Sep 08, 2025 at 04:43:25PM -0700, Tim Harvey wrote:
> >On Fri, Aug 15, 2025 at 4:50 AM Peng Fan <peng.fan@nxp.com> wrote:
> >>
> >> Add phy supply for netc emdio on board.
> >>
> >>
> >
> >Hi Peng,
> >
> >I'm not sure where you are with regards to this patch but it indicates
> >to me that you may have the enetc_port2 10G port with the AQR113 PHY
> >working on the imx95-19x19-evk with upstream Linux. If so I would be
> >interested in knowing what other patches you have for that. I'm seeing
> >the interface fail to register with:
> >nxp_enetc4 0002:00:10.0 eth2: PHY has no common interfaces
> >
> >I'm wondering if there is some support missing from enetc or the
> >aquantia phy required to work with 10gbase-r.
>
> Thanks for raising this.
>
> This is just to add the missing pieces of device tree.
>

Hi Peng,

Ok, but phy-supply is not currently valid for netc_emdio

> For 10g port, it is still not ready for upstream.
>

For 10G it looks like at least the XPCS support is missing. Do you
know if anyone at NXP is working on this or if and when it is
scheduled?

Best Regards,

Tim


> For 10g port, it is still not ready for upstream.
>
> You could only use the rgmii-id port. This change itself does not break
> the rgmii-id port.
>
> Thanks,
> Peng
>
> >
> >Best Regards,
> >
> >Tim
Re: [PATCH 08/13] arm64: dts: imx95-19x19-evk: Add phy supply for netc
Posted by Frank Li 1 month, 2 weeks ago 
On Fri, Aug 15, 2025 at 05:03:54PM +0800, Peng Fan wrote:
> Add phy supply for netc emdio on board.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
>  arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts b/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts
> index 39815b21d235d2f8cfa49720d3be49d056ea039a..feee6da65d37cae7413b09216014d6fe2b76b032 100644
> --- a/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts
> +++ b/arch/arm64/boot/dts/freescale/imx95-19x19-evk.dts
> @@ -226,6 +226,25 @@ sound-wm8962 {
>  				"IN3R", "AMIC",
>  				"IN1R", "AMIC";
>  	};
> +
> +	reg_aqr_stby: regulator-aqr-stby {
> +		compatible = "regulator-fixed";
> +		regulator-name = "aqr-stby";
> +		regulator-min-microvolt = <3300000>;
> +		regulator-max-microvolt = <3300000>;
> +		gpio = <&i2c5_pcal6408 1 GPIO_ACTIVE_HIGH>;
> +		enable-active-high;
> +	};
> +
> +	reg_aqr_en: regulator-aqr-en {

keep order as node name

regulator-aqr-en should before regulator-aqr-stby

Frank

> +		compatible = "regulator-fixed";
> +		regulator-name = "aqr-en";
> +		regulator-min-microvolt = <3300000>;
> +		regulator-max-microvolt = <3300000>;
> +		vin-supply = <&reg_aqr_stby>;
> +		gpio = <&i2c5_pcal6408 0 GPIO_ACTIVE_HIGH>;
> +		enable-active-high;
> +	};
>  };
>
>  &adc1 {
> @@ -502,6 +521,7 @@ &netc_blk_ctrl {
>  &netc_emdio {
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&pinctrl_emdio>;
> +	phy-supply = <&reg_aqr_en>;
>  	status = "okay";
>
>  	ethphy0: ethernet-phy@1 {
>
> --
> 2.37.1
>

Patchew 2.1-devel-b67e4c2

© 2016 - 2025 Red Hat, Inc.