media: dt-bindings: samsung,s5p-mfc: Fix iommu properties schemas

The iommus and iommu-names property schemas have several issues. First,
'iommus-names' in the if/then schemas is the wrong name. As all the names
are the same, they can be defined at the top level instead. Then the
if/then schemas just need to define how many entries. The iommus if/then
schemas are also redundant. Best I can tell, the desire was to require 2
entries for "samsung,exynos5433-mfc", "samsung,mfc-v5", "samsung,mfc-v6",
and "samsung,mfc-v8".

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Aakarsh Jain <aakarsh.jain@samsung.com>
Link: https://lore.kernel.org/r/20231214195553.862920-1-robh@kernel.org
Signed-off-by: Rob Herring <robh@kernel.org>
This commit is contained in:
Rob Herring
2023-12-14 13:55:52 -06:00
parent 9de97e2a4e
commit a1499b7541

View File

@@ -49,7 +49,9 @@ properties:
iommu-names:
minItems: 1
maxItems: 2
items:
- const: left
- const: right
power-domains:
maxItems: 1
@@ -84,7 +86,7 @@ allOf:
- const: sclk_mfc
iommus:
maxItems: 1
iommus-names: false
iommu-names: false
- if:
properties:
@@ -102,11 +104,9 @@ allOf:
- const: aclk
- const: aclk_xiu
iommus:
maxItems: 2
iommus-names:
items:
- const: left
- const: right
minItems: 2
iommu-names:
minItems: 2
- if:
properties:
@@ -123,11 +123,9 @@ allOf:
- const: mfc
- const: sclk_mfc
iommus:
maxItems: 2
iommus-names:
items:
- const: left
- const: right
minItems: 2
iommu-names:
minItems: 2
- if:
properties:
@@ -144,11 +142,9 @@ allOf:
items:
- const: mfc
iommus:
maxItems: 2
iommus-names:
items:
- const: left
- const: right
minItems: 2
iommu-names:
minItems: 2
- if:
properties:
@@ -161,9 +157,6 @@ allOf:
clocks:
minItems: 1
maxItems: 2
iommus:
minItems: 1
maxItems: 2
examples:
- |