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
| ||
|
Date: Thu, 8 Sep 2022 15:30:37 +0200 From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org> To: matthias.bgg@...nel.org, Yong Wu <yong.wu@...iatek.com>, Joerg Roedel <joro@...tes.org>, Will Deacon <will@...nel.org> Cc: devicetree@...r.kernel.org, Robin Murphy <robin.murphy@....com>, linux-kernel@...r.kernel.org, Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>, iommu@...ts.linux.dev, Rob Herring <robh+dt@...nel.org>, linux-mediatek@...ts.infradead.org, Matthias Brugger <matthias.bgg@...il.com>, linux-arm-kernel@...ts.infradead.org, Matthias Brugger <mbrugger@...e.com> Subject: Re: [PATCH v1] dt-bindings: iommu: mediatek: mt8195: Fix max interrupts On 08/09/2022 15:25, matthias.bgg@...nel.org wrote: > From: Matthias Brugger <mbrugger@...e.com> > > The mt8195-iommu-infra has five banks and one interrupt for each. > Reflect that in the binding. > > Signed-off-by: Matthias Brugger <mbrugger@...e.com> > > --- > > .../bindings/iommu/mediatek,iommu.yaml | 19 ++++++++++++++++++- > 1 file changed, 18 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/iommu/mediatek,iommu.yaml b/Documentation/devicetree/bindings/iommu/mediatek,iommu.yaml > index fee0241b50988..621aca2adfb7f 100644 > --- a/Documentation/devicetree/bindings/iommu/mediatek,iommu.yaml > +++ b/Documentation/devicetree/bindings/iommu/mediatek,iommu.yaml > @@ -91,7 +91,7 @@ properties: > maxItems: 1 > > interrupts: > - maxItems: 1 Keep here wide constraints - minItems:1, maxItems:5 > + description: each bank has one dedicated interrupt > > clocks: > items: > @@ -183,6 +183,19 @@ allOf: > required: > - mediatek,infracfg > > + - if: > + properties: > + compatible: > + contains: > + const: mediatek,mt8195-iommu-infra > + > + then: > + properties: > + interrupts: > + maxItems: 5 and then minItems:5 > + required: > + - interrupts This seems unrelated and not really needed. > + > - if: # The IOMMUs don't have larbs. > not: > properties: > @@ -191,8 +204,12 @@ allOf: > const: mediatek,mt8195-iommu-infra > > then: > + properties: > + interrupts: > + maxItems: 1 > required: > - mediatek,larbs > + - interrupts This seems unrelated and not really needed. > > additionalProperties: false > Best regards, Krzysztof
Powered by blists - more mailing lists