2023-06-26 22:22:11

by Rob Herring

Subject: [PATCH] ARM: dts: hisilicon: Fix "status" values

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

Signed-off-by: Rob Herring <[email protected]>
---
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

2023-07-18 16:45:25

by Rob Herring

Subject: Re: [PATCH] ARM: dts: hisilicon: Fix "status" values

On Mon, Jun 26, 2023 at 4:10 PM Rob Herring <[email protected]> wrote:
>
> The defined value for "status" is "disabled", not "disable".
>
> Signed-off-by: Rob Herring <[email protected]>
> ---
> 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
>

2023-07-19 07:52:07

by Wei Xu

Subject: Re: [PATCH] ARM: dts: hisilicon: Fix "status" values

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 <[email protected]>

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 {
>