2021-11-09 20:28:00

by Geert Uytterhoeven

Subject: [PATCH v2] bindings: media: venus: Drop redundant maxItems for power-domain-names

make dt_binding_check:

Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml: ignoring, error in schema: properties: power-domain-names
warning: no schema found in file: Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml
Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml: properties:power-domain-names: {'required': ['maxItems']} is not allowed for {'minItems': 2, 'maxItems': 3, 'items': [{'const': 'venus'}, {'const': 'vcodec0'}, {'const': 'cx'}]}
hint: "maxItems" is not needed with an "items" list
from schema $id: http://devicetree.org/meta-schemas/items.yaml#

Fixes: e48b839b6699c226 ("media: dt-bindings: media: venus: Add sc7280 dt schema")
Signed-off-by: Geert Uytterhoeven <[email protected]>
Acked-by: Rob Herring <[email protected]>
---
v2:
- Add Acked-by,
- s/bogus/redundant/,
- Include full error message.
---
Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml | 1 -
1 file changed, 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml b/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml
index fa54c560e0bde3cb..e2874683b4d5faf3 100644
--- a/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml
+++ b/Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml
@@ -30,7 +30,6 @@ properties:

power-domain-names:
minItems: 2
- maxItems: 3
items:
- const: venus
- const: vcodec0
--
2.25.1

2021-11-12 04:23:20

by Rob Herring (Arm)

Subject: Re: [PATCH v2] bindings: media: venus: Drop redundant maxItems for power-domain-names

On Tue, 09 Nov 2021 11:29:41 +0100, Geert Uytterhoeven wrote:
> make dt_binding_check:
>
> Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml: ignoring, error in schema: properties: power-domain-names
> warning: no schema found in file: Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml
> Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml: properties:power-domain-names: {'required': ['maxItems']} is not allowed for {'minItems': 2, 'maxItems': 3, 'items': [{'const': 'venus'}, {'const': 'vcodec0'}, {'const': 'cx'}]}
> hint: "maxItems" is not needed with an "items" list
> from schema $id: http://devicetree.org/meta-schemas/items.yaml#
>
> Fixes: e48b839b6699c226 ("media: dt-bindings: media: venus: Add sc7280 dt schema")
> Signed-off-by: Geert Uytterhoeven <[email protected]>
> Acked-by: Rob Herring <[email protected]>
> ---
> v2:
> - Add Acked-by,
> - s/bogus/redundant/,
> - Include full error message.
> ---
> Documentation/devicetree/bindings/media/qcom,sc7280-venus.yaml | 1 -
> 1 file changed, 1 deletion(-)
>

Applied, thanks!