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";
> };
>
> ð0 {
> - status = "ok";
> + status = "okay";
> };
>
> ð1 {
> - status = "ok";
> + status = "okay";
> };
>
> ð2 {
> - status = "ok";
> + status = "okay";
> };
>
> ð3 {
> - 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";
> };
>
> ð0 {
> - status = "ok";
> + status = "okay";
> };
>
> ð1 {
> - status = "ok";
> + status = "okay";
> };
>
> ð2 {
> - status = "ok";
> + status = "okay";
> };
>
> ð3 {
> - status = "ok";
> + status = "okay";
> };
>
> &sas1 {
> - status = "ok";
> + status = "okay";
> };
>
> &p0_pcie2_a {
> - status = "ok";
> + status = "okay";
> };
>
prev parent reply other threads:[~2020-09-16 1:46 UTC|newest]
Thread overview: 2+ 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-09-16 1:46 ` Wei Xu [this message]
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 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 NNTP newsgroup(s).