[PATCH v2 05/12] arm: dts: mt7623: fix all Warnings (unit_address_vs_reg)

2018-04-11 Thread sean.wang 
From: Sean Wang 

Fix below a lot of Warnings (unit_address_vs_reg) that dtc complains so
much for

Node /oscillator@1 has a unit name, but no reg property
Node /oscillator@0 has a unit name, but no reg property
Node /pinctrl@10005000/cir@0 has a unit name, but no reg property
Node /pinctrl@10005000/i2c@0 has a unit name, but no reg property
Node /pinctrl@10005000/i2c@1 has a unit name, but no reg property
Node /pinctrl@10005000/i2s@0 has a unit name, but no reg property
Node /pinctrl@10005000/i2s@1 has a unit name, but no reg property
Node /pinctrl@10005000/keys@0 has a unit name, but no reg property
Node /pinctrl@10005000/leds@0 has a unit name, but no reg property
Node /pinctrl@10005000/pwm@0 has a unit name, but no reg property
Node /pinctrl@10005000/spi@0 has a unit name, but no reg property
Node /pinctrl@10005000/uart@0 has a unit name, but no reg property
Node /pinctrl@10005000/uart@1 has a unit name, but no reg property
Node /pinctrl@10005000/uart@2 has a unit name, but no reg property
Node /ethernet@1b10/mdio-bus/switch@0/ports has a reg or ranges
property, but no unit name

Signed-off-by: Sean Wang 
Cc: Rob Herring 
Cc: Mark Rutland 
Cc: devicet...@vger.kernel.org
---
 arch/arm/boot/dts/mt7623.dtsi |  4 ++--
 arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts | 27 ---
 arch/arm/boot/dts/mt7623n-rfb.dtsi|  2 +-
 3 files changed, 15 insertions(+), 18 deletions(-)

diff --git a/arch/arm/boot/dts/mt7623.dtsi b/arch/arm/boot/dts/mt7623.dtsi
index 406a9f3..b9b5619 100644
--- a/arch/arm/boot/dts/mt7623.dtsi
+++ b/arch/arm/boot/dts/mt7623.dtsi
@@ -124,14 +124,14 @@
#clock-cells = <0>;
};
 
