Re: [PATCH v1 11/19] arm: dts: mt7623: use - instead of _ in DT node name

[Matthias Brugger]

On 02/23/2018 11:16 AM, sean.w...@mediatek.com wrote:
> From: Sean Wang 
> 
> It should be good that no use "_" is in DT node name. Consequently,
> those nodes in certain files which have an inappropriate name containing
> "_" are all being replaced with "-".
> 
> Signed-off-by: Sean Wang 
> Cc: Rob Herring 
> Cc: Mark Rutland 
> Cc: devicet...@vger.kernel.org

Pushed to v4.16-next/dts32

Thanks!

> ---
>  arch/arm/boot/dts/mt7623.dtsi | 12 +++
>  arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts | 50 
> +--
>  arch/arm/boot/dts/mt7623n-rfb-nand.dts|  6 ++--
>  3 files changed, 34 insertions(+), 34 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/mt7623.dtsi b/arch/arm/boot/dts/mt7623.dtsi
> index 5cf93a4..4b68448 100644
> --- a/arch/arm/boot/dts/mt7623.dtsi
> +++ b/arch/arm/boot/dts/mt7623.dtsi
> @@ -28,7 +28,7 @@
>   compatible = "mediatek,mt7623";
>   interrupt-parent = <&sysirq>;
>  
> - cpu_opp_table: opp_table {
> + cpu_opp_table: opp-table {
>   compatible = "operating-points-v2";
>   opp-shared;
>  
> @@ -145,32 +145,32 @@
>   };
>  
>   thermal-zones {
> - cpu_thermal: cpu_thermal {
> + cpu_thermal: cpu-thermal {
>   polling-delay-passive = <1000>;
>   polling-delay = <1000>;
>  
>   thermal-sensors = <&thermal 0>;
>  
>   trips {
> - cpu_passive: cpu_passive {
> + cpu_passive: cpu-passive {
>   temperature = <47000>;
>   hysteresis = <2000>;
>   type = "passive";
>   };
>  
> - cpu_active: cpu_active {
> + cpu_active: cpu-active {
>   temperature = <67000>;
>   hysteresis = <2000>;
>   type = "active";
>   };
>  
> - cpu_hot: cpu_hot {
> + cpu_hot: cpu-hot {
>   temperature = <87000>;
>   hysteresis = <2000>;
>   type = "hot";
>   };
>  
> - cpu_crit {
> + cpu-crit {
>   temperature = <107000>;
>   hysteresis = <2000>;
>   type = "critical";
> diff --git a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts 
> b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
> index f2d3cfb..a6295ef 100644
> --- a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
> +++ b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
> @@ -66,7 +66,7 @@
>   regulator-always-on;
>   };
>  
> - gpio_keys {
> + gpio-keys {
>   compatible = "gpio-keys";
>   pinctrl-names = "default";
>   pinctrl-0 = <&key_pins_a>;
> @@ -263,14 +263,14 @@
>  
>  &pio {
>   cir_pins_a:cir@0 {
> - pins_cir {
> + pins-cir {
>   pinmux = ;
>   bias-disable;
>   };
>   };
>  
>   i2c0_pins_a: i2c@0 {
> - pins_i2c0 {
> + pins-i2c0 {
>   pinmux = ,
>;
>   bias-disable;
> @@ -278,7 +278,7 @@
>   };
>  
>   i2c1_pins_a: i2c@1 {
> - pin_i2c1 {
> + pin-i2c1 {
>   pinmux = ,
>;
>   bias-disable;
> @@ -286,7 +286,7 @@
>   };
>  
>   i2s0_pins_a: i2s@0 {
> - pin_i2s0 {
> + pin-i2s0 {
>   pinmux = ,
>,
>,
> @@ -298,7 +298,7 @@
>   };
>  
>   i2s1_pins_a: i2s@1 {
> - pin_i2s1 {
> + pin-i2s1 {
>   pinmux = ,
>,
>,
> @@ -310,7 +310,7 @@
>   };
>  
>   key_pins_a: keys@0 {
> - pins_keys {
> + pins-keys {
>   pinmux = ,
> ;
>   input-enable;
> @@ -318,7 +318,7 @@
>   };
>  
>   led_pins_a: leds@0 {
> - pins_leds {
> + pins-leds {
>   pinmux = ,
>,
>  

[PATCH v1 11/19] arm: dts: mt7623: use - instead of _ in DT node name

[sean.wang]

From: Sean Wang 

It should be good that no use "_" is in DT node name. Consequently,
those nodes in certain files which have an inappropriate name containing
"_" are all being replaced with "-".

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

diff --git a/arch/arm/boot/dts/mt7623.dtsi b/arch/arm/boot/dts/mt7623.dtsi
index 5cf93a4..4b68448 100644
--- a/arch/arm/boot/dts/mt7623.dtsi
+++ b/arch/arm/boot/dts/mt7623.dtsi
@@ -28,7 +28,7 @@
compatible = "mediatek,mt7623";
interrupt-parent = <&sysirq>;
 
-   cpu_opp_table: opp_table {
+   cpu_opp_table: opp-table {
compatible = "operating-points-v2";
opp-shared;
 
@@ -145,32 +145,32 @@
};
 
thermal-zones {
-   cpu_thermal: cpu_thermal {
+   cpu_thermal: cpu-thermal {
polling-delay-passive = <1000>;
polling-delay = <1000>;
 
thermal-sensors = <&thermal 0>;
 
trips {
-   cpu_passive: cpu_passive {
+   cpu_passive: cpu-passive {
temperature = <47000>;
hysteresis = <2000>;
type = "passive";
};
 
-   cpu_active: cpu_active {
+   cpu_active: cpu-active {
temperature = <67000>;
hysteresis = <2000>;
type = "active";
};
 
-   cpu_hot: cpu_hot {
+   cpu_hot: cpu-hot {
temperature = <87000>;
hysteresis = <2000>;
type = "hot";
};
 
-   cpu_crit {
+   cpu-crit {
temperature = <107000>;
hysteresis = <2000>;
type = "critical";
diff --git a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts 
b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
index f2d3cfb..a6295ef 100644
--- a/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
+++ b/arch/arm/boot/dts/mt7623n-bananapi-bpi-r2.dts
@@ -66,7 +66,7 @@
regulator-always-on;
};
 
-   gpio_keys {
+   gpio-keys {
compatible = "gpio-keys";
pinctrl-names = "default";
pinctrl-0 = <&key_pins_a>;
@@ -263,14 +263,14 @@
 
 &pio {
cir_pins_a:cir@0 {
-   pins_cir {
+   pins-cir {
pinmux = ;
bias-disable;
};
};
 
i2c0_pins_a: i2c@0 {
-   pins_i2c0 {
+   pins-i2c0 {
pinmux = ,
 ;
bias-disable;
@@ -278,7 +278,7 @@
};
 
i2c1_pins_a: i2c@1 {
-   pin_i2c1 {
+   pin-i2c1 {
pinmux = ,
 ;
bias-disable;
@@ -286,7 +286,7 @@
};
 
i2s0_pins_a: i2s@0 {
-   pin_i2s0 {
+   pin-i2s0 {
pinmux = ,
 ,
 ,
@@ -298,7 +298,7 @@
};
 
i2s1_pins_a: i2s@1 {
-   pin_i2s1 {
+   pin-i2s1 {
pinmux = ,
 ,
 ,
@@ -310,7 +310,7 @@
};
 
key_pins_a: keys@0 {
-   pins_keys {
+   pins-keys {
pinmux = ,
  ;
input-enable;
@@ -318,7 +318,7 @@
};
 
led_pins_a: leds@0 {
-   pins_leds {
+   pins-leds {
pinmux = ,
 ,
 ;
@@ -326,7 +326,7 @@
};
 
mmc0_pins_default: mmc0default {
-   pins_cmd_dat {
+   pins-cmd-dat {
pinmux = ,