Re: [PATCH v3 1/2] dt-bindings: mailbox: Add support for bcm74110

[Justin Chen]

On 4/7/25 11:05 PM, Krzysztof Kozlowski wrote:
> On 07/04/2025 22:57, Justin Chen wrote:
>
>
> > from v2 of my patch was copied from said generated DTS.
> >
> > Apologies as I navigate through the different yaml keywords here. The HW
> > isn't changing, I am just struggling with representing the HW using the
> > different keywords. And the different implications of using said keywords.
> >
> > Here is what I have for v4.
> >        items:
> >          - description: RX doorbell and watermark interrupts
> >          - description: TX doorbell and watermark interrupts
> > +    description:
> > +      RX interrupts are required to notify the host of pending messages. TX
> > +      interrupts are optional. The TX doorbell interrupt is not used by the
> > +      host, but watermark interrupts may be used to notify a host
> > waiting on
> > +      a full out queue.
> What does it mean optional? Board decides on SoC connections? Given SoC
> is fixed isn't it?

Maybe I misunderstand optional here. We have optional from the SoC 
perspective and optional from a driver perspective. I was thinking if we 
have a HW feature that the software may choose to use, it is considered 
optional. If that is not the correct understanding, I can make 
adjustments. I will remove minItems, which from understanding, means 
both IRQs are required.

Thanks,
Justin

> Best regards,
> Krzysztof