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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <aWo0t1EoVhStmOj7@sumit-xelite>
Date: Fri, 16 Jan 2026 18:23:11 +0530
From: Sumit Garg <sumit.garg@...nel.org>
To: Krzysztof Kozlowski <krzk@...nel.org>
Cc: linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
	andersson@...nel.org, konradybcio@...nel.org, robh@...nel.org,
	krzk+dt@...nel.org, conor+dt@...nel.org, akhilpo@....qualcomm.com,
	vikash.garodia@....qualcomm.com, dikshita.agarwal@....qualcomm.com,
	robin.clark@....qualcomm.com, lumag@...nel.org,
	loic.poulain@....qualcomm.com, jorge.ramirez@....qualcomm.com,
	linux-kernel@...r.kernel.org,
	Sumit Garg <sumit.garg@....qualcomm.com>
Subject: Re: [PATCH v2 1/3] dt-bindings: display: msm: qcm2290-mdss: Fix
 iommus property

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@....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@....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