Re: [PATCH v2 10/10] arch: arm: dts: k3-am62-sk-u-boot: Add missing "bootph-all" property to phy_gmii_sel node

[Roger Quadros <rogerq@kernel.org>]

On 25/04/2024 15:08, Chintan Vankar wrote:
> Add "bootph-all" property to CPSW MAC's PHY node phy_gmii_sel.
> 
> Signed-off-by: Chintan Vankar <c-vankar@ti.com>
> ---
> 
> Changes from v1 to v2:
> - This patch is newly added in this series to enable CPSW MAC's PHY
>   node "phy_gmii_sel". As per discussion at here:
>   https://lore.kernel.org/r/20240112130127.rvvrhz7p4vmlyalh@smother/
>   changes made by this patch can be dropped in the future when the
>   DT-Sync is performed with am62-main.dtsi containing this change in
>   the Linux DT which will match U-Boot's DT.

I don't think bootph-all exists in am62-main.dtsi. It should come from
board.dts

> 
>  arch/arm/dts/k3-am625-sk-u-boot.dtsi | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/dts/k3-am625-sk-u-boot.dtsi b/arch/arm/dts/k3-am625-sk-u-boot.dtsi
> index fa778b0ff4..e9a1afde95 100644
> --- a/arch/arm/dts/k3-am625-sk-u-boot.dtsi
> +++ b/arch/arm/dts/k3-am625-sk-u-boot.dtsi
> @@ -46,3 +46,7 @@
>  &cpsw_port2 {
>  	status = "disabled";
>  };
> +
> +&phy_gmii_sel {
> +	bootph-all;
> +};

-- 
cheers,
-roger