Re: [PATCH v2 1/3] dt-bindings: display: msm: qcm2290-mdss: Fix iommus property

[Sumit Garg <sumit.garg@kernel.org>]

On Fri, Jan 16, 2026 at 09:46:54AM +0100, Krzysztof Kozlowski wrote:
> On Fri, Jan 16, 2026 at 11:50:02AM +0530, Sumit Garg wrote:
> > From: Sumit Garg <sumit.garg@oss.qualcomm.com>
> > 
> > Fix IOMMU DT propety for display via dropping SMMU stream IDs which
> > relates to secure context bank. Assigning Linux kernel (HLOS) VMID
> > to secure context bank stream IDs is incorrect.
> > 
> > The min value is added for iommus property to ensure in future when
> > secure context bank stream IDs are properly supported then the iommus
> > property is extensible.
> > 
> > These DT bindings changes should be backwards compatible.
> > 
> > Signed-off-by: Sumit Garg <sumit.garg@oss.qualcomm.com>
> > ---
> >  .../devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml   | 5 ++---
> >  1 file changed, 2 insertions(+), 3 deletions(-)
> > 
> > diff --git a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml
> > index f0cdb5422688..5c888f07bc0b 100644
> > --- a/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml
> > +++ b/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml
> > @@ -33,6 +33,7 @@ properties:
> >        - const: core
> >  
> >    iommus:
> > +    minItems: 1
> 
> Same comment as other changes like that, which I already gave guideline
> - you need to list the items (minItems stay), because you now claim the
> order matters and is strictly defined.

Not sure if I claimed that order matters now but rather the secure bank
stream IDs were incorrectly represented earlier in the DT.

There has been ongoing disscusion related to how stream ID associated
with different translation context can be represented in DT here [1].
With that only the secure bank stream IDs can be properly represented.

Here I just followed the approach taken by Adreno GPU bindings for the
iommus property [2].

[1] https://lore.kernel.org/all/20250928171718.436440-1-charan.kalla@oss.qualcomm.com/
[2] Documentation/devicetree/bindings/display/msm/gpu.yaml +82

-Sumit