2023-01-27 09:53:30

by Krzysztof Kozlowski

Subject: [PATCH] ARM: dts: qcom: use "okay" for status

"okay" over "ok" is preferred:

serial@f991f000: status:0: 'ok' is not one of ['okay', 'disabled', 'reserved']

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
arch/arm/boot/dts/qcom-msm8226-samsung-s3ve3g.dts | 2 +-
arch/arm/boot/dts/qcom-sdx55-t55.dts | 6 +++---
arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts | 6 +++---
3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/arch/arm/boot/dts/qcom-msm8226-samsung-s3ve3g.dts b/arch/arm/boot/dts/qcom-msm8226-samsung-s3ve3g.dts
index 6a082ad4418a..288cacd5d1fa 100644
--- a/arch/arm/boot/dts/qcom-msm8226-samsung-s3ve3g.dts
+++ b/arch/arm/boot/dts/qcom-msm8226-samsung-s3ve3g.dts
@@ -20,5 +20,5 @@ chosen {
};

&blsp1_uart3 {
- status = "ok";
+ status = "okay";
};
diff --git a/arch/arm/boot/dts/qcom-sdx55-t55.dts b/arch/arm/boot/dts/qcom-sdx55-t55.dts
index 61ac5f54cd57..ccf2b8b42b16 100644
--- a/arch/arm/boot/dts/qcom-sdx55-t55.dts
+++ b/arch/arm/boot/dts/qcom-sdx55-t55.dts
@@ -233,7 +233,7 @@ ldo16 {
};

&blsp1_uart3 {
- status = "ok";
+ status = "okay";
};

&ipa {
@@ -243,11 +243,11 @@ &ipa {
};

&qpic_bam {
- status = "ok";
+ status = "okay";
};

&qpic_nand {
- status = "ok";
+ status = "okay";

nand@0 {
reg = <0>;
diff --git a/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts b/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts
index c9c1f7da1261..db7b3d5b0bd0 100644
--- a/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts
+++ b/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts
@@ -233,7 +233,7 @@ ldo16 {
};

&blsp1_uart3 {
- status = "ok";
+ status = "okay";
};

&ipa {
@@ -258,11 +258,11 @@ &pcie_ep {
};

&qpic_bam {
- status = "ok";
+ status = "okay";
};

&qpic_nand {
- status = "ok";
+ status = "okay";

nand@0 {
reg = <0>;
--
2.34.1

2023-01-27 10:04:34

by Konrad Dybcio

Subject: Re: [PATCH] ARM: dts: qcom: use "okay" for status

On 27.01.2023 10:53, Krzysztof Kozlowski wrote:
> "okay" over "ok" is preferred:
>
> serial@f991f000: status:0: 'ok' is not one of ['okay', 'disabled', 'reserved']
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
Reviewed-by: Konrad Dybcio <[email protected]>

Konrad
> arch/arm/boot/dts/qcom-msm8226-samsung-s3ve3g.dts | 2 +-
> arch/arm/boot/dts/qcom-sdx55-t55.dts | 6 +++---
> arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts | 6 +++---
> 3 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm/boot/dts/qcom-msm8226-samsung-s3ve3g.dts b/arch/arm/boot/dts/qcom-msm8226-samsung-s3ve3g.dts
> index 6a082ad4418a..288cacd5d1fa 100644
> --- a/arch/arm/boot/dts/qcom-msm8226-samsung-s3ve3g.dts
> +++ b/arch/arm/boot/dts/qcom-msm8226-samsung-s3ve3g.dts
> @@ -20,5 +20,5 @@ chosen {
> };
>
> &blsp1_uart3 {
> - status = "ok";
> + status = "okay";
> };
> diff --git a/arch/arm/boot/dts/qcom-sdx55-t55.dts b/arch/arm/boot/dts/qcom-sdx55-t55.dts
> index 61ac5f54cd57..ccf2b8b42b16 100644
> --- a/arch/arm/boot/dts/qcom-sdx55-t55.dts
> +++ b/arch/arm/boot/dts/qcom-sdx55-t55.dts
> @@ -233,7 +233,7 @@ ldo16 {
> };
>
> &blsp1_uart3 {
> - status = "ok";
> + status = "okay";
> };
>
> &ipa {
> @@ -243,11 +243,11 @@ &ipa {
> };
>
> &qpic_bam {
> - status = "ok";
> + status = "okay";
> };
>
> &qpic_nand {
> - status = "ok";
> + status = "okay";
>
> nand@0 {
> reg = <0>;
> diff --git a/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts b/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts
> index c9c1f7da1261..db7b3d5b0bd0 100644
> --- a/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts
> +++ b/arch/arm/boot/dts/qcom-sdx55-telit-fn980-tlb.dts
> @@ -233,7 +233,7 @@ ldo16 {
> };
>
> &blsp1_uart3 {
> - status = "ok";
> + status = "okay";
> };
>
> &ipa {
> @@ -258,11 +258,11 @@ &pcie_ep {
> };
>
> &qpic_bam {
> - status = "ok";
> + status = "okay";
> };
>
> &qpic_nand {
> - status = "ok";
> + status = "okay";
>
> nand@0 {
> reg = <0>;

2023-02-09 04:32:03

by Bjorn Andersson

Subject: Re: [PATCH] ARM: dts: qcom: use "okay" for status

On Fri, 27 Jan 2023 10:53:19 +0100, Krzysztof Kozlowski wrote:
> "okay" over "ok" is preferred:
>
> serial@f991f000: status:0: 'ok' is not one of ['okay', 'disabled', 'reserved']
>
>

Applied, thanks!

[1/1] ARM: dts: qcom: use "okay" for status
commit: 8676ff664b0ec6de46adf1abc59b7903c6b6049c

Best regards,
--
Bjorn Andersson <[email protected]>