All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gregory CLEMENT <gregory.clement@bootlin.com>
To: Adrian Schmutzler <freifunk@adrianschmutzler.de>,
	Jason Cooper <jason@lakedaemon.net>, Andrew Lunn <andrew@lunn.ch>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
	Rob Herring <robh+dt@kernel.org>,
	linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org
Subject: Re: [PATCH] ARM: dts: kirkwood: replace status value "ok" by "okay"
Date: Wed, 23 Sep 2020 16:38:43 +0200	[thread overview]
Message-ID: <87d02cimmk.fsf@BL-laptop> (raw)
In-Reply-To: <20200830193543.50530-1-freifunk@adrianschmutzler.de>

Hello Adrian,

> 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.
>

Applied on mvebu/dt

Thanks,

Gregory

> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> ---
>  arch/arm/boot/dts/kirkwood-dockstar.dts             | 2 +-
>  arch/arm/boot/dts/kirkwood-dreamplug.dts            | 2 +-
>  arch/arm/boot/dts/kirkwood-goflexnet.dts            | 2 +-
>  arch/arm/boot/dts/kirkwood-guruplug-server-plus.dts | 2 +-
>  arch/arm/boot/dts/kirkwood-iconnect.dts             | 2 +-
>  arch/arm/boot/dts/kirkwood-iomega_ix2_200.dts       | 2 +-
>  arch/arm/boot/dts/kirkwood-nsa3x0-common.dtsi       | 2 +-
>  arch/arm/boot/dts/kirkwood.dtsi                     | 4 ++--
>  8 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/arch/arm/boot/dts/kirkwood-dockstar.dts b/arch/arm/boot/dts/kirkwood-dockstar.dts
> index 6a3f1bf6d9f1..264938dfa4d9 100644
> --- a/arch/arm/boot/dts/kirkwood-dockstar.dts
> +++ b/arch/arm/boot/dts/kirkwood-dockstar.dts
> @@ -34,7 +34,7 @@
>  			};
>  		};
>  		serial@12000 {
> -			status = "ok";
> +			status = "okay";
>  		};
>  	};
>  	gpio-leds {
> diff --git a/arch/arm/boot/dts/kirkwood-dreamplug.dts b/arch/arm/boot/dts/kirkwood-dreamplug.dts
> index 7f326e267494..328516351e84 100644
> --- a/arch/arm/boot/dts/kirkwood-dreamplug.dts
> +++ b/arch/arm/boot/dts/kirkwood-dreamplug.dts
> @@ -34,7 +34,7 @@
>  			};
>  		};
>  		serial@12000 {
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		spi@10600 {
> diff --git a/arch/arm/boot/dts/kirkwood-goflexnet.dts b/arch/arm/boot/dts/kirkwood-goflexnet.dts
> index 02d87e0a1061..d4cb3cd3e2a2 100644
> --- a/arch/arm/boot/dts/kirkwood-goflexnet.dts
> +++ b/arch/arm/boot/dts/kirkwood-goflexnet.dts
> @@ -66,7 +66,7 @@
>  			};
>  		};
>  		serial@12000 {
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		sata@80000 {
> diff --git a/arch/arm/boot/dts/kirkwood-guruplug-server-plus.dts b/arch/arm/boot/dts/kirkwood-guruplug-server-plus.dts
> index ff1260ee3fe8..dfb41393941d 100644
> --- a/arch/arm/boot/dts/kirkwood-guruplug-server-plus.dts
> +++ b/arch/arm/boot/dts/kirkwood-guruplug-server-plus.dts
> @@ -38,7 +38,7 @@
>  			};
>  		};
>  		serial@12000 {
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		sata@80000 {
> diff --git a/arch/arm/boot/dts/kirkwood-iconnect.dts b/arch/arm/boot/dts/kirkwood-iconnect.dts
> index 4a512d80912c..95af7aa1fdcb 100644
> --- a/arch/arm/boot/dts/kirkwood-iconnect.dts
> +++ b/arch/arm/boot/dts/kirkwood-iconnect.dts
> @@ -72,7 +72,7 @@
>  			};
>  		};
>  		serial@12000 {
> -			status = "ok";
> +			status = "okay";
>  		};
>  	};
>  
> diff --git a/arch/arm/boot/dts/kirkwood-iomega_ix2_200.dts b/arch/arm/boot/dts/kirkwood-iomega_ix2_200.dts
> index 62272d58664f..2338f495d517 100644
> --- a/arch/arm/boot/dts/kirkwood-iomega_ix2_200.dts
> +++ b/arch/arm/boot/dts/kirkwood-iomega_ix2_200.dts
> @@ -112,7 +112,7 @@
>  		};
>  
>  		serial@12000 {
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		sata@80000 {
> diff --git a/arch/arm/boot/dts/kirkwood-nsa3x0-common.dtsi b/arch/arm/boot/dts/kirkwood-nsa3x0-common.dtsi
> index 2c4037b07282..8f73197f251a 100644
> --- a/arch/arm/boot/dts/kirkwood-nsa3x0-common.dtsi
> +++ b/arch/arm/boot/dts/kirkwood-nsa3x0-common.dtsi
> @@ -45,7 +45,7 @@
>  		};
>  
>  		serial@12000 {
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		sata@80000 {
> diff --git a/arch/arm/boot/dts/kirkwood.dtsi b/arch/arm/boot/dts/kirkwood.dtsi
> index 6c8d94beae78..fca31a5d5ac7 100644
> --- a/arch/arm/boot/dts/kirkwood.dtsi
> +++ b/arch/arm/boot/dts/kirkwood.dtsi
> @@ -369,7 +369,7 @@
>  			clocks = <&gate_clk 14>;
>  			clock-names = "sata";
>  			#phy-cells = <0>;
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		sata_phy1: sata-phy@84000 {
> @@ -378,7 +378,7 @@
>  			clocks = <&gate_clk 15>;
>  			clock-names = "sata";
>  			#phy-cells = <0>;
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		audio0: audio-controller@a0000 {
> -- 
> 2.20.1
>

-- 
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: Gregory CLEMENT <gregory.clement@bootlin.com>
To: Adrian Schmutzler <freifunk@adrianschmutzler.de>,
	Jason Cooper <jason@lakedaemon.net>, Andrew Lunn <andrew@lunn.ch>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
	Rob Herring <robh+dt@kernel.org>,
	linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org
Subject: Re: [PATCH] ARM: dts: kirkwood: replace status value "ok" by "okay"
Date: Wed, 23 Sep 2020 16:38:43 +0200	[thread overview]
Message-ID: <87d02cimmk.fsf@BL-laptop> (raw)
In-Reply-To: <20200830193543.50530-1-freifunk@adrianschmutzler.de>

Hello Adrian,

> 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.
>

Applied on mvebu/dt

Thanks,

Gregory

> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> ---
>  arch/arm/boot/dts/kirkwood-dockstar.dts             | 2 +-
>  arch/arm/boot/dts/kirkwood-dreamplug.dts            | 2 +-
>  arch/arm/boot/dts/kirkwood-goflexnet.dts            | 2 +-
>  arch/arm/boot/dts/kirkwood-guruplug-server-plus.dts | 2 +-
>  arch/arm/boot/dts/kirkwood-iconnect.dts             | 2 +-
>  arch/arm/boot/dts/kirkwood-iomega_ix2_200.dts       | 2 +-
>  arch/arm/boot/dts/kirkwood-nsa3x0-common.dtsi       | 2 +-
>  arch/arm/boot/dts/kirkwood.dtsi                     | 4 ++--
>  8 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/arch/arm/boot/dts/kirkwood-dockstar.dts b/arch/arm/boot/dts/kirkwood-dockstar.dts
> index 6a3f1bf6d9f1..264938dfa4d9 100644
> --- a/arch/arm/boot/dts/kirkwood-dockstar.dts
> +++ b/arch/arm/boot/dts/kirkwood-dockstar.dts
> @@ -34,7 +34,7 @@
>  			};
>  		};
>  		serial@12000 {
> -			status = "ok";
> +			status = "okay";
>  		};
>  	};
>  	gpio-leds {
> diff --git a/arch/arm/boot/dts/kirkwood-dreamplug.dts b/arch/arm/boot/dts/kirkwood-dreamplug.dts
> index 7f326e267494..328516351e84 100644
> --- a/arch/arm/boot/dts/kirkwood-dreamplug.dts
> +++ b/arch/arm/boot/dts/kirkwood-dreamplug.dts
> @@ -34,7 +34,7 @@
>  			};
>  		};
>  		serial@12000 {
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		spi@10600 {
> diff --git a/arch/arm/boot/dts/kirkwood-goflexnet.dts b/arch/arm/boot/dts/kirkwood-goflexnet.dts
> index 02d87e0a1061..d4cb3cd3e2a2 100644
> --- a/arch/arm/boot/dts/kirkwood-goflexnet.dts
> +++ b/arch/arm/boot/dts/kirkwood-goflexnet.dts
> @@ -66,7 +66,7 @@
>  			};
>  		};
>  		serial@12000 {
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		sata@80000 {
> diff --git a/arch/arm/boot/dts/kirkwood-guruplug-server-plus.dts b/arch/arm/boot/dts/kirkwood-guruplug-server-plus.dts
> index ff1260ee3fe8..dfb41393941d 100644
> --- a/arch/arm/boot/dts/kirkwood-guruplug-server-plus.dts
> +++ b/arch/arm/boot/dts/kirkwood-guruplug-server-plus.dts
> @@ -38,7 +38,7 @@
>  			};
>  		};
>  		serial@12000 {
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		sata@80000 {
> diff --git a/arch/arm/boot/dts/kirkwood-iconnect.dts b/arch/arm/boot/dts/kirkwood-iconnect.dts
> index 4a512d80912c..95af7aa1fdcb 100644
> --- a/arch/arm/boot/dts/kirkwood-iconnect.dts
> +++ b/arch/arm/boot/dts/kirkwood-iconnect.dts
> @@ -72,7 +72,7 @@
>  			};
>  		};
>  		serial@12000 {
> -			status = "ok";
> +			status = "okay";
>  		};
>  	};
>  
> diff --git a/arch/arm/boot/dts/kirkwood-iomega_ix2_200.dts b/arch/arm/boot/dts/kirkwood-iomega_ix2_200.dts
> index 62272d58664f..2338f495d517 100644
> --- a/arch/arm/boot/dts/kirkwood-iomega_ix2_200.dts
> +++ b/arch/arm/boot/dts/kirkwood-iomega_ix2_200.dts
> @@ -112,7 +112,7 @@
>  		};
>  
>  		serial@12000 {
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		sata@80000 {
> diff --git a/arch/arm/boot/dts/kirkwood-nsa3x0-common.dtsi b/arch/arm/boot/dts/kirkwood-nsa3x0-common.dtsi
> index 2c4037b07282..8f73197f251a 100644
> --- a/arch/arm/boot/dts/kirkwood-nsa3x0-common.dtsi
> +++ b/arch/arm/boot/dts/kirkwood-nsa3x0-common.dtsi
> @@ -45,7 +45,7 @@
>  		};
>  
>  		serial@12000 {
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		sata@80000 {
> diff --git a/arch/arm/boot/dts/kirkwood.dtsi b/arch/arm/boot/dts/kirkwood.dtsi
> index 6c8d94beae78..fca31a5d5ac7 100644
> --- a/arch/arm/boot/dts/kirkwood.dtsi
> +++ b/arch/arm/boot/dts/kirkwood.dtsi
> @@ -369,7 +369,7 @@
>  			clocks = <&gate_clk 14>;
>  			clock-names = "sata";
>  			#phy-cells = <0>;
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		sata_phy1: sata-phy@84000 {
> @@ -378,7 +378,7 @@
>  			clocks = <&gate_clk 15>;
>  			clock-names = "sata";
>  			#phy-cells = <0>;
> -			status = "ok";
> +			status = "okay";
>  		};
>  
>  		audio0: audio-controller@a0000 {
> -- 
> 2.20.1
>

-- 
Gregory Clement, Bootlin
Embedded Linux and Kernel engineering
http://bootlin.com

  parent reply	other threads:[~2020-09-23 14:40 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-30 19:35 [PATCH] ARM: dts: kirkwood: replace status value "ok" by "okay" Adrian Schmutzler
2020-08-30 19:35 ` Adrian Schmutzler
2020-08-31 13:48 ` Andrew Lunn
2020-08-31 13:48   ` Andrew Lunn
2020-09-23 14:38 ` Gregory CLEMENT [this message]
2020-09-23 14:38   ` Gregory CLEMENT

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=87d02cimmk.fsf@BL-laptop \
    --to=gregory.clement@bootlin.com \
    --cc=andrew@lunn.ch \
    --cc=devicetree@vger.kernel.org \
    --cc=freifunk@adrianschmutzler.de \
    --cc=jason@lakedaemon.net \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=robh+dt@kernel.org \
    --cc=sebastian.hesselbarth@gmail.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.