[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20220701170854.GA1092383-robh@kernel.org>
Date: Fri, 1 Jul 2022 11:08:54 -0600
From: Rob Herring <robh@...nel.org>
To: Max Krummenacher <max.oss.09@...il.com>
Cc: max.krummenacher@...adex.com,
Laurent Pinchart <laurent.pinchart@...asonboard.com>,
Dave Stevenson <dave.stevenson@...pberrypi.com>,
Maxime Ripard <mripard@...nel.org>,
Marek Vasut <marex@...x.de>,
Christoph Niedermaier <cniedermaier@...electronics.com>,
Francesco Dolcini <francesco.dolcini@...adex.com>,
Daniel Vetter <daniel@...ll.ch>,
David Airlie <airlied@...ux.ie>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Laurent Pinchart <laurent.pinchart+renesas@...asonboard.com>,
Sam Ravnborg <sam@...nborg.org>,
Thierry Reding <thierry.reding@...il.com>,
devicetree@...r.kernel.org, dri-devel@...ts.freedesktop.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 1/4] dt-bindings: display: panel-common: allow for
additional port node properties
On Tue, Jun 28, 2022 at 08:18:35PM +0200, Max Krummenacher wrote:
> From: Max Krummenacher <max.krummenacher@...adex.com>
>
> Allow bindings which reference panel-common.yaml to add additional
> properties under the port node.
> I.e. 'panel-dpi' needs to add a new property to 'port/endpoint'.
>
> Signed-off-by: Max Krummenacher <max.krummenacher@...adex.com>
>
> ---
>
> Changes in v3:
> - New commit to allow for additional port node properties
>
> .../devicetree/bindings/display/panel/panel-common.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/display/panel/panel-common.yaml b/Documentation/devicetree/bindings/display/panel/panel-common.yaml
> index 5b38dc89cb21..ff8dc07ef3b5 100644
> --- a/Documentation/devicetree/bindings/display/panel/panel-common.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/panel-common.yaml
> @@ -68,7 +68,7 @@ properties:
>
> # Connectivity
> port:
> - $ref: /schemas/graph.yaml#/properties/port
> + $ref: /schemas/graph.yaml#/$defs/port-base
This will allow extra properties for everyone using this. That means
either bus-format needs to go in here (so that it is the only extra
property allowed) or we should remove 'port' here and push this into all
the users.
But we should reach agreement on bus-format before doing anything.
Rob
Powered by blists - more mailing lists