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