2023-01-27 10:18:33

by Krzysztof Kozlowski

Subject: [PATCH] arm64: dts: microchip: use "okay" for status

"okay" over "ok" is preferred for status property.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
arch/arm64/boot/dts/microchip/sparx5_pcb134_board.dtsi | 2 +-
arch/arm64/boot/dts/microchip/sparx5_pcb135_board.dtsi | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/microchip/sparx5_pcb134_board.dtsi b/arch/arm64/boot/dts/microchip/sparx5_pcb134_board.dtsi
index bf12be5e8d84..ed94a80bf05f 100644
--- a/arch/arm64/boot/dts/microchip/sparx5_pcb134_board.dtsi
+++ b/arch/arm64/boot/dts/microchip/sparx5_pcb134_board.dtsi
@@ -538,7 +538,7 @@ i2c_sfp20: i2c_sfp20 {
};

&mdio3 {
- status = "ok";
+ status = "okay";
phy64: ethernet-phy@64 {
reg = <28>;
};
diff --git a/arch/arm64/boot/dts/microchip/sparx5_pcb135_board.dtsi b/arch/arm64/boot/dts/microchip/sparx5_pcb135_board.dtsi
index ec90bda7ed6a..0760cf2e48bc 100644
--- a/arch/arm64/boot/dts/microchip/sparx5_pcb135_board.dtsi
+++ b/arch/arm64/boot/dts/microchip/sparx5_pcb135_board.dtsi
@@ -208,7 +208,7 @@ sfp_eth63: sfp-eth63 {
};

&mdio0 {
- status = "ok";
+ status = "okay";
phy0: ethernet-phy@0 {
reg = <0>;
};
@@ -284,7 +284,7 @@ phy23: ethernet-phy@23 {
};

&mdio1 {
- status = "ok";
+ status = "okay";
phy24: ethernet-phy@24 {
reg = <0>;
};
@@ -360,7 +360,7 @@ phy47: ethernet-phy@47 {
};

&mdio3 {
- status = "ok";
+ status = "okay";
phy64: ethernet-phy@64 {
reg = <28>;
};
--
2.34.1

2023-01-27 13:34:17

by Steen Hegelund

Subject: Re: [PATCH] arm64: dts: microchip: use "okay" for status

Hi Krzysztof,

This looks "okay" to me.

BR
Steen

Reviewed-by: Steen Hegelund <[email protected]>

On Fri, 2023-01-27 at 11:18 +0100, Krzysztof Kozlowski wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the
> content is safe
>
> "okay" over "ok" is preferred for status property.
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
>  arch/arm64/boot/dts/microchip/sparx5_pcb134_board.dtsi | 2 +-
>  arch/arm64/boot/dts/microchip/sparx5_pcb135_board.dtsi | 6 +++---
>  2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/microchip/sparx5_pcb134_board.dtsi
> b/arch/arm64/boot/dts/microchip/sparx5_pcb134_board.dtsi
> index bf12be5e8d84..ed94a80bf05f 100644
> --- a/arch/arm64/boot/dts/microchip/sparx5_pcb134_board.dtsi
> +++ b/arch/arm64/boot/dts/microchip/sparx5_pcb134_board.dtsi
> @@ -538,7 +538,7 @@ i2c_sfp20: i2c_sfp20 {
>  };
>
>  &mdio3 {
> -       status = "ok";
> +       status = "okay";
>         phy64: ethernet-phy@64 {
>                 reg = <28>;
>         };
> diff --git a/arch/arm64/boot/dts/microchip/sparx5_pcb135_board.dtsi
> b/arch/arm64/boot/dts/microchip/sparx5_pcb135_board.dtsi
> index ec90bda7ed6a..0760cf2e48bc 100644
> --- a/arch/arm64/boot/dts/microchip/sparx5_pcb135_board.dtsi
> +++ b/arch/arm64/boot/dts/microchip/sparx5_pcb135_board.dtsi
> @@ -208,7 +208,7 @@ sfp_eth63: sfp-eth63 {
>  };
>
>  &mdio0 {
> -       status = "ok";
> +       status = "okay";
>         phy0: ethernet-phy@0 {
>                 reg = <0>;
>         };
> @@ -284,7 +284,7 @@ phy23: ethernet-phy@23 {
>  };
>
>  &mdio1 {
> -       status = "ok";
> +       status = "okay";
>         phy24: ethernet-phy@24 {
>                 reg = <0>;
>         };
> @@ -360,7 +360,7 @@ phy47: ethernet-phy@47 {
>  };
>
>  &mdio3 {
> -       status = "ok";
> +       status = "okay";
>         phy64: ethernet-phy@64 {
>                 reg = <28>;
>         };
> --
> 2.34.1
>

2023-01-28 10:18:12

by Krzysztof Kozlowski

Subject: Re: [PATCH] arm64: dts: microchip: use "okay" for status

On Fri, 27 Jan 2023 11:18:24 +0100, Krzysztof Kozlowski wrote:
> "okay" over "ok" is preferred for status property.
>
>

Applied, thanks!

Let me know if anyone preferred to take it via sub-arch/SoC maintainer tree.
I'll drop it then.

[1/1] arm64: dts: microchip: use "okay" for status
https://git.kernel.org/krzk/linux-dt/c/d105729968404fd42a624bfea01b2f8e46a4947a

Best regards,
--
Krzysztof Kozlowski <[email protected]>