]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
dt-bindings: fsl-qdma: fix interrupts 'if' check logic
authorFrank Li <Frank.Li@nxp.com>
Mon, 1 Jul 2024 19:57:16 +0000 (15:57 -0400)
committerVinod Koul <vkoul@kernel.org>
Wed, 10 Jul 2024 10:12:40 +0000 (15:42 +0530)
All compatible string include 'fsl,ls1021a-qdma'. Previous if check are
always true.

if:
  properties:
    compatible:
      contains:
       enum:
         - fsl,ls1021a-qdma

Change to check other compatible strings to get correct logic and fix
below CHECK_DTB warnings.

arch/arm64/boot/dts/freescale/fsl-ls1028a-kontron-sl28-var1.dtb:
dma-controller@8380000: interrupts: [[0, 43, 4], [0, 251, 4], [0, 252, 4], [0, 253, 4], [0, 254, 4]] is too long

Signed-off-by: Frank Li <Frank.Li@nxp.com>
Reviewed-by: Rob Herring (Arm) <robh@kernel.org>
Link: https://lore.kernel.org/r/20240701195717.1843041-1-Frank.Li@nxp.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Documentation/devicetree/bindings/dma/fsl-qdma.yaml

index 1b689a2529c871c26892d85607f4fedcb8cfdf57..1b9ebdbe528a1ba6126b94784819a759d2e32ce9 100644 (file)
@@ -89,8 +89,16 @@ allOf:
         compatible:
           contains:
             enum:
-              - fsl,ls1021a-qdma
+              - fsl,ls1028a-qdma
+              - fsl,ls1043a-qdma
+              - fsl,ls1046a-qdma
     then:
+      properties:
+        interrupts:
+          minItems: 5
+        interrupt-names:
+          minItems: 5
+    else:
       properties:
         interrupts:
           maxItems: 3