Re: [PATCH v2 1/4] arm64: dts: ti: var-som-am62p: fix Ethernet PHY configuration

[Vignesh Raghavendra <vigneshr@ti.com>]

On 29/06/26 02:26, Stefano Radaelli wrote:
> From: Stefano Radaelli <stefano.r@variscite.com>
> 
> Fix the Ethernet device tree description on the VAR-SOM-AM62P.
> 
> Enable the CPSW Ethernet controller and correct the Ethernet PHY
> description by modelling the PHY power supply and adding the required
> board-specific PHY properties.
> 
> Signed-off-by: Stefano Radaelli <stefano.r@variscite.com>
> ---
> v1->v2:
>  - Remove unused eth property
> 
>  arch/arm64/boot/dts/ti/k3-am62p5-var-som.dtsi | 16 +++++++++++++---
>  1 file changed, 13 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am62p5-var-som.dtsi b/arch/arm64/boot/dts/ti/k3-am62p5-var-som.dtsi
> index fc5a3942cde0..ca2483a04b9d 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62p5-var-som.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am62p5-var-som.dtsi
> @@ -113,6 +113,15 @@ reg_3v3_phy: regulator-3v3-phy {
>  		regulator-always-on;
>  	};
>  
> +	reg_eth_phy_vdd: regulator-eth-vdd {
> +		compatible = "regulator-fixed";
> +		regulator-name = "reg_eth_phy_vdd";
> +		regulator-min-microvolt = <3300000>;
> +		regulator-max-microvolt = <3300000>;
> +		gpio = <&main_gpio0 46 GPIO_ACTIVE_HIGH>;
> +		enable-active-high;
> +	};
> +
>  	sound {
>  		compatible = "simple-audio-card";
>  		simple-audio-card,bitclock-master = <&codec_dai>;
> @@ -149,6 +158,7 @@ &audio_refclk1 {
>  &cpsw3g {
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&pinctrl_rgmii1>;
> +	status = "okay";
>  };
>  
>  &cpsw3g_mdio {
> @@ -159,9 +169,9 @@ &cpsw3g_mdio {
>  	cpsw3g_phy0: ethernet-phy@4 {
>  		compatible = "ethernet-phy-id0283.bc30";
>  		reg = <4>;
> -		reset-gpios = <&main_gpio0 46 GPIO_ACTIVE_LOW>;
> -		reset-assert-us = <10000>;
> -		reset-deassert-us = <100000>;

So are the reset delays no longer required? Or they taken care of
somewhere else?

> +		bootph-all;
> +		enet-phy-lane-no-swap;
> +		vdd-supply = <&reg_eth_phy_vdd>;
>  	};
>  };
>  

-- 
Regards
Vignesh
https://ti.com/opensource