Re: [PATCH 1/7] dtbindings: display: bcm2711-hdmi: Correct bindings for 2712

[Krzysztof Kozlowski <krzk@xxxxxxxxxx>]

On 02/12/2024 15:31, Dave Stevenson wrote:
> The previous patch just adding the compatible missed out that the


"The previous" is meaningless in this context. There is nothing
previous. Refer to commit (see submitting patches).

> number of interrupts changed
> 
> Fixes: 62948c62abca ("dt-bindings: display: Add BCM2712 HDMI bindings")

There is no such commit in linux-next. Fix patches while they are on the
mailing list. If you refer here to broken DRM process, then basically
you are on your own because we won't able to see it.

> Signed-off-by: Dave Stevenson <dave.stevenson@xxxxxxxxxxxxxxx>

Typo in subject prefix. It's dt-bindings.

> ---
>  .../bindings/display/brcm,bcm2711-hdmi.yaml        | 44 +++++++++++++++-------
>  1 file changed, 30 insertions(+), 14 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/display/brcm,bcm2711-hdmi.yaml b/Documentation/devicetree/bindings/display/brcm,bcm2711-hdmi.yaml
> index 6d11f5955b51..6af342c9b6b8 100644
> --- a/Documentation/devicetree/bindings/display/brcm,bcm2711-hdmi.yaml
> +++ b/Documentation/devicetree/bindings/display/brcm,bcm2711-hdmi.yaml
> @@ -56,22 +56,38 @@ properties:
>        - const: cec
>  
>    interrupts:
> -    items:
> -      - description: CEC TX interrupt
> -      - description: CEC RX interrupt
> -      - description: CEC stuck at low interrupt
> -      - description: Wake-up interrupt
> -      - description: Hotplug connected interrupt
> -      - description: Hotplug removed interrupt
> +    oneOf:
> +      - items:
> +        - description: CEC TX interrupt
> +        - description: CEC RX interrupt
> +        - description: CEC stuck at low interrupt
> +        - description: Wake-up interrupt
> +        - description: Hotplug connected interrupt
> +        - description: Hotplug removed interrupt

You need to narrow these per variant in allOf:if:then.
Best regards,
Krzysztof