-   rtc32k: oscillator@1 {
+   rtc32k: oscillator-1 {
compatible = "fixed-clock";
#clock-cells = <0>;
clock-frequency = <32000>;
clock-output-names = "rtc32k";
};
 
-   clk26m: oscillator@0 {
+   clk26m: oscillator-0 {
compatible = "fixed-clock";
#clock-cells = <0>;
clock-frequency = <2600>;
diff --git a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts 
b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
index d2a354e..71023a6 100644
--- a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
+++ b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
@@ -145,8 +145,6 @@
 
switch@0 {
compatible = "mediatek,mt7530";
-   #address-cells = <1>;
-   #size-cells = <0>;
reg = <0>;
reset-gpios = < 33 0>;
core-supply = <_vpa_reg>;
@@ -155,7 +153,6 @@
ports {
#address-cells = <1>;
#size-cells = <0>;
-   reg = <0>;
 
port@0 {
reg = <0>;
@@ -259,14 +256,14 @@
 };
 
  {
-   cir_pins_a:cir@0 {
+   cir_pins_a:cir-default {
pins-cir {
pinmux = ;
bias-disable;
};
};
 
-   i2c0_pins_a: i2c@0 {
+   i2c0_pins_a: i2c0-default {
pins-i2c0 {
pinmux = ,
 ;
@@ -274,7 +271,7 @@
};
};
 
-   i2c1_pins_a: i2c@1 {
+   i2c1_pins_a: i2c1-default {
pin-i2c1 {
pinmux = ,
 ;
@@ -282,7 +279,7 @@
};
};
 
-   i2s0_pins_a: i2s@0 {
+   i2s0_pins_a: i2s0-default {
pin-i2s0 {
pinmux = ,
 ,
@@ -294,7 +291,7 @@
};
};
 
-   i2s1_pins_a: i2s@1 {
+   i2s1_pins_a: i2s1-default {
pin-i2s1 {
pinmux = ,
 ,
@@ -306,7 +303,7 @@
};
};
 
-   key_pins_a: keys@0 {
+   key_pins_a: keys-alt {
pins-keys {
pinmux = ,
  ;
@@ -314,7 +311,7 @@
};
};
 
-   led_pins_a: leds@0 {
+   led_pins_a: leds-alt {
pins-leds {
pinmux = ,
 ,
@@ -433,7 +430,7 @@
};
};
 
-   pwm_pins_a: pwm@0 {
+   pwm_pins_a: pwm-default {
pins-pwm {
pinmux = ,
 ,
@@ -443,7 +440,7 @@
};
};
 
-   spi0_pins_a: spi@0 {
+   spi0_pins_a: spi0-default {
pins-spi {
pinmux = ,
,
@@ -453,21 +450,21 @@
};

[PATCH v2 05/12] arm: dts: mt7623: fix all Warnings (unit_address_vs_reg)

2018-04-11 Thread sean.wang 
From: Sean Wang 

Fix below a lot of Warnings (unit_address_vs_reg) that dtc complains so
much for

Node /oscillator@1 has a unit name, but no reg property
Node /oscillator@0 has a unit name, but no reg property
Node /pinctrl@10005000/cir@0 has a unit name, but no reg property
Node /pinctrl@10005000/i2c@0 has a unit name, but no reg property
Node /pinctrl@10005000/i2c@1 has a unit name, but no reg property
Node /pinctrl@10005000/i2s@0 has a unit name, but no reg property
Node /pinctrl@10005000/i2s@1 has a unit name, but no reg property
Node /pinctrl@10005000/keys@0 has a unit name, but no reg property
Node /pinctrl@10005000/leds@0 has a unit name, but no reg property
Node /pinctrl@10005000/pwm@0 has a unit name, but no reg property
Node /pinctrl@10005000/spi@0 has a unit name, but no reg property
Node /pinctrl@10005000/uart@0 has a unit name, but no reg property
Node /pinctrl@10005000/uart@1 has a unit name, but no reg property
Node /pinctrl@10005000/uart@2 has a unit name, but no reg property
Node /ethernet@1b10/mdio-bus/switch@0/ports has a reg or ranges
property, but no unit name

Signed-off-by: Sean Wang 
Cc: Rob Herring 
Cc: Mark Rutland 
Cc: devicet...@vger.kernel.org
---
 arch/arm/boot/dts/mt7623.dtsi |  4 ++--
 arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts | 27 ---
 arch/arm/boot/dts/mt7623n-rfb.dtsi|  2 +-
 3 files changed, 15 insertions(+), 18 deletions(-)

diff --git a/arch/arm/boot/dts/mt7623.dtsi b/arch/arm/boot/dts/mt7623.dtsi
index 406a9f3..b9b5619 100644
--- a/arch/arm/boot/dts/mt7623.dtsi
+++ b/arch/arm/boot/dts/mt7623.dtsi
@@ -124,14 +124,14 @@
#clock-cells = <0>;
};
 
-   rtc32k: oscillator@1 {
+   rtc32k: oscillator-1 {
compatible = "fixed-clock";
#clock-cells = <0>;
clock-frequency = <32000>;
clock-output-names = "rtc32k";
};
 
-   clk26m: oscillator@0 {
+   clk26m: oscillator-0 {
compatible = "fixed-clock";
#clock-cells = <0>;
clock-frequency = <2600>;
diff --git a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts 
b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
index d2a354e..71023a6 100644
--- a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
+++ b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
@@ -145,8 +145,6 @@
 
switch@0 {
compatible = "mediatek,mt7530";
-   #address-cells = <1>;
-   #size-cells = <0>;
reg = <0>;
reset-gpios = < 33 0>;
core-supply = <_vpa_reg>;
@@ -155,7 +153,6 @@
ports {
#address-cells = <1>;
#size-cells = <0>;
-   reg = <0>;
 
port@0 {
reg = <0>;
@@ -259,14 +256,14 @@
 };
 
  {
-   cir_pins_a:cir@0 {
+   cir_pins_a:cir-default {
pins-cir {
pinmux = ;
bias-disable;
};
};
 
-   i2c0_pins_a: i2c@0 {
+   i2c0_pins_a: i2c0-default {
pins-i2c0 {
pinmux = ,
 ;
@@ -274,7 +271,7 @@
};
};
 
-   i2c1_pins_a: i2c@1 {
+   i2c1_pins_a: i2c1-default {
pin-i2c1 {
pinmux = ,
 ;
@@ -282,7 +279,7 @@
};
};
 
-   i2s0_pins_a: i2s@0 {
+   i2s0_pins_a: i2s0-default {
pin-i2s0 {
pinmux = ,
 ,
@@ -294,7 +291,7 @@
};
};
 
-   i2s1_pins_a: i2s@1 {
+   i2s1_pins_a: i2s1-default {
pin-i2s1 {
pinmux = ,
 ,
@@ -306,7 +303,7 @@
};
};
 
-   key_pins_a: keys@0 {
+   key_pins_a: keys-alt {
pins-keys {
pinmux = ,
  ;
@@ -314,7 +311,7 @@
};
};
 
-   led_pins_a: leds@0 {
+   led_pins_a: leds-alt {
pins-leds {
pinmux = ,
 ,
@@ -433,7 +430,7 @@
};
};
 
-   pwm_pins_a: pwm@0 {
+   pwm_pins_a: pwm-default {
pins-pwm {
pinmux = ,
 ,
@@ -443,7 +440,7 @@
};
};
 
-   spi0_pins_a: spi@0 {
+   spi0_pins_a: spi0-default {
pins-spi {
pinmux = ,
,
@@ -453,21 +450,21 @@
};
};
 
-   uart0_pins_a: uart@0 {
+   uart0_pins_a: uart0-default {