All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wei Xu <xuwei5@hisilicon.com>
To: Adrian Schmutzler <freifunk@adrianschmutzler.de>,
	Rob Herring <robh+dt@kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<devicetree@vger.kernel.org>
Subject: Re: [PATCH] arm64: dts: hisilicon: replace status value "ok" by "okay"
Date: Wed, 16 Sep 2020 09:46:27 +0800	[thread overview]
Message-ID: <5F616E73.4080707@hisilicon.com> (raw)
In-Reply-To: <20200830200551.1474-1-freifunk@adrianschmutzler.de>

Hi Adrian,

On 2020/8/31 4:05, Adrian Schmutzler wrote:
> While the DT parser recognizes "ok" as a valid value for the
> "status" property, it is actually mentioned nowhere. Use the
> proper value "okay" instead, as done in the majority of files
> already.
> 
> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>

Thanks!
Applied to the hisilicon arm64 dt tree.

Best Regards,
Wei

> ---
>  .../boot/dts/hisilicon/hi3660-hikey960.dts    |  6 +++---
>  .../boot/dts/hisilicon/hi3670-hikey970.dts    |  2 +-
>  .../arm64/boot/dts/hisilicon/hi6220-hikey.dts | 18 ++++++++---------
>  arch/arm64/boot/dts/hisilicon/hi6220.dtsi     |  2 +-
>  arch/arm64/boot/dts/hisilicon/hip05-d02.dts   |  6 +++---
>  arch/arm64/boot/dts/hisilicon/hip06-d03.dts   | 18 ++++++++---------
>  arch/arm64/boot/dts/hisilicon/hip07-d05.dts   | 20 +++++++++----------
>  7 files changed, 36 insertions(+), 36 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts b/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts
> index c1b614dabb8e..963300eede17 100644
> --- a/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts
> +++ b/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts
> @@ -530,7 +530,7 @@
>  	rt1711h: rt1711h@4e {
>  		compatible = "richtek,rt1711h";
>  		reg = <0x4e>;
> -		status = "ok";
> +		status = "okay";
>  		interrupt-parent = <&gpio27>;
>  		interrupts = <3 IRQ_TYPE_LEVEL_LOW>;
>  		pinctrl-names = "default";
> @@ -570,7 +570,7 @@
>  	};
>  
>  	adv7533: adv7533@39 {
> -		status = "ok";
> +		status = "okay";
>  		compatible = "adi,adv7533";
>  		reg = <0x39>;
>  		adi,dsi-lanes = <4>;
> @@ -656,7 +656,7 @@
>  		     &sdio_cfg_func>;
>  	/* WL_EN */
>  	vmmc-supply = <&wlan_en>;
> -	status = "ok";
> +	status = "okay";
>  
>  	wlcore: wlcore@2 {
>  		compatible = "ti,wl1837";
> diff --git a/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts b/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts
> index 7dac33d4fd5c..7f9f9886c349 100644
> --- a/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts
> +++ b/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts
> @@ -418,7 +418,7 @@
>  		     &sdio_cfg_func>;
>  	/* WL_EN */
>  	vmmc-supply = <&wlan_en>;
> -	status = "ok";
> +	status = "okay";
>  
>  	wlcore: wlcore@2 {
>  		compatible = "ti,wl1837";
> diff --git a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
> index 533ed523888d..91d08673c02e 100644
> --- a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
> +++ b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
> @@ -267,7 +267,7 @@
>  &uart1 {
>  	assigned-clocks = <&sys_ctrl HI6220_UART1_SRC>;
>  	assigned-clock-rates = <150000000>;
> -	status = "ok";
> +	status = "okay";
>  
>  	bluetooth {
>  		compatible = "ti,wl1835-st";
> @@ -278,21 +278,21 @@
>  };
>  
>  &uart2 {
> -	status = "ok";
> +	status = "okay";
>  	label = "LS-UART0";
>  };
>  
>  &uart3 {
> -	status = "ok";
> +	status = "okay";
>  	label = "LS-UART1";
>  };
>  
>  &ade {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &dsi {
> -	status = "ok";
> +	status = "okay";
>  
>  	ports {
>  		/* 1 for output port */
> @@ -489,17 +489,17 @@
>  
>  
>  &i2c0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &i2c1 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &i2c2 {
>  	#address-cells = <1>;
>  	#size-cells = <0>;
> -	status = "ok";
> +	status = "okay";
>  
>  	adv7533: adv7533@39 {
>  		compatible = "adi,adv7533";
> @@ -541,5 +541,5 @@
>  };
>  
>  &spi0 {
> -	status = "ok";
> +	status = "okay";
>  };
> diff --git a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi
> index 3d189d9f0d24..3bab4bc6ff99 100644
> --- a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi
> +++ b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi
> @@ -371,7 +371,7 @@
>  			clocks = <&sys_ctrl HI6220_EDMAC_ACLK>;
>  			dma-no-cci;
>  			dma-type = "hi6220_dma";
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		dual_timer0: timer@f8008000 {
> diff --git a/arch/arm64/boot/dts/hisilicon/hip05-d02.dts b/arch/arm64/boot/dts/hisilicon/hip05-d02.dts
> index e93c65ede06c..369b69b17b91 100644
> --- a/arch/arm64/boot/dts/hisilicon/hip05-d02.dts
> +++ b/arch/arm64/boot/dts/hisilicon/hip05-d02.dts
> @@ -42,15 +42,15 @@
>  };
>  
>  &uart0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &peri_gpio0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &lbc {
> -	status = "ok";
> +	status = "okay";
>  	#address-cells = <2>;
>  	#size-cells = <1>;
>  	ranges = <0 0 0x0 0x90000000 0x08000000>,
> diff --git a/arch/arm64/boot/dts/hisilicon/hip06-d03.dts b/arch/arm64/boot/dts/hisilicon/hip06-d03.dts
> index 677862beebef..9f4a930e734d 100644
> --- a/arch/arm64/boot/dts/hisilicon/hip06-d03.dts
> +++ b/arch/arm64/boot/dts/hisilicon/hip06-d03.dts
> @@ -22,37 +22,37 @@
>  };
>  
>  &ipmi0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &uart0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth1 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth2 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth3 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &sas1 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &usb_ohci {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &usb_ehci {
> -	status = "ok";
> +	status = "okay";
>  };
> diff --git a/arch/arm64/boot/dts/hisilicon/hip07-d05.dts b/arch/arm64/boot/dts/hisilicon/hip07-d05.dts
> index fcbdffe0868b..81a2312c8a26 100644
> --- a/arch/arm64/boot/dts/hisilicon/hip07-d05.dts
> +++ b/arch/arm64/boot/dts/hisilicon/hip07-d05.dts
> @@ -50,41 +50,41 @@
>  };
>  
>  &uart0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &ipmi0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &usb_ohci {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &usb_ehci {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth1 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth2 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth3 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &sas1 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &p0_pcie2_a {
> -	status = "ok";
> +	status = "okay";
>  };
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: Wei Xu <xuwei5@hisilicon.com>
To: Adrian Schmutzler <freifunk@adrianschmutzler.de>,
	Rob Herring <robh+dt@kernel.org>,
	<linux-arm-kernel@lists.infradead.org>,
	<devicetree@vger.kernel.org>
Subject: Re: [PATCH] arm64: dts: hisilicon: replace status value "ok" by "okay"
Date: Wed, 16 Sep 2020 09:46:27 +0800	[thread overview]
Message-ID: <5F616E73.4080707@hisilicon.com> (raw)
In-Reply-To: <20200830200551.1474-1-freifunk@adrianschmutzler.de>

Hi Adrian,

On 2020/8/31 4:05, Adrian Schmutzler wrote:
> While the DT parser recognizes "ok" as a valid value for the
> "status" property, it is actually mentioned nowhere. Use the
> proper value "okay" instead, as done in the majority of files
> already.
> 
> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>

Thanks!
Applied to the hisilicon arm64 dt tree.

Best Regards,
Wei

> ---
>  .../boot/dts/hisilicon/hi3660-hikey960.dts    |  6 +++---
>  .../boot/dts/hisilicon/hi3670-hikey970.dts    |  2 +-
>  .../arm64/boot/dts/hisilicon/hi6220-hikey.dts | 18 ++++++++---------
>  arch/arm64/boot/dts/hisilicon/hi6220.dtsi     |  2 +-
>  arch/arm64/boot/dts/hisilicon/hip05-d02.dts   |  6 +++---
>  arch/arm64/boot/dts/hisilicon/hip06-d03.dts   | 18 ++++++++---------
>  arch/arm64/boot/dts/hisilicon/hip07-d05.dts   | 20 +++++++++----------
>  7 files changed, 36 insertions(+), 36 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts b/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts
> index c1b614dabb8e..963300eede17 100644
> --- a/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts
> +++ b/arch/arm64/boot/dts/hisilicon/hi3660-hikey960.dts
> @@ -530,7 +530,7 @@
>  	rt1711h: rt1711h@4e {
>  		compatible = "richtek,rt1711h";
>  		reg = <0x4e>;
> -		status = "ok";
> +		status = "okay";
>  		interrupt-parent = <&gpio27>;
>  		interrupts = <3 IRQ_TYPE_LEVEL_LOW>;
>  		pinctrl-names = "default";
> @@ -570,7 +570,7 @@
>  	};
>  
>  	adv7533: adv7533@39 {
> -		status = "ok";
> +		status = "okay";
>  		compatible = "adi,adv7533";
>  		reg = <0x39>;
>  		adi,dsi-lanes = <4>;
> @@ -656,7 +656,7 @@
>  		     &sdio_cfg_func>;
>  	/* WL_EN */
>  	vmmc-supply = <&wlan_en>;
> -	status = "ok";
> +	status = "okay";
>  
>  	wlcore: wlcore@2 {
>  		compatible = "ti,wl1837";
> diff --git a/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts b/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts
> index 7dac33d4fd5c..7f9f9886c349 100644
> --- a/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts
> +++ b/arch/arm64/boot/dts/hisilicon/hi3670-hikey970.dts
> @@ -418,7 +418,7 @@
>  		     &sdio_cfg_func>;
>  	/* WL_EN */
>  	vmmc-supply = <&wlan_en>;
> -	status = "ok";
> +	status = "okay";
>  
>  	wlcore: wlcore@2 {
>  		compatible = "ti,wl1837";
> diff --git a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
> index 533ed523888d..91d08673c02e 100644
> --- a/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
> +++ b/arch/arm64/boot/dts/hisilicon/hi6220-hikey.dts
> @@ -267,7 +267,7 @@
>  &uart1 {
>  	assigned-clocks = <&sys_ctrl HI6220_UART1_SRC>;
>  	assigned-clock-rates = <150000000>;
> -	status = "ok";
> +	status = "okay";
>  
>  	bluetooth {
>  		compatible = "ti,wl1835-st";
> @@ -278,21 +278,21 @@
>  };
>  
>  &uart2 {
> -	status = "ok";
> +	status = "okay";
>  	label = "LS-UART0";
>  };
>  
>  &uart3 {
> -	status = "ok";
> +	status = "okay";
>  	label = "LS-UART1";
>  };
>  
>  &ade {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &dsi {
> -	status = "ok";
> +	status = "okay";
>  
>  	ports {
>  		/* 1 for output port */
> @@ -489,17 +489,17 @@
>  
>  
>  &i2c0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &i2c1 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &i2c2 {
>  	#address-cells = <1>;
>  	#size-cells = <0>;
> -	status = "ok";
> +	status = "okay";
>  
>  	adv7533: adv7533@39 {
>  		compatible = "adi,adv7533";
> @@ -541,5 +541,5 @@
>  };
>  
>  &spi0 {
> -	status = "ok";
> +	status = "okay";
>  };
> diff --git a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi
> index 3d189d9f0d24..3bab4bc6ff99 100644
> --- a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi
> +++ b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi
> @@ -371,7 +371,7 @@
>  			clocks = <&sys_ctrl HI6220_EDMAC_ACLK>;
>  			dma-no-cci;
>  			dma-type = "hi6220_dma";
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		dual_timer0: timer@f8008000 {
> diff --git a/arch/arm64/boot/dts/hisilicon/hip05-d02.dts b/arch/arm64/boot/dts/hisilicon/hip05-d02.dts
> index e93c65ede06c..369b69b17b91 100644
> --- a/arch/arm64/boot/dts/hisilicon/hip05-d02.dts
> +++ b/arch/arm64/boot/dts/hisilicon/hip05-d02.dts
> @@ -42,15 +42,15 @@
>  };
>  
>  &uart0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &peri_gpio0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &lbc {
> -	status = "ok";
> +	status = "okay";
>  	#address-cells = <2>;
>  	#size-cells = <1>;
>  	ranges = <0 0 0x0 0x90000000 0x08000000>,
> diff --git a/arch/arm64/boot/dts/hisilicon/hip06-d03.dts b/arch/arm64/boot/dts/hisilicon/hip06-d03.dts
> index 677862beebef..9f4a930e734d 100644
> --- a/arch/arm64/boot/dts/hisilicon/hip06-d03.dts
> +++ b/arch/arm64/boot/dts/hisilicon/hip06-d03.dts
> @@ -22,37 +22,37 @@
>  };
>  
>  &ipmi0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &uart0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth1 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth2 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth3 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &sas1 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &usb_ohci {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &usb_ehci {
> -	status = "ok";
> +	status = "okay";
>  };
> diff --git a/arch/arm64/boot/dts/hisilicon/hip07-d05.dts b/arch/arm64/boot/dts/hisilicon/hip07-d05.dts
> index fcbdffe0868b..81a2312c8a26 100644
> --- a/arch/arm64/boot/dts/hisilicon/hip07-d05.dts
> +++ b/arch/arm64/boot/dts/hisilicon/hip07-d05.dts
> @@ -50,41 +50,41 @@
>  };
>  
>  &uart0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &ipmi0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &usb_ohci {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &usb_ehci {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth0 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth1 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth2 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &eth3 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &sas1 {
> -	status = "ok";
> +	status = "okay";
>  };
>  
>  &p0_pcie2_a {
> -	status = "ok";
> +	status = "okay";
>  };
> 

  reply	other threads:[~2020-09-16  1:48 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-30 20:05 [PATCH] arm64: dts: hisilicon: replace status value "ok" by "okay" Adrian Schmutzler
2020-08-30 20:05 ` Adrian Schmutzler
2020-09-16  1:46 ` Wei Xu [this message]
2020-09-16  1:46   ` Wei Xu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5F616E73.4080707@hisilicon.com \
    --to=xuwei5@hisilicon.com \
    --cc=devicetree@vger.kernel.org \
    --cc=freifunk@adrianschmutzler.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=robh+dt@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.