LKML Archive mirror
 help / color / mirror / Atom feed
* [PATCH] ARM: dts: hisilicon: Fix "status" values
@ 2023-06-26 22:10 Rob Herring
  2023-07-18 16:22 ` Rob Herring
  2023-07-19  7:43 ` Wei Xu
  0 siblings, 2 replies; 3+ messages in thread
From: Rob Herring @ 2023-06-26 22:10 UTC (permalink / raw
  To: Wei Xu, Krzysztof Kozlowski, Conor Dooley
  Cc: linux-arm-kernel, devicetree, linux-kernel

The defined value for "status" is "disabled", not "disable".

Signed-off-by: Rob Herring <robh@kernel.org>
---
 arch/arm/boot/dts/hisilicon/hi3519.dtsi | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/arch/arm/boot/dts/hisilicon/hi3519.dtsi b/arch/arm/boot/dts/hisilicon/hi3519.dtsi
index c524c854d319..a42b71cdc5d7 100644
--- a/arch/arm/boot/dts/hisilicon/hi3519.dtsi
+++ b/arch/arm/boot/dts/hisilicon/hi3519.dtsi
@@ -54,7 +54,7 @@ uart0: serial@12100000 {
 			interrupts = <GIC_SPI 4 IRQ_TYPE_LEVEL_HIGH>;
 			clocks = <&crg HI3519_UART0_CLK>, <&crg HI3519_UART0_CLK>;
 			clock-names = "uartclk", "apb_pclk";
-			status = "disable";
+			status = "disabled";
 		};
 
 		uart1: serial@12101000 {
@@ -63,7 +63,7 @@ uart1: serial@12101000 {
 			interrupts = <GIC_SPI 5 IRQ_TYPE_LEVEL_HIGH>;
 			clocks = <&crg HI3519_UART1_CLK>, <&crg HI3519_UART1_CLK>;
 			clock-names = "uartclk", "apb_pclk";
-			status = "disable";
+			status = "disabled";
 		};
 
 		uart2: serial@12102000 {
@@ -72,7 +72,7 @@ uart2: serial@12102000 {
 			interrupts = <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
 			clocks = <&crg HI3519_UART2_CLK>, <&crg HI3519_UART2_CLK>;
 			clock-names = "uartclk", "apb_pclk";
-			status = "disable";
+			status = "disabled";
 		};
 
 		uart3: serial@12103000 {
@@ -81,7 +81,7 @@ uart3: serial@12103000 {
 			interrupts = <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>;
 			clocks = <&crg HI3519_UART3_CLK>, <&crg HI3519_UART3_CLK>;
 			clock-names = "uartclk", "apb_pclk";
-			status = "disable";
+			status = "disabled";
 		};
 
 		uart4: serial@12104000 {
@@ -90,7 +90,7 @@ uart4: serial@12104000 {
 			interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>;
 			clocks = <&crg HI3519_UART4_CLK>, <&crg HI3519_UART4_CLK>;
 			clock-names = "uartclk", "apb_pclk";
-			status = "disable";
+			status = "disabled";
 		};
 
 		dual_timer0: timer@12000000 {
@@ -100,7 +100,7 @@ dual_timer0: timer@12000000 {
 			reg = <0x12000000 0x1000>;
 			clocks = <&clk_3m>;
 			clock-names = "apb_pclk";
-			status = "disable";
+			status = "disabled";
 		};
 
 		dual_timer1: timer@12001000 {
@@ -110,7 +110,7 @@ dual_timer1: timer@12001000 {
 			reg = <0x12001000 0x1000>;
 			clocks = <&clk_3m>;
 			clock-names = "apb_pclk";
-			status = "disable";
+			status = "disabled";
 		};
 
 		dual_timer2: timer@12002000 {
@@ -120,7 +120,7 @@ dual_timer2: timer@12002000 {
 			reg = <0x12002000 0x1000>;
 			clocks = <&clk_3m>;
 			clock-names = "apb_pclk";
-			status = "disable";
+			status = "disabled";
 		};
 
 		spi_bus0: spi@12120000 {
@@ -132,7 +132,7 @@ spi_bus0: spi@12120000 {
 			num-cs = <1>;
 			#address-cells = <1>;
 			#size-cells = <0>;
-			status = "disable";
+			status = "disabled";
 		};
 
 		spi_bus1: spi@12121000 {
@@ -144,7 +144,7 @@ spi_bus1: spi@12121000 {
 			num-cs = <1>;
 			#address-cells = <1>;
 			#size-cells = <0>;
-			status = "disable";
+			status = "disabled";
 		};
 
 		spi_bus2: spi@12122000 {
@@ -156,7 +156,7 @@ spi_bus2: spi@12122000 {
 			num-cs = <1>;
 			#address-cells = <1>;
 			#size-cells = <0>;
-			status = "disable";
+			status = "disabled";
 		};
 
 		sysctrl: system-controller@12020000 {
-- 
2.40.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: dts: hisilicon: Fix "status" values
  2023-06-26 22:10 [PATCH] ARM: dts: hisilicon: Fix "status" values Rob Herring
@ 2023-07-18 16:22 ` Rob Herring
  2023-07-19  7:43 ` Wei Xu
  1 sibling, 0 replies; 3+ messages in thread
From: Rob Herring @ 2023-07-18 16:22 UTC (permalink / raw
  To: Wei Xu, Krzysztof Kozlowski, Conor Dooley
  Cc: linux-arm-kernel, devicetree, linux-kernel

On Mon, Jun 26, 2023 at 4:10 PM Rob Herring <robh@kernel.org> wrote:
>
> The defined value for "status" is "disabled", not "disable".
>
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
>  arch/arm/boot/dts/hisilicon/hi3519.dtsi | 22 +++++++++++-----------
>  1 file changed, 11 insertions(+), 11 deletions(-)

Ping!

>
> diff --git a/arch/arm/boot/dts/hisilicon/hi3519.dtsi b/arch/arm/boot/dts/hisilicon/hi3519.dtsi
> index c524c854d319..a42b71cdc5d7 100644
> --- a/arch/arm/boot/dts/hisilicon/hi3519.dtsi
> +++ b/arch/arm/boot/dts/hisilicon/hi3519.dtsi
> @@ -54,7 +54,7 @@ uart0: serial@12100000 {
>                         interrupts = <GIC_SPI 4 IRQ_TYPE_LEVEL_HIGH>;
>                         clocks = <&crg HI3519_UART0_CLK>, <&crg HI3519_UART0_CLK>;
>                         clock-names = "uartclk", "apb_pclk";
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 uart1: serial@12101000 {
> @@ -63,7 +63,7 @@ uart1: serial@12101000 {
>                         interrupts = <GIC_SPI 5 IRQ_TYPE_LEVEL_HIGH>;
>                         clocks = <&crg HI3519_UART1_CLK>, <&crg HI3519_UART1_CLK>;
>                         clock-names = "uartclk", "apb_pclk";
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 uart2: serial@12102000 {
> @@ -72,7 +72,7 @@ uart2: serial@12102000 {
>                         interrupts = <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
>                         clocks = <&crg HI3519_UART2_CLK>, <&crg HI3519_UART2_CLK>;
>                         clock-names = "uartclk", "apb_pclk";
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 uart3: serial@12103000 {
> @@ -81,7 +81,7 @@ uart3: serial@12103000 {
>                         interrupts = <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>;
>                         clocks = <&crg HI3519_UART3_CLK>, <&crg HI3519_UART3_CLK>;
>                         clock-names = "uartclk", "apb_pclk";
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 uart4: serial@12104000 {
> @@ -90,7 +90,7 @@ uart4: serial@12104000 {
>                         interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>;
>                         clocks = <&crg HI3519_UART4_CLK>, <&crg HI3519_UART4_CLK>;
>                         clock-names = "uartclk", "apb_pclk";
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 dual_timer0: timer@12000000 {
> @@ -100,7 +100,7 @@ dual_timer0: timer@12000000 {
>                         reg = <0x12000000 0x1000>;
>                         clocks = <&clk_3m>;
>                         clock-names = "apb_pclk";
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 dual_timer1: timer@12001000 {
> @@ -110,7 +110,7 @@ dual_timer1: timer@12001000 {
>                         reg = <0x12001000 0x1000>;
>                         clocks = <&clk_3m>;
>                         clock-names = "apb_pclk";
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 dual_timer2: timer@12002000 {
> @@ -120,7 +120,7 @@ dual_timer2: timer@12002000 {
>                         reg = <0x12002000 0x1000>;
>                         clocks = <&clk_3m>;
>                         clock-names = "apb_pclk";
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 spi_bus0: spi@12120000 {
> @@ -132,7 +132,7 @@ spi_bus0: spi@12120000 {
>                         num-cs = <1>;
>                         #address-cells = <1>;
>                         #size-cells = <0>;
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 spi_bus1: spi@12121000 {
> @@ -144,7 +144,7 @@ spi_bus1: spi@12121000 {
>                         num-cs = <1>;
>                         #address-cells = <1>;
>                         #size-cells = <0>;
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 spi_bus2: spi@12122000 {
> @@ -156,7 +156,7 @@ spi_bus2: spi@12122000 {
>                         num-cs = <1>;
>                         #address-cells = <1>;
>                         #size-cells = <0>;
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 sysctrl: system-controller@12020000 {
> --
> 2.40.1
>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] ARM: dts: hisilicon: Fix "status" values
  2023-06-26 22:10 [PATCH] ARM: dts: hisilicon: Fix "status" values Rob Herring
  2023-07-18 16:22 ` Rob Herring
@ 2023-07-19  7:43 ` Wei Xu
  1 sibling, 0 replies; 3+ messages in thread
From: Wei Xu @ 2023-07-19  7:43 UTC (permalink / raw
  To: Rob Herring, Krzysztof Kozlowski, Conor Dooley
  Cc: linux-arm-kernel, devicetree, linux-kernel

Hi Rob,

On 2023/6/27 6:10, Rob Herring wrote:
> The defined value for "status" is "disabled", not "disable".
> 
> Signed-off-by: Rob Herring <robh@kernel.org>

Applied to the HiSilicon arm32 dt tree.
Thanks!

Best Regards,
Wei

> ---
>  arch/arm/boot/dts/hisilicon/hi3519.dtsi | 22 +++++++++++-----------
>  1 file changed, 11 insertions(+), 11 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/hisilicon/hi3519.dtsi b/arch/arm/boot/dts/hisilicon/hi3519.dtsi
> index c524c854d319..a42b71cdc5d7 100644
> --- a/arch/arm/boot/dts/hisilicon/hi3519.dtsi
> +++ b/arch/arm/boot/dts/hisilicon/hi3519.dtsi
> @@ -54,7 +54,7 @@ uart0: serial@12100000 {
>  			interrupts = <GIC_SPI 4 IRQ_TYPE_LEVEL_HIGH>;
>  			clocks = <&crg HI3519_UART0_CLK>, <&crg HI3519_UART0_CLK>;
>  			clock-names = "uartclk", "apb_pclk";
> -			status = "disable";
> +			status = "disabled";
>  		};
>  
>  		uart1: serial@12101000 {
> @@ -63,7 +63,7 @@ uart1: serial@12101000 {
>  			interrupts = <GIC_SPI 5 IRQ_TYPE_LEVEL_HIGH>;
>  			clocks = <&crg HI3519_UART1_CLK>, <&crg HI3519_UART1_CLK>;
>  			clock-names = "uartclk", "apb_pclk";
> -			status = "disable";
> +			status = "disabled";
>  		};
>  
>  		uart2: serial@12102000 {
> @@ -72,7 +72,7 @@ uart2: serial@12102000 {
>  			interrupts = <GIC_SPI 6 IRQ_TYPE_LEVEL_HIGH>;
>  			clocks = <&crg HI3519_UART2_CLK>, <&crg HI3519_UART2_CLK>;
>  			clock-names = "uartclk", "apb_pclk";
> -			status = "disable";
> +			status = "disabled";
>  		};
>  
>  		uart3: serial@12103000 {
> @@ -81,7 +81,7 @@ uart3: serial@12103000 {
>  			interrupts = <GIC_SPI 7 IRQ_TYPE_LEVEL_HIGH>;
>  			clocks = <&crg HI3519_UART3_CLK>, <&crg HI3519_UART3_CLK>;
>  			clock-names = "uartclk", "apb_pclk";
> -			status = "disable";
> +			status = "disabled";
>  		};
>  
>  		uart4: serial@12104000 {
> @@ -90,7 +90,7 @@ uart4: serial@12104000 {
>  			interrupts = <GIC_SPI 8 IRQ_TYPE_LEVEL_HIGH>;
>  			clocks = <&crg HI3519_UART4_CLK>, <&crg HI3519_UART4_CLK>;
>  			clock-names = "uartclk", "apb_pclk";
> -			status = "disable";
> +			status = "disabled";
>  		};
>  
>  		dual_timer0: timer@12000000 {
> @@ -100,7 +100,7 @@ dual_timer0: timer@12000000 {
>  			reg = <0x12000000 0x1000>;
>  			clocks = <&clk_3m>;
>  			clock-names = "apb_pclk";
> -			status = "disable";
> +			status = "disabled";
>  		};
>  
>  		dual_timer1: timer@12001000 {
> @@ -110,7 +110,7 @@ dual_timer1: timer@12001000 {
>  			reg = <0x12001000 0x1000>;
>  			clocks = <&clk_3m>;
>  			clock-names = "apb_pclk";
> -			status = "disable";
> +			status = "disabled";
>  		};
>  
>  		dual_timer2: timer@12002000 {
> @@ -120,7 +120,7 @@ dual_timer2: timer@12002000 {
>  			reg = <0x12002000 0x1000>;
>  			clocks = <&clk_3m>;
>  			clock-names = "apb_pclk";
> -			status = "disable";
> +			status = "disabled";
>  		};
>  
>  		spi_bus0: spi@12120000 {
> @@ -132,7 +132,7 @@ spi_bus0: spi@12120000 {
>  			num-cs = <1>;
>  			#address-cells = <1>;
>  			#size-cells = <0>;
> -			status = "disable";
> +			status = "disabled";
>  		};
>  
>  		spi_bus1: spi@12121000 {
> @@ -144,7 +144,7 @@ spi_bus1: spi@12121000 {
>  			num-cs = <1>;
>  			#address-cells = <1>;
>  			#size-cells = <0>;
> -			status = "disable";
> +			status = "disabled";
>  		};
>  
>  		spi_bus2: spi@12122000 {
> @@ -156,7 +156,7 @@ spi_bus2: spi@12122000 {
>  			num-cs = <1>;
>  			#address-cells = <1>;
>  			#size-cells = <0>;
> -			status = "disable";
> +			status = "disabled";
>  		};
>  
>  		sysctrl: system-controller@12020000 {
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-07-19  7:44 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-26 22:10 [PATCH] ARM: dts: hisilicon: Fix "status" values Rob Herring
2023-07-18 16:22 ` Rob Herring
2023-07-19  7:43 ` Wei Xu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).