Re: [PATCH] dt-bindings: opp: drop maxItems from inner items

[Dhruva Gole]

Hi,

On 05/03/24 05:13, David Heidelberg wrote:
> With recent changes within matrix dimensions calculation,
> dropping maxItems: 1 provides a warning-free run.
>
> Fixes warning such as:
> arch/arm64/boot/dts/qcom/sdm845-oneplus-enchilada.dtb: opp-table: opp-200000000:opp-hz:0: [200000000, 0, 0, 150000000, 0, 0, 0, 0, 300000000] is too long
>
> Fixes: 3cb16ad69bef ("dt-bindings: opp: accept array of frequencies")
> Suggested-by: Rob Herring <robh@xxxxxxxxxx>
> Signed-off-by: David Heidelberg <david@xxxxxxx>
> ---
>   follow-up of https://lore.kernel.org/lkml/20231229191038.247258-1-david@xxxxxxx/T/
>
>   Documentation/devicetree/bindings/opp/opp-v2-base.yaml | 2 --
>   1 file changed, 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/opp/opp-v2-base.yaml b/Documentation/devicetree/bindings/opp/opp-v2-base.yaml
> index e2f8f7af3cf4..b1bb87c865ed 100644
> --- a/Documentation/devicetree/bindings/opp/opp-v2-base.yaml
> +++ b/Documentation/devicetree/bindings/opp/opp-v2-base.yaml
> @@ -57,8 +57,6 @@ patternProperties:
>             specific binding.
>           minItems: 1
>           maxItems: 32
> -        items:
> -          maxItems: 1

Thanks for the fix,

Reviewed-by: Dhruva Gole <d-gole@xxxxxx>

--
Thanks and Regards,
Dhruva Gole