Skip to contentSkip to conversation

arm: dts: vf6: fix DTC unit warnings

PATCH New Heiko Schocher 2 participants 2 messages Apr 18, 2016 Arm (32-bit)

lore
Share & tools
lore
#0 lore raw

PATCH v2 1/1 arm: dts: vf6: fix DTC unit warnings

Fix following warnings for all vf6 based boards:

Warning (unit_address_vs_reg): Node /soc/ifc at 1530000/board-control at 3,0/mdio-mux-emi1 has a reg or ranges property, but no unit name Warning (unit_address_vs_reg): Node /soc/aips-bus at 40000000 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /soc/aips-bus at 40080000 has a unit name, but no reg property

---
This warnings pop up with the DTC compiler:

$ /tmp/dtc/dtc -v
Version: DTC 1.4.1-gbeef80b8

This fixes the compile warnings for:
https://travis-ci.org/u-boot/u-boot/jobs/122880431

see:
https://travis-ci.org/hsdenx/u-boot/jobs/123238780

Changes in v2:
- added comment from bhuvanchandra.dv at toradex.com :
used the changes Stefan Agner posted o LKML:
https://lkml.org/lkml/2016/4/2/11

 arch/arm/dts/ls1021a-qds.dtsi | 2 +-
 arch/arm/dts/vf.dtsi          | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)
arch/arm/dts/ls1021a-qds.dtsi +1 -1
index ca9e835..4cb6337 100644
@@ -138,7 +138,7 @@
 		device-width = <1>;
 		ranges = <0 3 0 0x100>;
 
-		mdio-mux-emi1 {
+		mdio-mux-emi1 at 54 {
 			compatible = "mdio-mux-mmioreg";
 			mdio-parent-bus = <&mdio0>;
 			#address-cells = <1>;
arch/arm/dts/vf.dtsi +2 -0
index 1530d2f..c10c564 100644
@@ -32,6 +32,7 @@
 			compatible = "fsl,aips-bus", "simple-bus";
 			#address-cells = <1>;
 			#size-cells = <1>;
+			reg = <0x40000000 0x00070000>;
 			ranges;
 
 			uart0: serial at 40027000 {
@@ -119,6 +120,7 @@
 			compatible = "fsl,aips-bus", "simple-bus";
 			#address-cells = <1>;
 			#size-cells = <1>;
+			reg = <0x40080000 0x0007f000>;
 			ranges;
 
 			uart4: serial at 400a9000 {
Signature · 1 line
2.5.0
#1 lore raw

PATCH v2 1/1 arm: dts: vf6: fix DTC unit warnings

Quoted 1 line · Heiko Schocher

Fix following warnings for all vf6 based boards:

Freescale Vybrid and ls1021xx are different SoCs, may be we can split this patchset.

Apart from that.

Quoted 61 lines

Warning (unit_address_vs_reg): Node /soc/ifc at 1530000/board-control at 3,0/mdio-mux-emi1 has a reg or ranges property, but no unit name Warning (unit_address_vs_reg): Node /soc/aips-bus at 40000000 has a unit name, but no reg property
Warning (unit_address_vs_reg): Node /soc/aips-bus at 40080000 has a unit name, but no reg property

---
This warnings pop up with the DTC compiler:

$ /tmp/dtc/dtc -v
Version: DTC 1.4.1-gbeef80b8

This fixes the compile warnings for:
https://travis-ci.org/u-boot/u-boot/jobs/122880431

see:
https://travis-ci.org/hsdenx/u-boot/jobs/123238780

Changes in v2:
- added comment from bhuvanchandra.dv at toradex.com :
used the changes Stefan Agner posted o LKML:

   https://lkml.org/lkml/2016/4/2/11
  arch/arm/dts/ls1021a-qds.dtsi | 2 +-
  arch/arm/dts/vf.dtsi          | 2 ++
  2 files changed, 3 insertions(+), 1 deletion(-)
arch/arm/dts/ls1021a-qds.dtsi +1 -1
index ca9e835..4cb6337 100644
@@ -138,7 +138,7 @@
  		device-width = <1>;
  		ranges = <0 3 0 0x100>;
 
-		mdio-mux-emi1 {
+		mdio-mux-emi1 at 54 {
  			compatible = "mdio-mux-mmioreg";
  			mdio-parent-bus = <&mdio0>;
  			#address-cells = <1>;
arch/arm/dts/vf.dtsi +2 -0
index 1530d2f..c10c564 100644
@@ -32,6 +32,7 @@
  			compatible = "fsl,aips-bus", "simple-bus";
  			#address-cells = <1>;
  			#size-cells = <1>;
+			reg = <0x40000000 0x00070000>;
  			ranges;
 
  			uart0: serial at 40027000 {
@@ -119,6 +120,7 @@
  			compatible = "fsl,aips-bus", "simple-bus";
  			#address-cells = <1>;
  			#size-cells = <1>;
+			reg = <0x40080000 0x0007f000>;
  			ranges;
 
  			uart4: serial at 400a9000 {
Signature · 2 lines
Best regards,
Bhuvan