From: Maxime Ripard <maxime@cerno.tech> To: Jyri Sarha <jsarha@ti.com> Cc: dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, yamonkar@cadence.com, praneeth@ti.com, sjakhade@cadence.com, robh+dt@kernel.org, peter.ujfalusi@ti.com, tomi.valkeinen@ti.com, laurent.pinchart@ideasonboard.com, subhajit_paul@ti.com, sam@ravnborg.org Subject: Re: [PATCH v3 1/5] dt-bindings: display: ti, k2g-dss: Add dt-schema yaml binding Date: Fri, 13 Dec 2019 11:42:23 +0100 [thread overview] Message-ID: <20191213104223.u55zlm2zwpgarvil@gilmour.lan> (raw) In-Reply-To: <7c2e386d48cf0c8b932804f24e3f6ea81fe3d3a3.1576158368.git.jsarha@ti.com> [-- Attachment #1: Type: text/plain, Size: 2583 bytes --] Hi, On Thu, Dec 12, 2019 at 03:51:57PM +0200, Jyri Sarha wrote: > Add dt-schema yaml bindig for K2G DSS, an ultra-light version of TI > Keystone Display SubSystem. > > Version history: > > v2: no change > > v3: - Add ports node > - Add includes to dts example > - reindent dts example > > Signed-off-by: Jyri Sarha <jsarha@ti.com> > --- > .../bindings/display/ti/ti,k2g-dss.yaml | 118 ++++++++++++++++++ > 1 file changed, 118 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/ti/ti,k2g-dss.yaml > > diff --git a/Documentation/devicetree/bindings/display/ti/ti,k2g-dss.yaml b/Documentation/devicetree/bindings/display/ti/ti,k2g-dss.yaml > new file mode 100644 > index 000000000000..90158e8b6aad > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/ti/ti,k2g-dss.yaml > @@ -0,0 +1,118 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +# Copyright 2019 Texas Instruments Incorporated > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/display/ti/ti,k2g-dss.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: Texas Instruments K2G Display Subsystem > + > +maintainers: > + - Jyri Sarha <jsarha@ti.com> > + - Tomi Valkeinen <tomi.valkeinen@ti.com> > + > +description: | > + The K2G DSS is an ultra-light version of TI Keystone Display > + SubSystem. It has only one output port and video plane. The > + output is DPI. > + > +properties: > + compatible: > + const: ti,k2g-dss > + > + reg: > + maxItems: 5 > + minItems: 5 When they are equal, you need only one of the two, the tools will add the other. Also, adding a description of what those registers are supposed to be would be great. > + > + reg-names: > + items: > + - const: cfg > + - const: common > + - const: vid1 > + - const: ovr1 > + - const: vp1 > + > + clocks: > + maxItems: 2 > + minItems: 2 Ditto > + > + clock-names: > + items: > + - const: fck > + - const: vp1 > + > + interrupts: > + maxItems: 1 > + > + power-domains: > + maxItems: 1 > + description: phandle to the associated power domain > + > + ports: > + type: object > + description: > + Ports as described in Documentation/devictree/bindings/graph.txt > + properties: > + "#address-cells": > + const: 1 > + > + "#size-cells": > + const: 0 > + > + port@0: > + type: object > + description: > + The DSS DPI output port node If there's only one port, why do you need ports in the first place? Thanks! Maxime [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Maxime Ripard <maxime@cerno.tech> To: Jyri Sarha <jsarha@ti.com> Cc: devicetree@vger.kernel.org, yamonkar@cadence.com, praneeth@ti.com, tomi.valkeinen@ti.com, subhajit_paul@ti.com, dri-devel@lists.freedesktop.org, peter.ujfalusi@ti.com, robh+dt@kernel.org, laurent.pinchart@ideasonboard.com, sjakhade@cadence.com, sam@ravnborg.org Subject: Re: [PATCH v3 1/5] dt-bindings: display: ti, k2g-dss: Add dt-schema yaml binding Date: Fri, 13 Dec 2019 11:42:23 +0100 [thread overview] Message-ID: <20191213104223.u55zlm2zwpgarvil@gilmour.lan> (raw) In-Reply-To: <7c2e386d48cf0c8b932804f24e3f6ea81fe3d3a3.1576158368.git.jsarha@ti.com> [-- Attachment #1.1: Type: text/plain, Size: 2583 bytes --] Hi, On Thu, Dec 12, 2019 at 03:51:57PM +0200, Jyri Sarha wrote: > Add dt-schema yaml bindig for K2G DSS, an ultra-light version of TI > Keystone Display SubSystem. > > Version history: > > v2: no change > > v3: - Add ports node > - Add includes to dts example > - reindent dts example > > Signed-off-by: Jyri Sarha <jsarha@ti.com> > --- > .../bindings/display/ti/ti,k2g-dss.yaml | 118 ++++++++++++++++++ > 1 file changed, 118 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/ti/ti,k2g-dss.yaml > > diff --git a/Documentation/devicetree/bindings/display/ti/ti,k2g-dss.yaml b/Documentation/devicetree/bindings/display/ti/ti,k2g-dss.yaml > new file mode 100644 > index 000000000000..90158e8b6aad > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/ti/ti,k2g-dss.yaml > @@ -0,0 +1,118 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +# Copyright 2019 Texas Instruments Incorporated > +%YAML 1.2 > +--- > +$id: "http://devicetree.org/schemas/display/ti/ti,k2g-dss.yaml#" > +$schema: "http://devicetree.org/meta-schemas/core.yaml#" > + > +title: Texas Instruments K2G Display Subsystem > + > +maintainers: > + - Jyri Sarha <jsarha@ti.com> > + - Tomi Valkeinen <tomi.valkeinen@ti.com> > + > +description: | > + The K2G DSS is an ultra-light version of TI Keystone Display > + SubSystem. It has only one output port and video plane. The > + output is DPI. > + > +properties: > + compatible: > + const: ti,k2g-dss > + > + reg: > + maxItems: 5 > + minItems: 5 When they are equal, you need only one of the two, the tools will add the other. Also, adding a description of what those registers are supposed to be would be great. > + > + reg-names: > + items: > + - const: cfg > + - const: common > + - const: vid1 > + - const: ovr1 > + - const: vp1 > + > + clocks: > + maxItems: 2 > + minItems: 2 Ditto > + > + clock-names: > + items: > + - const: fck > + - const: vp1 > + > + interrupts: > + maxItems: 1 > + > + power-domains: > + maxItems: 1 > + description: phandle to the associated power domain > + > + ports: > + type: object > + description: > + Ports as described in Documentation/devictree/bindings/graph.txt > + properties: > + "#address-cells": > + const: 1 > + > + "#size-cells": > + const: 0 > + > + port@0: > + type: object > + description: > + The DSS DPI output port node If there's only one port, why do you need ports in the first place? Thanks! Maxime [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] [-- Attachment #2: Type: text/plain, Size: 160 bytes --] _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2019-12-13 10:42 UTC|newest] Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-12-12 13:51 [PATCH v3 0/5] drm/tidss: New driver for TI Keystone platform Display SubSystem Jyri Sarha 2019-12-12 13:51 ` Jyri Sarha 2019-12-12 13:51 ` [PATCH v3 1/5] dt-bindings: display: ti,k2g-dss: Add dt-schema yaml binding Jyri Sarha 2019-12-12 13:51 ` [PATCH v3 1/5] dt-bindings: display: ti, k2g-dss: " Jyri Sarha 2019-12-13 10:42 ` Maxime Ripard [this message] 2019-12-13 10:42 ` Maxime Ripard 2019-12-12 13:51 ` [PATCH v3 2/5] dt-bindings: display: ti,am65x-dss: " Jyri Sarha 2019-12-12 13:51 ` [PATCH v3 2/5] dt-bindings: display: ti, am65x-dss: " Jyri Sarha 2019-12-12 13:51 ` [PATCH v3 3/5] dt-bindings: display: ti,j721e-dss: " Jyri Sarha 2019-12-12 13:51 ` [PATCH v3 3/5] dt-bindings: display: ti, j721e-dss: " Jyri Sarha 2019-12-12 13:52 ` [PATCH v3 4/5] drm/tidss: New driver for TI Keystone platform Display SubSystem Jyri Sarha 2019-12-12 13:52 ` Jyri Sarha 2019-12-12 13:52 ` [PATCH v3 5/5] MAINTAINERS: add entry for tidss Jyri Sarha 2019-12-12 13:52 ` Jyri Sarha 2019-12-13 10:30 ` Daniel Vetter 2019-12-13 10:30 ` Daniel Vetter 2019-12-13 11:17 ` Tomi Valkeinen 2019-12-13 11:17 ` Tomi Valkeinen 2019-12-13 20:15 ` Daniel Vetter 2019-12-13 20:15 ` Daniel Vetter
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=20191213104223.u55zlm2zwpgarvil@gilmour.lan \ --to=maxime@cerno.tech \ --cc=devicetree@vger.kernel.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=jsarha@ti.com \ --cc=laurent.pinchart@ideasonboard.com \ --cc=peter.ujfalusi@ti.com \ --cc=praneeth@ti.com \ --cc=robh+dt@kernel.org \ --cc=sam@ravnborg.org \ --cc=sjakhade@cadence.com \ --cc=subhajit_paul@ti.com \ --cc=tomi.valkeinen@ti.com \ --cc=yamonkar@cadence.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: linkBe 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.