From: Maxime Ripard <maxime@cerno.tech> To: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Cc: devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org, Rob Herring <robh+dt@kernel.org>, Philipp Zabel <p.zabel@pengutronix.de>, Mark Yao <mark.yao@rock-chips.com>, Sandy Huang <hjc@rock-chips.com>, Chen-Yu Tsai <wens@csie.org> Subject: Re: [PATCH/RFC 2/6] dt-bindings: display: bridge: renesas,dw-hdmi: Convert binding to YAML Date: Mon, 6 Apr 2020 09:57:05 +0200 [thread overview] Message-ID: <20200406075705.a67tjx7jamdvqpl3@gilmour.lan> (raw) In-Reply-To: <20200405233935.27599-3-laurent.pinchart+renesas@ideasonboard.com> [-- Attachment #1: Type: text/plain, Size: 3737 bytes --] Hi, On Mon, Apr 06, 2020 at 02:39:31AM +0300, Laurent Pinchart wrote: > diff --git a/Documentation/devicetree/bindings/display/bridge/renesas,dw-hdmi.yaml b/Documentation/devicetree/bindings/display/bridge/renesas,dw-hdmi.yaml > new file mode 100644 > index 000000000000..9a543740c81d > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/bridge/renesas,dw-hdmi.yaml > @@ -0,0 +1,142 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/display/bridge/renesas,dw-hdmi.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Renesas R-Car DWC HDMI TX Encoder > + > +maintainers: > + - Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > + > +description: | > + The HDMI transmitter is a Synopsys DesignWare HDMI 1.4 TX controller IP > + with a companion PHY IP. > + > +allOf: > + - $ref: synopsys,dw-hdmi.yaml# > + > +properties: > + compatible: > + items: > + - enum: > + - renesas,r8a774a1-hdmi # for R8A774A1 (RZ/G2M) compatible HDMI TX > + - renesas,r8a774b1-hdmi # for R8A774B1 (RZ/G2N) compatible HDMI TX > + - renesas,r8a7795-hdmi # for R8A7795 (R-Car H3) compatible HDMI TX > + - renesas,r8a7796-hdmi # for R8A7796 (R-Car M3-W) compatible HDMI TX > + - renesas,r8a77965-hdmi # for R8A77965 (R-Car M3-N) compatible HDMI TX > + - const: renesas,rcar-gen3-hdmi > + > + reg: true > + > + reg-io-width: > + const: 1 > + > + clocks: > + minItems: 2 > + maxItems: 2 You don't need both, if one is missing the other will be filled by the dt-schema tools. In this particular case, I guess maxItems would make more sense. > + > + clock-names: > + items: > + - const: iahb > + - const: isfr > + > + interrupts: true > + > + ports: > + type: object > + description: | > + This device has three video ports. Their connections are modelled using the > + OF graph bindings specified in Documentation/devicetree/bindings/graph.txt. > + Each port shall have a single endpoint. > + > + properties: > + '#address-cells': > + const: 1 > + > + '#size-cells': > + const: 0 > + > + port@0: > + type: object > + description: Parallel RGB input port > + > + port@1: > + type: object > + description: HDMI output port > + > + port@2: > + type: object > + description: Sound input port > + > + required: > + - port@0 > + - port@1 > + - port@2 > + > + additionalProperties: false > + > + power-domains: > + maxItems: 1 > + > +required: > + - compatible > + - reg > + - clocks > + - clock-names > + - interrupts > + - ports > + > +additionalProperties: false In the case where you have some kind of generic schema and then a more specific one like you have here, unevaluatedProperties make more sense that additionalProperties. additionalProperties checks that there are no extra properties on the current schema, which is a problem here since you have to duplicate the entire list of properties found in the generic schema, while unevaluatedProperties checks that there are no extra properties in the entire set of all schemas that apply to this node. This way, you can just put what is different from the generic schema, and you don't have to keep it in sync. It's a feature that has been added in the spec of the schemas that went on right after the one we support in the tools, so for now the kernel meta-schemas only allows that property to be there (just like deprecated) but won't do anything. This should be fixed quite soon however, the library we depend on has started to work on that spec apparently. 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: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Cc: devicetree@vger.kernel.org, Sandy Huang <hjc@rock-chips.com>, dri-devel@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org, Chen-Yu Tsai <wens@csie.org>, Rob Herring <robh+dt@kernel.org>, Mark Yao <mark.yao@rock-chips.com> Subject: Re: [PATCH/RFC 2/6] dt-bindings: display: bridge: renesas,dw-hdmi: Convert binding to YAML Date: Mon, 6 Apr 2020 09:57:05 +0200 [thread overview] Message-ID: <20200406075705.a67tjx7jamdvqpl3@gilmour.lan> (raw) In-Reply-To: <20200405233935.27599-3-laurent.pinchart+renesas@ideasonboard.com> [-- Attachment #1.1: Type: text/plain, Size: 3737 bytes --] Hi, On Mon, Apr 06, 2020 at 02:39:31AM +0300, Laurent Pinchart wrote: > diff --git a/Documentation/devicetree/bindings/display/bridge/renesas,dw-hdmi.yaml b/Documentation/devicetree/bindings/display/bridge/renesas,dw-hdmi.yaml > new file mode 100644 > index 000000000000..9a543740c81d > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/bridge/renesas,dw-hdmi.yaml > @@ -0,0 +1,142 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/display/bridge/renesas,dw-hdmi.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Renesas R-Car DWC HDMI TX Encoder > + > +maintainers: > + - Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> > + > +description: | > + The HDMI transmitter is a Synopsys DesignWare HDMI 1.4 TX controller IP > + with a companion PHY IP. > + > +allOf: > + - $ref: synopsys,dw-hdmi.yaml# > + > +properties: > + compatible: > + items: > + - enum: > + - renesas,r8a774a1-hdmi # for R8A774A1 (RZ/G2M) compatible HDMI TX > + - renesas,r8a774b1-hdmi # for R8A774B1 (RZ/G2N) compatible HDMI TX > + - renesas,r8a7795-hdmi # for R8A7795 (R-Car H3) compatible HDMI TX > + - renesas,r8a7796-hdmi # for R8A7796 (R-Car M3-W) compatible HDMI TX > + - renesas,r8a77965-hdmi # for R8A77965 (R-Car M3-N) compatible HDMI TX > + - const: renesas,rcar-gen3-hdmi > + > + reg: true > + > + reg-io-width: > + const: 1 > + > + clocks: > + minItems: 2 > + maxItems: 2 You don't need both, if one is missing the other will be filled by the dt-schema tools. In this particular case, I guess maxItems would make more sense. > + > + clock-names: > + items: > + - const: iahb > + - const: isfr > + > + interrupts: true > + > + ports: > + type: object > + description: | > + This device has three video ports. Their connections are modelled using the > + OF graph bindings specified in Documentation/devicetree/bindings/graph.txt. > + Each port shall have a single endpoint. > + > + properties: > + '#address-cells': > + const: 1 > + > + '#size-cells': > + const: 0 > + > + port@0: > + type: object > + description: Parallel RGB input port > + > + port@1: > + type: object > + description: HDMI output port > + > + port@2: > + type: object > + description: Sound input port > + > + required: > + - port@0 > + - port@1 > + - port@2 > + > + additionalProperties: false > + > + power-domains: > + maxItems: 1 > + > +required: > + - compatible > + - reg > + - clocks > + - clock-names > + - interrupts > + - ports > + > +additionalProperties: false In the case where you have some kind of generic schema and then a more specific one like you have here, unevaluatedProperties make more sense that additionalProperties. additionalProperties checks that there are no extra properties on the current schema, which is a problem here since you have to duplicate the entire list of properties found in the generic schema, while unevaluatedProperties checks that there are no extra properties in the entire set of all schemas that apply to this node. This way, you can just put what is different from the generic schema, and you don't have to keep it in sync. It's a feature that has been added in the spec of the schemas that went on right after the one we support in the tools, so for now the kernel meta-schemas only allows that property to be there (just like deprecated) but won't do anything. This should be fixed quite soon however, the library we depend on has started to work on that spec apparently. 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:[~2020-04-06 7:57 UTC|newest] Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-04-05 23:39 [PATCH/RFC 0/6] dt-bindings: display: Convert DWC HDMI TX bindings to YAML Laurent Pinchart 2020-04-05 23:39 ` Laurent Pinchart 2020-04-05 23:39 ` [PATCH/RFC 1/6] dt-bindings: display: bridge: Add YAML schema for Synopsys DW-HDMI Laurent Pinchart 2020-04-05 23:39 ` Laurent Pinchart 2020-04-06 7:50 ` Maxime Ripard 2020-04-06 7:50 ` Maxime Ripard 2020-04-05 23:39 ` [PATCH/RFC 2/6] dt-bindings: display: bridge: renesas,dw-hdmi: Convert binding to YAML Laurent Pinchart 2020-04-05 23:39 ` [PATCH/RFC 2/6] dt-bindings: display: bridge: renesas, dw-hdmi: " Laurent Pinchart 2020-04-06 7:57 ` Maxime Ripard [this message] 2020-04-06 7:57 ` [PATCH/RFC 2/6] dt-bindings: display: bridge: renesas,dw-hdmi: " Maxime Ripard 2020-04-06 11:28 ` Laurent Pinchart 2020-04-06 11:28 ` Laurent Pinchart 2020-04-06 11:32 ` Laurent Pinchart 2020-04-06 11:32 ` Laurent Pinchart 2020-04-06 17:04 ` Maxime Ripard 2020-04-06 17:04 ` Maxime Ripard 2020-04-06 17:06 ` Maxime Ripard 2020-04-06 17:06 ` Maxime Ripard 2020-04-06 8:51 ` Geert Uytterhoeven 2020-04-06 8:51 ` Geert Uytterhoeven 2020-04-05 23:39 ` [PATCH/RFC 3/6] dt-bindings: display: imx: hdmi: " Laurent Pinchart 2020-04-05 23:39 ` Laurent Pinchart 2020-04-05 23:39 ` [PATCH/RFC 4/6] dt-bindings: display: rockchip: dw-hdmi: " Laurent Pinchart 2020-04-05 23:39 ` Laurent Pinchart 2020-04-06 8:00 ` Maxime Ripard 2020-04-06 8:00 ` Maxime Ripard 2020-04-06 11:19 ` Laurent Pinchart 2020-04-06 11:19 ` Laurent Pinchart 2020-04-06 17:09 ` Maxime Ripard 2020-04-06 17:09 ` Maxime Ripard 2020-04-06 17:50 ` Laurent Pinchart 2020-04-06 17:50 ` Laurent Pinchart 2020-04-07 7:12 ` Maxime Ripard 2020-04-07 7:12 ` Maxime Ripard 2020-04-08 11:45 ` Laurent Pinchart 2020-04-08 11:45 ` Laurent Pinchart 2020-04-08 12:25 ` Maxime Ripard 2020-04-08 12:25 ` Maxime Ripard 2020-04-08 14:01 ` Laurent Pinchart 2020-04-08 14:01 ` Laurent Pinchart 2020-04-14 23:10 ` Rob Herring 2020-04-14 23:10 ` Rob Herring 2020-04-15 1:06 ` Laurent Pinchart 2020-04-15 1:06 ` Laurent Pinchart 2020-04-15 13:50 ` Rob Herring 2020-04-15 13:50 ` Rob Herring 2020-04-05 23:39 ` [PATCH/RFC 5/6] dt-bindings: display: sun8i-a83t-dw-hdmi: Reference dw-hdmi YAML schema Laurent Pinchart 2020-04-05 23:39 ` Laurent Pinchart 2020-04-14 23:10 ` Rob Herring 2020-04-14 23:10 ` Rob Herring 2020-04-05 23:39 ` [PATCH/RFC 6/6] dt-bindings: display: bridge: Remove deprecated dw_hdmi.txt Laurent Pinchart 2020-04-05 23:39 ` Laurent Pinchart 2020-04-14 23:11 ` Rob Herring 2020-04-14 23:11 ` Rob Herring 2020-04-05 23:55 ` [PATCH/RFC 0/6] dt-bindings: display: Convert DWC HDMI TX bindings to YAML Laurent Pinchart 2020-04-05 23:55 ` Laurent Pinchart
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=20200406075705.a67tjx7jamdvqpl3@gilmour.lan \ --to=maxime@cerno.tech \ --cc=devicetree@vger.kernel.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=hjc@rock-chips.com \ --cc=laurent.pinchart+renesas@ideasonboard.com \ --cc=linux-renesas-soc@vger.kernel.org \ --cc=mark.yao@rock-chips.com \ --cc=p.zabel@pengutronix.de \ --cc=robh+dt@kernel.org \ --cc=wens@csie.org \ /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.