Re: [PATCH 02/14] dt-bindings: opp: Increase maxItems for opp-hz property

From: Rob Herring
Date: Fri Jul 14 2023 - 12:17:45 EST


On Wed, Jul 12, 2023 at 04:01:57PM +0530, Manivannan Sadhasivam wrote:
> Current limit of 16 will be exhausted by platforms specifying the frequency
> for 9 clocks using opp-hz, like Qcom SDM845 SoC: 9 * 2 (64 bits) = 18

Are you sure about that? opp-hz is defined as uint64-matrix, so 16
64-bit entries should already be allowed. It doesn't matter how the .dts
is encoded (it used to though).

>
> So let's increase the limit to 32 which should be enough for most platforms
> (hopefully).
>
> Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx>
> ---
> Documentation/devicetree/bindings/opp/opp-v2-base.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/opp/opp-v2-base.yaml b/Documentation/devicetree/bindings/opp/opp-v2-base.yaml
> index 47e6f36b7637..e2f8f7af3cf4 100644
> --- a/Documentation/devicetree/bindings/opp/opp-v2-base.yaml
> +++ b/Documentation/devicetree/bindings/opp/opp-v2-base.yaml
> @@ -56,7 +56,7 @@ patternProperties:
> need to be configured and that is left for the implementation
> specific binding.
> minItems: 1
> - maxItems: 16
> + maxItems: 32
> items:
> maxItems: 1
>
> --
> 2.25.1
>