Message ID | 1520588080-31264-1-git-send-email-geert+renesas@glider.be (mailing list archive) |
---|---|
State | Accepted, archived |
Delegated to: | Hans Verkuil |
Headers |
Received: from vger.kernel.org ([209.132.180.67]) by www.linuxtv.org with esmtp (Exim 4.84_2) (envelope-from <linux-media-owner@vger.kernel.org>) id 1euEQN-00075M-Ty; Fri, 09 Mar 2018 09:34:52 +0000 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751211AbeCIJet (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Fri, 9 Mar 2018 04:34:49 -0500 Received: from andre.telenet-ops.be ([195.130.132.53]:49128 "EHLO andre.telenet-ops.be" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751167AbeCIJer (ORCPT <rfc822; linux-media@vger.kernel.org>); Fri, 9 Mar 2018 04:34:47 -0500 Received: from ayla.of.borg ([84.194.111.163]) by andre.telenet-ops.be with bizsmtp id Klal1x00o3XaVaC01lal8V; Fri, 09 Mar 2018 10:34:46 +0100 Received: from ramsan.of.borg ([192.168.97.29] helo=ramsan) by ayla.of.borg with esmtp (Exim 4.86_2) (envelope-from <geert@linux-m68k.org>) id 1euEQH-0006m7-BC; Fri, 09 Mar 2018 10:34:45 +0100 Received: from geert by ramsan with local (Exim 4.86_2) (envelope-from <geert@linux-m68k.org>) id 1euEQH-000892-9e; Fri, 09 Mar 2018 10:34:45 +0100 From: Geert Uytterhoeven <geert+renesas@glider.be> To: =?UTF-8?q?Niklas=20S=C3=B6derlund?= <niklas.soderlund@ragnatech.se>, Mauro Carvalho Chehab <mchehab@kernel.org>, Rob Herring <robh+dt@kernel.org>, Mark Rutland <mark.rutland@arm.com> Cc: Andy Whitcroft <apw@canonical.com>, Joe Perches <joe@perches.com>, linux-media@vger.kernel.org, linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org, Geert Uytterhoeven <geert+renesas@glider.be> Subject: [PATCH] dt-bindings: media: rcar_vin: Use status "okay" Date: Fri, 9 Mar 2018 10:34:40 +0100 Message-Id: <1520588080-31264-1-git-send-email-geert+renesas@glider.be> X-Mailer: git-send-email 2.7.4 Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org |
Commit Message
Geert Uytterhoeven
March 9, 2018, 9:34 a.m. UTC
According to the Devicetree Specification, "ok" is not a valid status.
Fixes: 47c71bd61b772cd7 ("[media] rcar_vin: add devicetree support")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
For the checkpatch TODO list?
https://www.devicetree.org/
Documentation/devicetree/bindings/media/rcar_vin.txt | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Comments
Hi Geert, Thanks for your patch. On 2018-03-09 10:34:40 +0100, Geert Uytterhoeven wrote: > According to the Devicetree Specification, "ok" is not a valid status. > > Fixes: 47c71bd61b772cd7 ("[media] rcar_vin: add devicetree support") > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> Reviewed-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se> > --- > For the checkpatch TODO list? > https://www.devicetree.org/ > > Documentation/devicetree/bindings/media/rcar_vin.txt | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/media/rcar_vin.txt b/Documentation/devicetree/bindings/media/rcar_vin.txt > index 68c5c497b7fa5551..a19517e1c669eb35 100644 > --- a/Documentation/devicetree/bindings/media/rcar_vin.txt > +++ b/Documentation/devicetree/bindings/media/rcar_vin.txt > @@ -81,7 +81,7 @@ Board setup example for Gen2 platforms (vin1 composite video input) > ------------------------------------------------------------------- > > &i2c2 { > - status = "ok"; > + status = "okay"; > pinctrl-0 = <&i2c2_pins>; > pinctrl-names = "default"; > > @@ -104,7 +104,7 @@ Board setup example for Gen2 platforms (vin1 composite video input) > pinctrl-0 = <&vin1_pins>; > pinctrl-names = "default"; > > - status = "ok"; > + status = "okay"; > > port { > #address-cells = <1>; > -- > 2.7.4 >
On Fri, Mar 09, 2018 at 10:34:40AM +0100, Geert Uytterhoeven wrote: > According to the Devicetree Specification, "ok" is not a valid status. > > Fixes: 47c71bd61b772cd7 ("[media] rcar_vin: add devicetree support") > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com> There are a few other matches, too, it'd be nice to fix them as well: $ git grep 'status.*"ok"' -- Documentation/devicetree/bindings/ Documentation/devicetree/bindings/ata/apm-xgene.txt:- status : Shall be "ok" if enabled or "disabled" if disabled. Documentation/devicetree/bindings/display/hisilicon/dw-dsi.txt: status = "ok"; Documentation/devicetree/bindings/display/ti/ti,omap-dss.txt: status = "ok"; Documentation/devicetree/bindings/display/ti/ti,omap-dss.txt: status = "ok"; Documentation/devicetree/bindings/media/rcar_vin.txt: status = "ok"; Documentation/devicetree/bindings/media/rcar_vin.txt: status = "ok"; Documentation/devicetree/bindings/net/apm-xgene-enet.txt:- status: Should be "ok" or "disabled" for enabled/disabled. Default is "ok". Documentation/devicetree/bindings/net/apm-xgene-enet.txt: status = "ok"; Documentation/devicetree/bindings/net/apm-xgene-enet.txt: status = "ok"; Documentation/devicetree/bindings/pci/hisilicon-pcie.txt:- status: Either "ok" or "disabled". Documentation/devicetree/bindings/pci/xgene-pci.txt:- status: Either "ok" or "disabled". Documentation/devicetree/bindings/pci/xgene-pci.txt: status = "ok"; Documentation/devicetree/bindings/phy/apm-xgene-phy.txt:- status : Shall be "ok" if enabled or "disabled" if disabled.
On Fri, Mar 09, 2018 at 10:34:40AM +0100, Geert Uytterhoeven wrote: > According to the Devicetree Specification, "ok" is not a valid status. Correct. > Fixes: 47c71bd61b772cd7 ("[media] rcar_vin: add devicetree support") > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > --- > For the checkpatch TODO list? > https://www.devicetree.org/ > > Documentation/devicetree/bindings/media/rcar_vin.txt | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/media/rcar_vin.txt b/Documentation/devicetree/bindings/media/rcar_vin.txt > index 68c5c497b7fa5551..a19517e1c669eb35 100644 > --- a/Documentation/devicetree/bindings/media/rcar_vin.txt > +++ b/Documentation/devicetree/bindings/media/rcar_vin.txt > @@ -81,7 +81,7 @@ Board setup example for Gen2 platforms (vin1 composite video input) > ------------------------------------------------------------------- > > &i2c2 { > - status = "ok"; > + status = "okay"; However, I prefer that status not be in examples as it applies to any node and the SoC/board split is not relevant to binding docs. I'd cleaned all these up except for the cases with SoC/board split. > pinctrl-0 = <&i2c2_pins>; > pinctrl-names = "default"; > > @@ -104,7 +104,7 @@ Board setup example for Gen2 platforms (vin1 composite video input) > pinctrl-0 = <&vin1_pins>; > pinctrl-names = "default"; > > - status = "ok"; > + status = "okay"; > > port { > #address-cells = <1>; > -- > 2.7.4 >
On Sun, Mar 18, 2018 at 07:47:57AM -0500, Rob Herring wrote: > On Fri, Mar 09, 2018 at 10:34:40AM +0100, Geert Uytterhoeven wrote: > > According to the Devicetree Specification, "ok" is not a valid status. > > Correct. > > > Fixes: 47c71bd61b772cd7 ("[media] rcar_vin: add devicetree support") > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > > --- > > For the checkpatch TODO list? > > https://www.devicetree.org/ > > > > Documentation/devicetree/bindings/media/rcar_vin.txt | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/media/rcar_vin.txt b/Documentation/devicetree/bindings/media/rcar_vin.txt > > index 68c5c497b7fa5551..a19517e1c669eb35 100644 > > --- a/Documentation/devicetree/bindings/media/rcar_vin.txt > > +++ b/Documentation/devicetree/bindings/media/rcar_vin.txt > > @@ -81,7 +81,7 @@ Board setup example for Gen2 platforms (vin1 composite video input) > > ------------------------------------------------------------------- > > > > &i2c2 { > > - status = "ok"; > > + status = "okay"; > > However, I prefer that status not be in examples as it applies to any > node and the SoC/board split is not relevant to binding docs. I'd > cleaned all these up except for the cases with SoC/board split. Hi Rob, I'm a little confused. This example is a board override for a node (implicitly) defined in the DT of an SoC. Could you clarify when the status should be omitted? > > > pinctrl-0 = <&i2c2_pins>; > > pinctrl-names = "default"; > > > > @@ -104,7 +104,7 @@ Board setup example for Gen2 platforms (vin1 composite video input) > > pinctrl-0 = <&vin1_pins>; > > pinctrl-names = "default"; > > > > - status = "ok"; > > + status = "okay"; > > > > port { > > #address-cells = <1>; > > -- > > 2.7.4 > > >
diff --git a/Documentation/devicetree/bindings/media/rcar_vin.txt b/Documentation/devicetree/bindings/media/rcar_vin.txt index 68c5c497b7fa5551..a19517e1c669eb35 100644 --- a/Documentation/devicetree/bindings/media/rcar_vin.txt +++ b/Documentation/devicetree/bindings/media/rcar_vin.txt @@ -81,7 +81,7 @@ Board setup example for Gen2 platforms (vin1 composite video input) ------------------------------------------------------------------- &i2c2 { - status = "ok"; + status = "okay"; pinctrl-0 = <&i2c2_pins>; pinctrl-names = "default"; @@ -104,7 +104,7 @@ Board setup example for Gen2 platforms (vin1 composite video input) pinctrl-0 = <&vin1_pins>; pinctrl-names = "default"; - status = "ok"; + status = "okay"; port { #address-cells = <1>;