Re: [PATCH v2 1/3] dt-bindings: remoteproc: qcom,pas-common: allow up to 3 iommus

[Krzysztof Kozlowski <krzk@kernel.org>]

On Fri, Jul 17, 2026 at 03:28:23PM +0530, Pratyush Meduri wrote:
> The ADSP PAS on several Qualcomm SoCs (e.g. SA8775P/lemans, QCS8300,
> Monaco) needs more than a single SMMU stream ID for the audio ML
> (machine-learning) offload use case. In addition to the primary
> ADSP stream ID, the DSP/SPF issues DMA transactions through extra
> stream IDs that are not covered by the existing single-entry mapping.
> 
> Relax the iommus constraint to allow between 1 and 3 entries so the
> additional ML offload stream IDs can be described, fixing dtb_check
> schema violations on device trees that add them.

What dtb_check violations? Do not reference non-existing warnings. If
this is real, then place the warning here - and be sure we can actually
reproduce it.

> 
> Signed-off-by: Pratyush Meduri <pratyush.meduri@oss.qualcomm.com>
> ---
>  .../devicetree/bindings/remoteproc/qcom,pas-common.yaml        | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/remoteproc/qcom,pas-common.yaml b/Documentation/devicetree/bindings/remoteproc/qcom,pas-common.yaml
> index 11faf655f530..fb656bda8527 100644
> --- a/Documentation/devicetree/bindings/remoteproc/qcom,pas-common.yaml
> +++ b/Documentation/devicetree/bindings/remoteproc/qcom,pas-common.yaml
> @@ -33,7 +33,8 @@ properties:
>      maxItems: 6
>  
>    iommus:
> -    maxItems: 1
> +    minItems: 1
> +    maxItems: 3

So now all devices have three streams as well? Or only these three? You
said only newer, so why are you raising this also for sm8150?

Beside, you need to list the items.

Best regards,
Krzysztof