Re: [PATCH v3 1/3] dt-bindings: display: mediatek: dpi: Add power domains

["CK Hu (胡俊光)" <ck.hu@mediatek.com>]

Hi, Rohit:

On Thu, 2024-08-22 at 06:46 +0000, Rohit Agarwal wrote:
>  	 
> External email : Please do not click links or open attachments until you have verified the sender or the content.
>  Add power domain binding to the mediatek DPI controller
> for MT8186.
> Also, add power domain binding for other SoCs like
> MT6795 and MT8173 that already had power domain property.

Reviewed-by: CK Hu <ck.hu@mediatek.com>

> 
> Signed-off-by: Rohit Agarwal <rohiagar@chromium.org>
> ---
>  .../bindings/display/mediatek/mediatek,dpi.yaml | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
> index 5ca7679d5427..3a82aec9021c 100644
> --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
> +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,dpi.yaml
> @@ -62,6 +62,9 @@ properties:
>        - const: default
>        - const: sleep
>  
> +  power-domains:
> +    maxItems: 1
> +
>    port:
>      $ref: /schemas/graph.yaml#/properties/port
>      description:
> @@ -76,6 +79,20 @@ required:
>    - clock-names
>    - port
>  
> +allOf:
> +  - if:
> +      not:
> +        properties:
> +          compatible:
> +            contains:
> +              enum:
> +                - mediatek,mt6795-dpi
> +                - mediatek,mt8173-dpi
> +                - mediatek,mt8186-dpi
> +    then:
> +      properties:
> +        power-domains: false
> +
>  additionalProperties: false
>  
>  examples:
> -- 
> 2.46.0.295.g3b9ea8a38a-goog
>