Re: [PATCH v2] imx8mp-dhcom-pdk3-overlay-rev100: Pass #address-cells/size-cells

2023-10-18 Thread Marek Vasut

On 10/18/23 19:54, Fabio Estevam wrote:

From: Fabio Estevam 

Pass #address-cells/size-cells in the mdio node to avoid the following
DTC warning:

   DTCOarch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dtbo
Warning (reg_format): /fragment@0/__overlay__:reg: property has invalid length 
(4 bytes) (#address-cells == 2, #size-cells == 1)
Warning (avoid_default_addr_size): /fragment@0/__overlay__: Relying on default 
#address-cells value
Warning (avoid_default_addr_size): /fragment@0/__overlay__: Relying on default 
#size-cells value

Signed-off-by: Fabio Estevam 
Reviewed-by: Marek Vasut 
---
Changes since v1:
- Remove the ethphy0g label. (Marek)

  arch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dts | 11 +--
  1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/arch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dts 
b/arch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dts
index f27e6429abe4..bad015536166 100644
--- a/arch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dts
+++ b/arch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dts
@@ -5,6 +5,13 @@
  /dts-v1/;
  /plugin/;
  
- {

-   reg = <7>;
+ {
+   mdio {
+   #address-cells = <1>;
+   #size-cells = <0>;
+
+   ethernet-phy@7 {


One more thing to fix, this should be ethernet-phy@5 , not 
ethernet-phy@7 . This is a local fix up for old hardware, which 
overrides the PHY MDIO address, see:


arch/arm/dts/imx8mp-dhcom-som.dtsi: ethphy0g: ethernet-phy@5 
{ /* Micrel KSZ9131RNXI */


[PATCH v2] imx8mp-dhcom-pdk3-overlay-rev100: Pass #address-cells/size-cells

2023-10-18 Thread Fabio Estevam
From: Fabio Estevam 

Pass #address-cells/size-cells in the mdio node to avoid the following
DTC warning:

  DTCOarch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dtbo
Warning (reg_format): /fragment@0/__overlay__:reg: property has invalid length 
(4 bytes) (#address-cells == 2, #size-cells == 1)
Warning (avoid_default_addr_size): /fragment@0/__overlay__: Relying on default 
#address-cells value
Warning (avoid_default_addr_size): /fragment@0/__overlay__: Relying on default 
#size-cells value

Signed-off-by: Fabio Estevam 
Reviewed-by: Marek Vasut 
---
Changes since v1:
- Remove the ethphy0g label. (Marek)

 arch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dts | 11 +--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/arch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dts 
b/arch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dts
index f27e6429abe4..bad015536166 100644
--- a/arch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dts
+++ b/arch/arm/dts/imx8mp-dhcom-pdk3-overlay-rev100.dts
@@ -5,6 +5,13 @@
 /dts-v1/;
 /plugin/;
 
- {
-   reg = <7>;
+ {
+   mdio {
+   #address-cells = <1>;
+   #size-cells = <0>;
+
+   ethernet-phy@7 {
+   reg = <7>;
+   };
+   };
 };
-- 
2.34.1