On 16/02/2022 14:21, Geert Uytterhoeven wrote:
> "make dt_binding_check":
>
> Documentation/devicetree/bindings/firmware/arm,scpi.example.dt.yaml: scpi: shmem: [[2], [1]] is too long
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> ---
> Exposed by commit 39bd2b6a3783b899 ("dt-bindings: Improve phandle-array
> schemas").
> ---
> Documentation/devicetree/bindings/firmware/arm,scpi.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Documentation/devicetree/bindings/firmware/arm,scpi.yaml b/Documentation/devicetree/bindings/firmware/arm,scpi.yaml
> index 800417a76bda0bd9..3735f221fdf24e0a 100644
> --- a/Documentation/devicetree/bindings/firmware/arm,scpi.yaml
> +++ b/Documentation/devicetree/bindings/firmware/arm,scpi.yaml
> @@ -51,6 +51,7 @@ properties:
> be any memory reserved for the purpose of this communication between the
> processors.
> minItems: 1
> + maxItems: 2
The same should be done to "mboxes" (I mean 39bd2b6a3783b899 and what
you are doing here) and the maxItems should match both, shouldn't it?
Best regards,
Krzysztof
