Re: [PATCH] ASoC: dt-bindings: fsl,xcvr: Adjust the number of interrupts

[Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>]

On 05/12/2023 21:20, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@xxxxxxx>
> 
> Unlike i.MX8MP, i.MX93 has two XCVR interrupts.
> 
> Describe the two interrupts for the i.MX93 to fix the following
> dt-schema warning:
> 
> imx93-11x11-evk.dtb: xcvr@42680000: interrupts: [[0, 203, 4], [0, 204, 4]] is too long
> 	from schema $id: http://devicetree.org/schemas/sound/fsl,xcvr.yaml#
> 
> Signed-off-by: Fabio Estevam <festevam@xxxxxxx>
> ---
>  .../devicetree/bindings/sound/fsl,xcvr.yaml    | 18 +++++++++++++++++-
>  1 file changed, 17 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/sound/fsl,xcvr.yaml b/Documentation/devicetree/bindings/sound/fsl,xcvr.yaml
> index 799b362ba498..2db69a953afa 100644
> --- a/Documentation/devicetree/bindings/sound/fsl,xcvr.yaml
> +++ b/Documentation/devicetree/bindings/sound/fsl,xcvr.yaml
> @@ -38,7 +38,10 @@ properties:
>        - const: txfifo
>  
>    interrupts:
> -    maxItems: 1
> +    items:
> +      - description: WAKEUPMIX Audio XCVR Interrupt 1
> +      - description: WAKEUPMIX Audio XCVR Interrupt 2
> +    minItems: 1
>  
>    clocks:
>      items:
> @@ -78,6 +81,19 @@ required:
>    - dma-names
>    - resets
>  
> +allOf:
> +  - if:
> +      properties:
> +        compatible:
> +          contains:
> +            enum:
> +              - fsl,imx93-xcvr
> +    then:
> +      properties:
> +        interrupts:
> +          minItems: 2
> +          maxItems: 2

Same comment as your lcdif. Why all devices support now two interrupts?

Best regards,
Krzysztof