Re: [PATCH v2] dt-bindings: display: Convert connectors to DT schema
Hi Rob On Tue, Jun 30, 2020 at 02:02:16PM -0600, Rob Herring wrote: > Convert the analog TV, DVI, HDMI, and VGA connector bindings to DT schema > format. > > Cc: Sam Ravnborg > Cc: Laurent Pinchart > Cc: Maxime Ripard > Signed-off-by: Rob Herring Reviewed-by: Sam Ravnborg > --- > v2: > - Make Laurent maintainer > - Add missing port and compatible required > - Drop copy-n-paste 'type' from dvi-connector > - Use 4 space indent on examples I assume you take this via the DT tree. Sam > --- > .../display/connector/analog-tv-connector.txt | 31 > .../connector/analog-tv-connector.yaml| 52 ++ > .../display/connector/dvi-connector.txt | 36 -- > .../display/connector/dvi-connector.yaml | 70 +++ > .../display/connector/hdmi-connector.txt | 31 > .../display/connector/hdmi-connector.yaml | 64 + > .../display/connector/vga-connector.txt | 36 -- > .../display/connector/vga-connector.yaml | 46 > 8 files changed, 232 insertions(+), 134 deletions(-) > delete mode 100644 > Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt > create mode 100644 > Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml > delete mode 100644 > Documentation/devicetree/bindings/display/connector/dvi-connector.txt > create mode 100644 > Documentation/devicetree/bindings/display/connector/dvi-connector.yaml > delete mode 100644 > Documentation/devicetree/bindings/display/connector/hdmi-connector.txt > create mode 100644 > Documentation/devicetree/bindings/display/connector/hdmi-connector.yaml > delete mode 100644 > Documentation/devicetree/bindings/display/connector/vga-connector.txt > create mode 100644 > Documentation/devicetree/bindings/display/connector/vga-connector.yaml > > diff --git > a/Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt > b/Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt > deleted file mode 100644 > index 883bcb2604c7.. > --- > a/Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt > +++ /dev/null > @@ -1,31 +0,0 @@ > -Analog TV Connector > -=== > - > -Required properties: > -- compatible: "composite-video-connector" or "svideo-connector" > - > -Optional properties: > -- label: a symbolic name for the connector > -- sdtv-standards: limit the supported TV standards on a connector to the > given > - ones. If not specified all TV standards are allowed. > - Possible TV standards are defined in > - include/dt-bindings/display/sdtv-standards.h. > - > -Required nodes: > -- Video port for TV input > - > -Example > > -#include > - > -tv: connector { > - compatible = "composite-video-connector"; > - label = "tv"; > - sdtv-standards = <(SDTV_STD_PAL | SDTV_STD_NTSC)>; > - > - port { > - tv_connector_in: endpoint { > - remote-endpoint = <&venc_out>; > - }; > - }; > -}; > diff --git > a/Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml > > b/Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml > new file mode 100644 > index ..eebe88fed999 > --- /dev/null > +++ > b/Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml > @@ -0,0 +1,52 @@ > +# SPDX-License-Identifier: GPL-2.0-only > +%YAML 1.2 > +--- > +$id: > http://devicetree.org/schemas/display/connector/analog-tv-connector.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Analog TV Connector > + > +maintainers: > + - Laurent Pinchart > + > +properties: > + compatible: > +enum: > + - composite-video-connector > + - svideo-connector > + > + label: true > + > + sdtv-standards: > +description: > + Limit the supported TV standards on a connector to the given ones. If > + not specified all TV standards are allowed. Possible TV standards are > + defined in include/dt-bindings/display/sdtv-standards.h. > +$ref: /schemas/types.yaml#/definitions/uint32 > + > + port: > +description: Connection to controller providing analog TV signals > + > +required: > + - compatible > + - port > + > +additionalProperties: false > + > +examples: > + - | > +#include > + > +connector { > +compatible = "composite-video-connector"; > +label = "tv"; > +sdtv-standards = <(SDTV_STD_PAL | SDTV_STD_NTSC)>; > + > +port { > +tv_connector_in: endpoint { > +remote-endpoint = <&venc_out>; > +}; > +}; > +}; > + > +... > diff --git > a/Documentation/devicetree/bindings/display/connector/dvi-connector.txt > b/Documentation/devicetree/bindings/display/connector/dvi-connector.txt > deleted file mode 100644 > index 207e42e9eba0..0
Re: [PATCH v2] dt-bindings: display: Convert connectors to DT schema
On Tue, Jun 30, 2020 at 02:02:16PM -0600, Rob Herring wrote: > Convert the analog TV, DVI, HDMI, and VGA connector bindings to DT schema > format. > > Cc: Sam Ravnborg > Cc: Laurent Pinchart > Cc: Maxime Ripard > Signed-off-by: Rob Herring Reviewed-by: Maxime Ripard Thanks! Maxime signature.asc Description: PGP signature ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH v2] dt-bindings: display: Convert connectors to DT schema
Hi Rob, Thank you for the patch. On Tue, Jun 30, 2020 at 02:02:16PM -0600, Rob Herring wrote: > Convert the analog TV, DVI, HDMI, and VGA connector bindings to DT schema > format. > > Cc: Sam Ravnborg > Cc: Laurent Pinchart > Cc: Maxime Ripard > Signed-off-by: Rob Herring Reviewed-by: Laurent Pinchart > --- > v2: > - Make Laurent maintainer > - Add missing port and compatible required > - Drop copy-n-paste 'type' from dvi-connector > - Use 4 space indent on examples > --- > .../display/connector/analog-tv-connector.txt | 31 > .../connector/analog-tv-connector.yaml| 52 ++ > .../display/connector/dvi-connector.txt | 36 -- > .../display/connector/dvi-connector.yaml | 70 +++ > .../display/connector/hdmi-connector.txt | 31 > .../display/connector/hdmi-connector.yaml | 64 + > .../display/connector/vga-connector.txt | 36 -- > .../display/connector/vga-connector.yaml | 46 > 8 files changed, 232 insertions(+), 134 deletions(-) > delete mode 100644 > Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt > create mode 100644 > Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml > delete mode 100644 > Documentation/devicetree/bindings/display/connector/dvi-connector.txt > create mode 100644 > Documentation/devicetree/bindings/display/connector/dvi-connector.yaml > delete mode 100644 > Documentation/devicetree/bindings/display/connector/hdmi-connector.txt > create mode 100644 > Documentation/devicetree/bindings/display/connector/hdmi-connector.yaml > delete mode 100644 > Documentation/devicetree/bindings/display/connector/vga-connector.txt > create mode 100644 > Documentation/devicetree/bindings/display/connector/vga-connector.yaml > > diff --git > a/Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt > b/Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt > deleted file mode 100644 > index 883bcb2604c7.. > --- > a/Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt > +++ /dev/null > @@ -1,31 +0,0 @@ > -Analog TV Connector > -=== > - > -Required properties: > -- compatible: "composite-video-connector" or "svideo-connector" > - > -Optional properties: > -- label: a symbolic name for the connector > -- sdtv-standards: limit the supported TV standards on a connector to the > given > - ones. If not specified all TV standards are allowed. > - Possible TV standards are defined in > - include/dt-bindings/display/sdtv-standards.h. > - > -Required nodes: > -- Video port for TV input > - > -Example > > -#include > - > -tv: connector { > - compatible = "composite-video-connector"; > - label = "tv"; > - sdtv-standards = <(SDTV_STD_PAL | SDTV_STD_NTSC)>; > - > - port { > - tv_connector_in: endpoint { > - remote-endpoint = <&venc_out>; > - }; > - }; > -}; > diff --git > a/Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml > > b/Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml > new file mode 100644 > index ..eebe88fed999 > --- /dev/null > +++ > b/Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml > @@ -0,0 +1,52 @@ > +# SPDX-License-Identifier: GPL-2.0-only > +%YAML 1.2 > +--- > +$id: > http://devicetree.org/schemas/display/connector/analog-tv-connector.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Analog TV Connector > + > +maintainers: > + - Laurent Pinchart > + > +properties: > + compatible: > +enum: > + - composite-video-connector > + - svideo-connector > + > + label: true > + > + sdtv-standards: > +description: > + Limit the supported TV standards on a connector to the given ones. If > + not specified all TV standards are allowed. Possible TV standards are > + defined in include/dt-bindings/display/sdtv-standards.h. > +$ref: /schemas/types.yaml#/definitions/uint32 > + > + port: > +description: Connection to controller providing analog TV signals > + > +required: > + - compatible > + - port > + > +additionalProperties: false > + > +examples: > + - | > +#include > + > +connector { > +compatible = "composite-video-connector"; > +label = "tv"; > +sdtv-standards = <(SDTV_STD_PAL | SDTV_STD_NTSC)>; > + > +port { > +tv_connector_in: endpoint { > +remote-endpoint = <&venc_out>; > +}; > +}; > +}; > + > +... > diff --git > a/Documentation/devicetree/bindings/display/connector/dvi-connector.txt > b/Documentation/devicetree/bindings/display/connector/dvi-connector.txt > deleted file mode 100644 > index 207e42e9eba0.. > --- a/
Re: [PATCH v2] dt-bindings: display: Convert connectors to DT schema
On Tue, Jun 30, 2020 at 02:02:16PM -0600, Rob Herring wrote: > Convert the analog TV, DVI, HDMI, and VGA connector bindings to DT schema > format. > > Cc: Sam Ravnborg > Cc: Laurent Pinchart > Cc: Maxime Ripard > Signed-off-by: Rob Herring Acked-by: Sam Ravnborg > --- > v2: > - Make Laurent maintainer > - Add missing port and compatible required > - Drop copy-n-paste 'type' from dvi-connector > - Use 4 space indent on examples > --- > .../display/connector/analog-tv-connector.txt | 31 > .../connector/analog-tv-connector.yaml| 52 ++ > .../display/connector/dvi-connector.txt | 36 -- > .../display/connector/dvi-connector.yaml | 70 +++ > .../display/connector/hdmi-connector.txt | 31 > .../display/connector/hdmi-connector.yaml | 64 + > .../display/connector/vga-connector.txt | 36 -- > .../display/connector/vga-connector.yaml | 46 > 8 files changed, 232 insertions(+), 134 deletions(-) > delete mode 100644 > Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt > create mode 100644 > Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml > delete mode 100644 > Documentation/devicetree/bindings/display/connector/dvi-connector.txt > create mode 100644 > Documentation/devicetree/bindings/display/connector/dvi-connector.yaml > delete mode 100644 > Documentation/devicetree/bindings/display/connector/hdmi-connector.txt > create mode 100644 > Documentation/devicetree/bindings/display/connector/hdmi-connector.yaml > delete mode 100644 > Documentation/devicetree/bindings/display/connector/vga-connector.txt > create mode 100644 > Documentation/devicetree/bindings/display/connector/vga-connector.yaml > > diff --git > a/Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt > b/Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt > deleted file mode 100644 > index 883bcb2604c7.. > --- > a/Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt > +++ /dev/null > @@ -1,31 +0,0 @@ > -Analog TV Connector > -=== > - > -Required properties: > -- compatible: "composite-video-connector" or "svideo-connector" > - > -Optional properties: > -- label: a symbolic name for the connector > -- sdtv-standards: limit the supported TV standards on a connector to the > given > - ones. If not specified all TV standards are allowed. > - Possible TV standards are defined in > - include/dt-bindings/display/sdtv-standards.h. > - > -Required nodes: > -- Video port for TV input > - > -Example > > -#include > - > -tv: connector { > - compatible = "composite-video-connector"; > - label = "tv"; > - sdtv-standards = <(SDTV_STD_PAL | SDTV_STD_NTSC)>; > - > - port { > - tv_connector_in: endpoint { > - remote-endpoint = <&venc_out>; > - }; > - }; > -}; > diff --git > a/Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml > > b/Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml > new file mode 100644 > index ..eebe88fed999 > --- /dev/null > +++ > b/Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml > @@ -0,0 +1,52 @@ > +# SPDX-License-Identifier: GPL-2.0-only > +%YAML 1.2 > +--- > +$id: > http://devicetree.org/schemas/display/connector/analog-tv-connector.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Analog TV Connector > + > +maintainers: > + - Laurent Pinchart > + > +properties: > + compatible: > +enum: > + - composite-video-connector > + - svideo-connector > + > + label: true > + > + sdtv-standards: > +description: > + Limit the supported TV standards on a connector to the given ones. If > + not specified all TV standards are allowed. Possible TV standards are > + defined in include/dt-bindings/display/sdtv-standards.h. > +$ref: /schemas/types.yaml#/definitions/uint32 > + > + port: > +description: Connection to controller providing analog TV signals > + > +required: > + - compatible > + - port > + > +additionalProperties: false > + > +examples: > + - | > +#include > + > +connector { > +compatible = "composite-video-connector"; > +label = "tv"; > +sdtv-standards = <(SDTV_STD_PAL | SDTV_STD_NTSC)>; > + > +port { > +tv_connector_in: endpoint { > +remote-endpoint = <&venc_out>; > +}; > +}; > +}; > + > +... > diff --git > a/Documentation/devicetree/bindings/display/connector/dvi-connector.txt > b/Documentation/devicetree/bindings/display/connector/dvi-connector.txt > deleted file mode 100644 > index 207e42e9eba0.. > --- a/Documentation/devicetree/bindings/display/co
[PATCH v2] dt-bindings: display: Convert connectors to DT schema
Convert the analog TV, DVI, HDMI, and VGA connector bindings to DT schema format. Cc: Sam Ravnborg Cc: Laurent Pinchart Cc: Maxime Ripard Signed-off-by: Rob Herring --- v2: - Make Laurent maintainer - Add missing port and compatible required - Drop copy-n-paste 'type' from dvi-connector - Use 4 space indent on examples --- .../display/connector/analog-tv-connector.txt | 31 .../connector/analog-tv-connector.yaml| 52 ++ .../display/connector/dvi-connector.txt | 36 -- .../display/connector/dvi-connector.yaml | 70 +++ .../display/connector/hdmi-connector.txt | 31 .../display/connector/hdmi-connector.yaml | 64 + .../display/connector/vga-connector.txt | 36 -- .../display/connector/vga-connector.yaml | 46 8 files changed, 232 insertions(+), 134 deletions(-) delete mode 100644 Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt create mode 100644 Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml delete mode 100644 Documentation/devicetree/bindings/display/connector/dvi-connector.txt create mode 100644 Documentation/devicetree/bindings/display/connector/dvi-connector.yaml delete mode 100644 Documentation/devicetree/bindings/display/connector/hdmi-connector.txt create mode 100644 Documentation/devicetree/bindings/display/connector/hdmi-connector.yaml delete mode 100644 Documentation/devicetree/bindings/display/connector/vga-connector.txt create mode 100644 Documentation/devicetree/bindings/display/connector/vga-connector.yaml diff --git a/Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt b/Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt deleted file mode 100644 index 883bcb2604c7.. --- a/Documentation/devicetree/bindings/display/connector/analog-tv-connector.txt +++ /dev/null @@ -1,31 +0,0 @@ -Analog TV Connector -=== - -Required properties: -- compatible: "composite-video-connector" or "svideo-connector" - -Optional properties: -- label: a symbolic name for the connector -- sdtv-standards: limit the supported TV standards on a connector to the given - ones. If not specified all TV standards are allowed. - Possible TV standards are defined in - include/dt-bindings/display/sdtv-standards.h. - -Required nodes: -- Video port for TV input - -Example -#include - -tv: connector { - compatible = "composite-video-connector"; - label = "tv"; - sdtv-standards = <(SDTV_STD_PAL | SDTV_STD_NTSC)>; - - port { - tv_connector_in: endpoint { - remote-endpoint = <&venc_out>; - }; - }; -}; diff --git a/Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml b/Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml new file mode 100644 index ..eebe88fed999 --- /dev/null +++ b/Documentation/devicetree/bindings/display/connector/analog-tv-connector.yaml @@ -0,0 +1,52 @@ +# SPDX-License-Identifier: GPL-2.0-only +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/display/connector/analog-tv-connector.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Analog TV Connector + +maintainers: + - Laurent Pinchart + +properties: + compatible: +enum: + - composite-video-connector + - svideo-connector + + label: true + + sdtv-standards: +description: + Limit the supported TV standards on a connector to the given ones. If + not specified all TV standards are allowed. Possible TV standards are + defined in include/dt-bindings/display/sdtv-standards.h. +$ref: /schemas/types.yaml#/definitions/uint32 + + port: +description: Connection to controller providing analog TV signals + +required: + - compatible + - port + +additionalProperties: false + +examples: + - | +#include + +connector { +compatible = "composite-video-connector"; +label = "tv"; +sdtv-standards = <(SDTV_STD_PAL | SDTV_STD_NTSC)>; + +port { +tv_connector_in: endpoint { +remote-endpoint = <&venc_out>; +}; +}; +}; + +... diff --git a/Documentation/devicetree/bindings/display/connector/dvi-connector.txt b/Documentation/devicetree/bindings/display/connector/dvi-connector.txt deleted file mode 100644 index 207e42e9eba0.. --- a/Documentation/devicetree/bindings/display/connector/dvi-connector.txt +++ /dev/null @@ -1,36 +0,0 @@ -DVI Connector -== - -Required properties: -- compatible: "dvi-connector" - -Optional properties: -- label: a symbolic name for the connector -- ddc-i2c-bus: phandle to the i2c bus that is connected to DVI DDC -- analog: the connector has DVI analog pins -- digital: the connector has DVI digital pins -