* [PATCH] arm64: dts: hisilicon: replace status value "ok" by "okay"
@ 2020-08-30 20:05 Adrian Schmutzler
2020-09-16 1:46 ` Wei Xu
0 siblings, 1 reply; 2+ messages in thread
From: Adrian Schmutzler @ 2020-08-30 20:05 UTC (permalink / raw)
To: Wei Xu, Rob Herring, linux-arm-kernel, devicetree
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>
---
.../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";
};
--
2.20.1
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH] arm64: dts: hisilicon: replace status value "ok" by "okay"
2020-08-30 20:05 [PATCH] arm64: dts: hisilicon: replace status value "ok" by "okay" Adrian Schmutzler
@ 2020-09-16 1:46 ` Wei Xu
0 siblings, 0 replies; 2+ messages in thread
From: Wei Xu @ 2020-09-16 1:46 UTC (permalink / raw)
To: Adrian Schmutzler, Rob Herring, linux-arm-kernel, devicetree
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";
> };
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-09-16 1:46 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
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 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).