| Message ID | 20241025104310.1210946-1-fshao@chromium.org (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | dt-bindings: display: mediatek: dp: Add #sound-dai-cells property | expand |
On Fri, Oct 25, 2024 at 06:42:45PM +0800, Fei Shao wrote: > The MediaTek DP hardware supports audio, and the "#sound-dai-cells" > property is required to describe DAI links and audio routing. > > Add "#sound-dai-cells" property to the binding and filter out non-DP > compatibles, as MediaTek eDP doesn't support audio. > > Signed-off-by: Fei Shao <fshao@chromium.org> > --- > > .../bindings/display/mediatek/mediatek,dp.yaml | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,dp.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,dp.yaml > index 2aef1eb32e11..c05c2b409780 100644 > --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,dp.yaml > +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,dp.yaml > @@ -42,6 +42,9 @@ properties: > interrupts: > maxItems: 1 > > + "#sound-dai-cells": > + const: 0 > + > ports: > $ref: /schemas/graph.yaml#/properties/ports > properties: > @@ -87,6 +90,19 @@ required: > You need to reference dai-common instead, since this is DAI. > additionalProperties: false and this becomes unevaluatedProperties: false Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,dp.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,dp.yaml index 2aef1eb32e11..c05c2b409780 100644 --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,dp.yaml +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,dp.yaml @@ -42,6 +42,9 @@ properties: interrupts: maxItems: 1 + "#sound-dai-cells": + const: 0 + ports: $ref: /schemas/graph.yaml#/properties/ports properties: @@ -87,6 +90,19 @@ required: additionalProperties: false +allOf: + - if: + not: + properties: + compatible: + contains: + enum: + - mediatek,mt8188-dp-tx + - mediatek,mt8195-dp-tx + then: + properties: + "#sound-dai-cells": false + examples: - | #include <dt-bindings/interrupt-controller/arm-gic.h>
The MediaTek DP hardware supports audio, and the "#sound-dai-cells" property is required to describe DAI links and audio routing. Add "#sound-dai-cells" property to the binding and filter out non-DP compatibles, as MediaTek eDP doesn't support audio. Signed-off-by: Fei Shao <fshao@chromium.org> --- .../bindings/display/mediatek/mediatek,dp.yaml | 16 ++++++++++++++++ 1 file changed, 16 insertions(+)