Re: [PATCH] dt-bindings: dmaengine: qcom: gpi: Add minItems for interrupts

Vinod Koul
2022-04-20T10:36:10Z

On 14-04-22, 13:44, Krzysztof Kozlowski wrote:
> On 14/04/2022 12:25, Vinod Koul wrote:
> >>> Interrupt lines for each GPI instance
> >>> + minItems: 1
> >>
> >> This should be some real case minimum, not just 1. Unless really only
> >> one interrupt is also possible in existing variations?
> >
> > So that depends on the channels available to use which can be worst case
> > of 1. Maximum is 13.. Most of the controllers are between 12-13, but we
> > dont want to change binding in future if controller has lesser channels
> > right?
>
> If the choice is per SoC-controller, then the best would be to limit in
> allOf:if:then. However maybe the number of channels depends also on
> other factor (e.g. secure world configuration)?

That is quite right. So we wont know how many channels are made
available..

So is min 1 acceptable or do you have an alternate ?

--
~Vinod