2024-01-26 09:00:41

by Chen-Yu Tsai

[permalink] [raw]
Subject: [PATCH] dt-bindings: display: bridge: it6505: Add #sound-dai-cells

The ITE IT6505 display bridge can take one I2S input and transmit it
over the DisplayPort link.

Add #sound-dai-cells (= 0) to the binding for it.

Signed-off-by: Chen-Yu Tsai <[email protected]>
---
The driver side changes [1] are still being worked on, but given the
hardware is very simple, it would be nice if we could land the binding
first and be able to introduct device trees that have this.

[1] https://lore.kernel.org/linux-arm-kernel/[email protected]/

.../devicetree/bindings/display/bridge/ite,it6505.yaml | 3 +++
1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml b/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
index 348b02f26041..7ec4decc9c21 100644
--- a/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
@@ -52,6 +52,9 @@ properties:
maxItems: 1
description: extcon specifier for the Power Delivery

+ "#sound-dai-cells":
+ const: 0
+
ports:
$ref: /schemas/graph.yaml#/properties/ports

--
2.43.0.429.g432eaa2c6b-goog



2024-01-26 10:35:29

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: display: bridge: it6505: Add #sound-dai-cells

On 26/01/2024 08:35, Chen-Yu Tsai wrote:
> The ITE IT6505 display bridge can take one I2S input and transmit it
> over the DisplayPort link.
>
> Add #sound-dai-cells (= 0) to the binding for it.
>
> Signed-off-by: Chen-Yu Tsai <[email protected]>
> ---
> The driver side changes [1] are still being worked on, but given the
> hardware is very simple, it would be nice if we could land the binding
> first and be able to introduct device trees that have this.
>
> [1] https://lore.kernel.org/linux-arm-kernel/[email protected]/
>
> .../devicetree/bindings/display/bridge/ite,it6505.yaml | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml b/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
> index 348b02f26041..7ec4decc9c21 100644
> --- a/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
> @@ -52,6 +52,9 @@ properties:
> maxItems: 1
> description: extcon specifier for the Power Delivery
>
> + "#sound-dai-cells":
> + const: 0

In such case you also want to $ref /schemas/sound/dai-common.yaml.

Best regards,
Krzysztof


2024-01-29 07:16:39

by Chen-Yu Tsai

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: display: bridge: it6505: Add #sound-dai-cells

On Fri, Jan 26, 2024 at 6:17 PM Krzysztof Kozlowski
<[email protected]> wrote:
>
> On 26/01/2024 08:35, Chen-Yu Tsai wrote:
> > The ITE IT6505 display bridge can take one I2S input and transmit it
> > over the DisplayPort link.
> >
> > Add #sound-dai-cells (= 0) to the binding for it.
> >
> > Signed-off-by: Chen-Yu Tsai <[email protected]>
> > ---
> > The driver side changes [1] are still being worked on, but given the
> > hardware is very simple, it would be nice if we could land the binding
> > first and be able to introduct device trees that have this.
> >
> > [1] https://lore.kernel.org/linux-arm-kernel/[email protected]/
> >
> > .../devicetree/bindings/display/bridge/ite,it6505.yaml | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml b/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
> > index 348b02f26041..7ec4decc9c21 100644
> > --- a/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
> > +++ b/Documentation/devicetree/bindings/display/bridge/ite,it6505.yaml
> > @@ -52,6 +52,9 @@ properties:
> > maxItems: 1
> > description: extcon specifier for the Power Delivery
> >
> > + "#sound-dai-cells":
> > + const: 0
>
> In such case you also want to $ref /schemas/sound/dai-common.yaml.

Ack. I assume this also means I should change "additionalProperties: false"
to "unevaluatedProperties: false" in this file.

ChenYu

> Best regards,
> Krzysztof
>