On 21/08/2024 11:26, Rohit Agarwal wrote:I was a bit confused here.
Add power domain binding to the mediatek DPI controllerMissing maxItems. I don't get why did you change this...
for MT8186.
Also, add power domain binding for other SoCs like
MT6795 and MT8173 that already had power domain property.
Signed-off-by: Rohit Agarwal <rohiagar@xxxxxxxxxxxx>
---
.../display/mediatek/mediatek,dpi.yaml | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
index 5ca7679d5427..864b781fdcea 100644
--- a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
+++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
@@ -62,6 +62,8 @@ properties:
- const: default
- const: sleep
+ power-domains: true
+This part can be dropped. Just disallow it for other devices.
port:
$ref: /schemas/graph.yaml#/properties/port
description:
@@ -76,6 +78,23 @@ required:
- clock-names
- port
+allOf:
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - mediatek,mt6795-dpi
+ - mediatek,mt8173-dpi
+ - mediatek,mt8186-dpi
+ then:
+ properties:
+ power-domains:
+ maxItems: 1
Best regards,
Krzysztof