| Message ID | 20230127101824.93684-1-krzysztof.kozlowski@linaro.org (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: dts: microchip: use "okay" for status | expand |
Hi Krzysztof, This looks "okay" to me. BR Steen Reviewed-by: Steen Hegelund <Steen.Hegelund@microchip.com> 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 <krzysztof.kozlowski@linaro.org> > --- > 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 >
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,
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>; };
"okay" over "ok" is preferred for status property. Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- 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(-)