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

[Koen Kooi <koen@xxxxxxxxxxxxxxxxxxxxx>]

> Op 28 sep. 2018, om 22:34 heeft Tony Lindgren <tony@xxxxxxxxxxx> het volgende geschreven:
> 
> 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>

Thanks for taking care of this!

Acked-by: Koen Kooi <koen@xxxxxxxxxxxxxxxxxxxxx>

> ---
> 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";
> };
> 
> -- 
> 2.19.0