Re: [PATCH V3] dt-bindings: clock: qcom: Remove required-opps from required list on SM8650

[Krzysztof Kozlowski]

On 30/07/2024 05:45, Jagadeesh Kona wrote:
> -
>  properties:
>    compatible:
>      enum:
> @@ -57,7 +54,24 @@ required:
>    - compatible
>    - clocks
>    - power-domains
> -  - required-opps
> +
> +allOf:
> +  - $ref: qcom,gcc.yaml#
> +  - if:
> +      properties:
> +        compatible:
> +          contains:
> +            enum:
> +              - qcom,sc8280xp-camcc
> +              - qcom,sm8450-camcc
> +              - qcom,sm8550-camcc
> +              - qcom,x1e80100-camcc
> +    then:
> +      required:
> +        - required-opps
> +    else:
> +      properties:
> +        required-opps: false


Why would required-opps be invalid for SM8650? What if we want some
higher opp for some reason? The point of v1 and v2 was oonly to require
required-opps on certain variants, not to disallow it in other cases.

Best regards,
Krzysztof