[RESEND,v3] arm64: dts: mediatek: align GPIO hog names with dtschema
diff mbox series

Message ID 20201002163940.7837-1-krzk@kernel.org
State New, archived
Headers show
Series
  • [RESEND,v3] arm64: dts: mediatek: align GPIO hog names with dtschema
Related show

Commit Message

Krzysztof Kozlowski Oct. 2, 2020, 4:39 p.m. UTC
The convention for node names is to use hyphens, not underscores.
dtschema for pca95xx expects GPIO hogs to end with 'hog' suffix.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 .../boot/dts/mediatek/pumpkin-common.dtsi     | 26 +++++++++----------
 1 file changed, 13 insertions(+), 13 deletions(-)

The dt-schema changes were applied.

Comments

Matthias Brugger Oct. 29, 2020, 3:52 p.m. UTC | #1
On 02/10/2020 18:39, Krzysztof Kozlowski wrote:
> The convention for node names is to use hyphens, not underscores.
> dtschema for pca95xx expects GPIO hogs to end with 'hog' suffix.
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

Pushed to v5.10-next/dts64

Thanks!

> ---
>   .../boot/dts/mediatek/pumpkin-common.dtsi     | 26 +++++++++----------
>   1 file changed, 13 insertions(+), 13 deletions(-)
> 
> The dt-schema changes were applied.
> 
> diff --git a/arch/arm64/boot/dts/mediatek/pumpkin-common.dtsi b/arch/arm64/boot/dts/mediatek/pumpkin-common.dtsi
> index 29d8cf6df46b..351a1905a074 100644
> --- a/arch/arm64/boot/dts/mediatek/pumpkin-common.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/pumpkin-common.dtsi
> @@ -63,91 +63,91 @@
>   		gpio-controller;
>   		#gpio-cells = <2>;
>   
> -		eint20_mux_sel0 {
> +		eint20-mux-sel0-hog {
>   			gpio-hog;
>   			gpios = <0 0>;
>   			input;
>   			line-name = "eint20_mux_sel0";
>   		};
>   
> -		expcon_mux_sel1 {
> +		expcon-mux-sel1-hog {
>   			gpio-hog;
>   			gpios = <1 0>;
>   			input;
>   			line-name = "expcon_mux_sel1";
>   		};
>   
> -		mrg_di_mux_sel2 {
> +		mrg-di-mux-sel2-hog {
>   			gpio-hog;
>   			gpios = <2 0>;
>   			input;
>   			line-name = "mrg_di_mux_sel2";
>   		};
>   
> -		sd_sdio_mux_sel3 {
> +		sd-sdio-mux-sel3-hog {
>   			gpio-hog;
>   			gpios = <3 0>;
>   			input;
>   			line-name = "sd_sdio_mux_sel3";
>   		};
>   
> -		sd_sdio_mux_ctrl7 {
> +		sd-sdio-mux-ctrl7-hog {
>   			gpio-hog;
>   			gpios = <7 0>;
>   			output-low;
>   			line-name = "sd_sdio_mux_ctrl7";
>   		};
>   
> -		hw_id0 {
> +		hw-id0-hog {
>   			gpio-hog;
>   			gpios = <8 0>;
>   			input;
>   			line-name = "hw_id0";
>   		};
>   
> -		hw_id1 {
> +		hw-id1-hog {
>   			gpio-hog;
>   			gpios = <9 0>;
>   			input;
>   			line-name = "hw_id1";
>   		};
>   
> -		hw_id2 {
> +		hw-id2-hog {
>   			gpio-hog;
>   			gpios = <10 0>;
>   			input;
>   			line-name = "hw_id2";
>   		};
>   
> -		fg_int_n {
> +		fg-int-n-hog {
>   			gpio-hog;
>   			gpios = <11 0>;
>   			input;
>   			line-name = "fg_int_n";
>   		};
>   
> -		usba_pwr_en {
> +		usba-pwr-en-hog {
>   			gpio-hog;
>   			gpios = <12 0>;
>   			output-high;
>   			line-name = "usba_pwr_en";
>   		};
>   
> -		wifi_3v3_pg {
> +		wifi-3v3-pg-hog {
>   			gpio-hog;
>   			gpios = <13 0>;
>   			input;
>   			line-name = "wifi_3v3_pg";
>   		};
>   
> -		cam_rst {
> +		cam-rst-hog {
>   			gpio-hog;
>   			gpios = <14 0>;
>   			output-low;
>   			line-name = "cam_rst";
>   		};
>   
> -		cam_pwdn {
> +		cam-pwdn-hog {
>   			gpio-hog;
>   			gpios = <15 0>;
>   			output-low;
>

Patch
diff mbox series

diff --git a/arch/arm64/boot/dts/mediatek/pumpkin-common.dtsi b/arch/arm64/boot/dts/mediatek/pumpkin-common.dtsi
index 29d8cf6df46b..351a1905a074 100644
--- a/arch/arm64/boot/dts/mediatek/pumpkin-common.dtsi
+++ b/arch/arm64/boot/dts/mediatek/pumpkin-common.dtsi
@@ -63,91 +63,91 @@ 
 		gpio-controller;
 		#gpio-cells = <2>;
 
-		eint20_mux_sel0 {
+		eint20-mux-sel0-hog {
 			gpio-hog;
 			gpios = <0 0>;
 			input;
 			line-name = "eint20_mux_sel0";
 		};
 
-		expcon_mux_sel1 {
+		expcon-mux-sel1-hog {
 			gpio-hog;
 			gpios = <1 0>;
 			input;
 			line-name = "expcon_mux_sel1";
 		};
 
-		mrg_di_mux_sel2 {
+		mrg-di-mux-sel2-hog {
 			gpio-hog;
 			gpios = <2 0>;
 			input;
 			line-name = "mrg_di_mux_sel2";
 		};
 
-		sd_sdio_mux_sel3 {
+		sd-sdio-mux-sel3-hog {
 			gpio-hog;
 			gpios = <3 0>;
 			input;
 			line-name = "sd_sdio_mux_sel3";
 		};
 
-		sd_sdio_mux_ctrl7 {
+		sd-sdio-mux-ctrl7-hog {
 			gpio-hog;
 			gpios = <7 0>;
 			output-low;
 			line-name = "sd_sdio_mux_ctrl7";
 		};
 
-		hw_id0 {
+		hw-id0-hog {
 			gpio-hog;
 			gpios = <8 0>;
 			input;
 			line-name = "hw_id0";
 		};
 
-		hw_id1 {
+		hw-id1-hog {
 			gpio-hog;
 			gpios = <9 0>;
 			input;
 			line-name = "hw_id1";
 		};
 
-		hw_id2 {
+		hw-id2-hog {
 			gpio-hog;
 			gpios = <10 0>;
 			input;
 			line-name = "hw_id2";
 		};
 
-		fg_int_n {
+		fg-int-n-hog {
 			gpio-hog;
 			gpios = <11 0>;
 			input;
 			line-name = "fg_int_n";
 		};
 
-		usba_pwr_en {
+		usba-pwr-en-hog {
 			gpio-hog;
 			gpios = <12 0>;
 			output-high;
 			line-name = "usba_pwr_en";
 		};
 
-		wifi_3v3_pg {
+		wifi-3v3-pg-hog {
 			gpio-hog;
 			gpios = <13 0>;
 			input;
 			line-name = "wifi_3v3_pg";
 		};
 
-		cam_rst {
+		cam-rst-hog {
 			gpio-hog;
 			gpios = <14 0>;
 			output-low;
 			line-name = "cam_rst";
 		};
 
-		cam_pwdn {
+		cam-pwdn-hog {
 			gpio-hog;
 			gpios = <15 0>;
 			output-low;