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
2024-04-25T12:31:23Z

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