linux-kernel - Re: [PATCH] ARM: dts: hisilicon: Fix "status" values

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAL_Jsq+x8qZuov0c7XFzvCxb6TqEudCrLDsOM-bkS9Gu7B4R4Q@mail.gmail.com>
Date:   Tue, 18 Jul 2023 10:22:50 -0600
From:   Rob Herring <robh@...nel.org>
To:     Wei Xu <xuwei5@...ilicon.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Conor Dooley <conor+dt@...nel.org>
Cc:     linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] ARM: dts: hisilicon: Fix "status" values

On Mon, Jun 26, 2023 at 4:10 PM Rob Herring <robh@...nel.org> wrote:
>
> The defined value for "status" is "disabled", not "disable".
>
> Signed-off-by: Rob Herring <robh@...nel.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@...00000 {
>                         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@...01000 {
> @@ -63,7 +63,7 @@ uart1: serial@...01000 {
>                         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@...02000 {
> @@ -72,7 +72,7 @@ uart2: serial@...02000 {
>                         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@...03000 {
> @@ -81,7 +81,7 @@ uart3: serial@...03000 {
>                         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@...04000 {
> @@ -90,7 +90,7 @@ uart4: serial@...04000 {
>                         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@...00000 {
> @@ -100,7 +100,7 @@ dual_timer0: timer@...00000 {
>                         reg = <0x12000000 0x1000>;
>                         clocks = <&clk_3m>;
>                         clock-names = "apb_pclk";
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 dual_timer1: timer@...01000 {
> @@ -110,7 +110,7 @@ dual_timer1: timer@...01000 {
>                         reg = <0x12001000 0x1000>;
>                         clocks = <&clk_3m>;
>                         clock-names = "apb_pclk";
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 dual_timer2: timer@...02000 {
> @@ -120,7 +120,7 @@ dual_timer2: timer@...02000 {
>                         reg = <0x12002000 0x1000>;
>                         clocks = <&clk_3m>;
>                         clock-names = "apb_pclk";
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 spi_bus0: spi@...20000 {
> @@ -132,7 +132,7 @@ spi_bus0: spi@...20000 {
>                         num-cs = <1>;
>                         #address-cells = <1>;
>                         #size-cells = <0>;
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 spi_bus1: spi@...21000 {
> @@ -144,7 +144,7 @@ spi_bus1: spi@...21000 {
>                         num-cs = <1>;
>                         #address-cells = <1>;
>                         #size-cells = <0>;
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 spi_bus2: spi@...22000 {
> @@ -156,7 +156,7 @@ spi_bus2: spi@...22000 {
>                         num-cs = <1>;
>                         #address-cells = <1>;
>                         #size-cells = <0>;
> -                       status = "disable";
> +                       status = "disabled";
>                 };
>
>                 sysctrl: system-controller@...20000 {
> --
> 2.40.1
>