.../bindings/iommu/mediatek,iommu.yaml | 19 ++++++++++++++++++- 1 file changed, 18 insertions(+), 1 deletion(-)
From: Matthias Brugger <mbrugger@suse.com>
The mt8195-iommu-infra has five banks and one interrupt for each.
Reflect that in the binding.
Signed-off-by: Matthias Brugger <mbrugger@suse.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
+ 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
+ required:
+ - interrupts
+
- 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
additionalProperties: false
--
2.37.3
On 08/09/2022 15:25, matthias.bgg@kernel.org wrote: > From: Matthias Brugger <mbrugger@suse.com> > > The mt8195-iommu-infra has five banks and one interrupt for each. > Reflect that in the binding. > > Signed-off-by: Matthias Brugger <mbrugger@suse.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
© 2016 - 2024 Red Hat, Inc.