linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] dt-bindings: iommu: mediatek: mt8195: Fix max interrupts
@ 2022-09-08 14:15 matthias.bgg
  2022-09-08 14:17 ` Krzysztof Kozlowski
  2022-09-09  7:49 ` AngeloGioacchino Del Regno
  0 siblings, 2 replies; 3+ messages in thread
From: matthias.bgg @ 2022-09-08 14:15 UTC (permalink / raw)
  To: Yong Wu, Joerg Roedel, Will Deacon, Krzysztof Kozlowski
  Cc: devicetree, Robin Murphy, linux-kernel, iommu, Rob Herring,
	linux-mediatek, Matthias Brugger, linux-arm-kernel,
	Matthias Brugger

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>

---

Changes in v2:
- keep wide constrains for minItems and maxItems
- delete unrealted required: - interrupts
- use minItems for mediatek,mt8195-iommu-infra case

 .../bindings/iommu/mediatek,iommu.yaml         | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/iommu/mediatek,iommu.yaml b/Documentation/devicetree/bindings/iommu/mediatek,iommu.yaml
index fee0241b50988..4461dc027d1c4 100644
--- a/Documentation/devicetree/bindings/iommu/mediatek,iommu.yaml
+++ b/Documentation/devicetree/bindings/iommu/mediatek,iommu.yaml
@@ -91,7 +91,9 @@ properties:
     maxItems: 1
 
   interrupts:
-    maxItems: 1
+    minItems: 1
+    maxItems: 5
+    description: each bank has one dedicated interrupt
 
   clocks:
     items:
@@ -183,6 +185,17 @@ allOf:
       required:
         - mediatek,infracfg
 
+  - if:
+      properties:
+        compatible:
+          contains:
+            const: mediatek,mt8195-iommu-infra
+
+    then:
+      properties:
+        interrupts:
+          minItems: 5
+
   - if: # The IOMMUs don't have larbs.
       not:
         properties:
@@ -191,6 +204,9 @@ allOf:
               const: mediatek,mt8195-iommu-infra
 
     then:
+      properties:
+        interrupts:
+          maxItems: 1
       required:
         - mediatek,larbs
 
-- 
2.37.3


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] dt-bindings: iommu: mediatek: mt8195: Fix max interrupts
  2022-09-08 14:15 [PATCH v2] dt-bindings: iommu: mediatek: mt8195: Fix max interrupts matthias.bgg
@ 2022-09-08 14:17 ` Krzysztof Kozlowski
  2022-09-09  7:49 ` AngeloGioacchino Del Regno
  1 sibling, 0 replies; 3+ messages in thread
From: Krzysztof Kozlowski @ 2022-09-08 14:17 UTC (permalink / raw)
  To: matthias.bgg, Yong Wu, Joerg Roedel, Will Deacon, Krzysztof Kozlowski
  Cc: devicetree, Robin Murphy, linux-kernel, iommu, Rob Herring,
	linux-mediatek, Matthias Brugger, linux-arm-kernel,
	Matthias Brugger

On 08/09/2022 16:15, 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>


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>


Best regards,
Krzysztof

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] dt-bindings: iommu: mediatek: mt8195: Fix max interrupts
  2022-09-08 14:15 [PATCH v2] dt-bindings: iommu: mediatek: mt8195: Fix max interrupts matthias.bgg
  2022-09-08 14:17 ` Krzysztof Kozlowski
@ 2022-09-09  7:49 ` AngeloGioacchino Del Regno
  1 sibling, 0 replies; 3+ messages in thread
From: AngeloGioacchino Del Regno @ 2022-09-09  7:49 UTC (permalink / raw)
  To: matthias.bgg, Yong Wu, Joerg Roedel, Will Deacon, Krzysztof Kozlowski
  Cc: devicetree, Robin Murphy, linux-kernel, iommu, Rob Herring,
	linux-mediatek, Matthias Brugger, linux-arm-kernel,
	Matthias Brugger

Il 08/09/22 16:15, matthias.bgg@kernel.org ha scritto:
> 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>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>



_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-09-09  7:54 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-08 14:15 [PATCH v2] dt-bindings: iommu: mediatek: mt8195: Fix max interrupts matthias.bgg
2022-09-08 14:17 ` Krzysztof Kozlowski
2022-09-09  7:49 ` AngeloGioacchino Del Regno

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).