Re: [PATCH net-next v2 4/7] ARM: dts: at91: sama5d2: add an mdio sub-node to macb

[Florian Fainelli]

On 7/21/20 10:13 AM, Codrin Ciubotariu wrote:
> Use the new macb bindings and add an mdio sub-node to contain all the
> phy nodes.
> 
> Signed-off-by: Codrin Ciubotariu 

Reviewed-by: Florian Fainelli 
-- 
Florian

[PATCH net-next v2 4/7] ARM: dts: at91: sama5d2: add an mdio sub-node to macb

[Codrin Ciubotariu]

Use the new macb bindings and add an mdio sub-node to contain all the
phy nodes.

Signed-off-by: Codrin Ciubotariu 
---

Changes in v2:
 - none

 arch/arm/boot/dts/at91-sama5d27_som1.dtsi   | 16 ++--
 arch/arm/boot/dts/at91-sama5d27_wlsom1.dtsi | 17 ++---
 arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts   | 13 -
 arch/arm/boot/dts/at91-sama5d2_xplained.dts | 12 
 4 files changed, 36 insertions(+), 22 deletions(-)

diff --git a/arch/arm/boot/dts/at91-sama5d27_som1.dtsi 
b/arch/arm/boot/dts/at91-sama5d27_som1.dtsi
index b1f994c0ae79..dfcee23dcce0 100644
--- a/arch/arm/boot/dts/at91-sama5d27_som1.dtsi
+++ b/arch/arm/boot/dts/at91-sama5d27_som1.dtsi
@@ -84,12 +84,16 @@ macb0: ethernet@f8008000 {
pinctrl-0 = <_macb0_default>;
phy-mode = "rmii";
 
-   ethernet-phy@0 {
-   reg = <0x0>;
-   interrupt-parent = <>;
-   interrupts = ;
-   pinctrl-names = "default";
-   pinctrl-0 = <_macb0_phy_irq>;
+   mdio {
+   #address-cells = <1>;
+   #size-cells = <0>;
+   ethernet-phy@0 {
+   reg = <0x0>;
+   interrupt-parent = <>;
+   interrupts = ;
+   pinctrl-names = "default";
+   pinctrl-0 = 
<_macb0_phy_irq>;
+   };
};
};
 
diff --git a/arch/arm/boot/dts/at91-sama5d27_wlsom1.dtsi 
b/arch/arm/boot/dts/at91-sama5d27_wlsom1.dtsi
index a06700e53e4c..9c4dce29d2fe 100644
--- a/arch/arm/boot/dts/at91-sama5d27_wlsom1.dtsi
+++ b/arch/arm/boot/dts/at91-sama5d27_wlsom1.dtsi
@@ -181,13 +181,16 @@  {
pinctrl-names = "default";
pinctrl-0 = <_macb0_default>;
phy-mode = "rmii";
-
-   ethernet-phy@0 {
-   reg = <0x0>;
-   interrupt-parent = <>;
-   interrupts = ;
-   pinctrl-names = "default";
-   pinctrl-0 = <_macb0_phy_irq>;
+   mdio {
+   #address-cells = <1>;
+   #size-cells = <0>;
+   ethernet-phy@0 {
+   reg = <0x0>;
+   interrupt-parent = <>;
+   interrupts = ;
+   pinctrl-names = "default";
+   pinctrl-0 = <_macb0_phy_irq>;
+   };
};
 };
 
diff --git a/arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts 
b/arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts
index c894c7c788a9..fc3375c43ef6 100644
--- a/arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts
+++ b/arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts
@@ -140,11 +140,14 @@ macb0: ethernet@f8008000 {
pinctrl-0 = <_macb0_default 
_macb0_phy_irq>;
phy-mode = "rmii";
status = "okay";
-
-   ethernet-phy@1 {
-   reg = <0x1>;
-   interrupt-parent = <>;
-   interrupts = <56 IRQ_TYPE_LEVEL_LOW>;
+   mdio {
+   #address-cells = <1>;
+   #size-cells = <0>;
+   ethernet-phy@1 {
+   reg = <0x1>;
+   interrupt-parent = <>;
+   interrupts = <56 
IRQ_TYPE_LEVEL_LOW>;
+   };
};
};
 
diff --git a/arch/arm/boot/dts/at91-sama5d2_xplained.dts 
b/arch/arm/boot/dts/at91-sama5d2_xplained.dts
index a927165ea7c2..a62f475d9d0a 100644
--- a/arch/arm/boot/dts/at91-sama5d2_xplained.dts
+++ b/arch/arm/boot/dts/at91-sama5d2_xplained.dts
@@ -149,10 +149,14 @@ macb0: ethernet@f8008000 {
phy-mode = "rmii";
status = "okay";
 
-   ethernet-phy@1 {
-   reg = <0x1>;
-   interrupt-parent = <>;
-   interrupts = ;
+   mdio {
+   #address-cells = <1>;
+   #size-cells = <0>;
+   ethernet-phy@1 {
+   reg =