Re: [PATCH] ARM: dts: am335x: Replace remaining legacy phy_id with phy-handle

[Grygorii Strashko <grygorii.strashko@xxxxxx>]

On 09/28/2018 03:34 PM, Tony Lindgren wrote:
> Looks like we still have two instances of phy_handle that did not
> get update by Grygorii's series. Let's replace these too with
> standard phy-handle.
>
> Cc: Arnd Bergmann <arnd@xxxxxxxx>
> Cc: Grygorii Strashko <grygorii.strashko@xxxxxx>
> Cc: Koen Kooi <koen@xxxxxxxxxxxxxxxxxxxxx>
> Cc: Neeraj Dantu <dantuguf14105@xxxxxxxxx>
> Reported-by: Arnd Bergmann <arnd@xxxxxxxx>
> Signed-off-by: Tony Lindgren <tony@xxxxxxxxxxx>

Reviewed-by: Grygorii Strashko <grygorii.strashko@xxxxxx>

> ---
>   arch/arm/boot/dts/am335x-osd3358-sm-red.dts | 6 +++++-
>   arch/arm/boot/dts/am335x-sancloud-bbe.dts   | 6 +++++-
>   2 files changed, 10 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/am335x-osd3358-sm-red.dts b/arch/arm/boot/dts/am335x-osd3358-sm-red.dts
> --- a/arch/arm/boot/dts/am335x-osd3358-sm-red.dts
> +++ b/arch/arm/boot/dts/am335x-osd3358-sm-red.dts
> @@ -424,7 +424,7 @@
>   };
>   
>   &cpsw_emac0 {
> -	phy_id = <&davinci_mdio>, <4>;
> +	phy-handle = <&ethphy0>;
>   	phy-mode = "rgmii-txid";
>   };
>   
> @@ -441,6 +441,10 @@
>   	pinctrl-0 = <&davinci_mdio_default>;
>   	pinctrl-1 = <&davinci_mdio_sleep>;
>   	status = "okay";
> +
> +	ethphy0: ethernet-phy@4 {
> +		reg = <4>;
> +	};
>   };
>   
>   &mmc1 {
> diff --git a/arch/arm/boot/dts/am335x-sancloud-bbe.dts b/arch/arm/boot/dts/am335x-sancloud-bbe.dts
> --- a/arch/arm/boot/dts/am335x-sancloud-bbe.dts
> +++ b/arch/arm/boot/dts/am335x-sancloud-bbe.dts
> @@ -103,10 +103,14 @@
>   	pinctrl-0 = <&davinci_mdio_default>;
>   	pinctrl-1 = <&davinci_mdio_sleep>;
>   	status = "okay";
> +
> +	ethphy0: ethernet-phy@0 {
> +		reg = <0>;
> +	};
>   };
>   
>   &cpsw_emac0 {
> -	phy_id = <&davinci_mdio>, <0>;
> +	phy-handle = <&ethphy0>;
>   	phy-mode = "rgmii-txid";
>   };
>   

--
regards,
-grygorii