* [PATCH v2] bindings: media: venus: Drop redundant maxItems for power-domain-names
@ 2021-11-09 10:29 Geert Uytterhoeven
2021-11-12 4:23 ` Rob Herring
0 siblings, 1 reply; 2+ messages in thread
From: Geert Uytterhoeven @ 2021-11-09 10:29 UTC (permalink / raw)
To: Andy Gross, Bjorn Andersson, Stanimir Varbanov,
Mauro Carvalho Chehab, Rob Herring, Dikshita Agarwal
Cc: linux-arm-msm, linux-media, devicetree, linux-kernel,
Geert Uytterhoeven, Rob Herring
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 <geert+renesas@glider.be>
Acked-by: Rob Herring <robh@kernel.org>
---
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
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] bindings: media: venus: Drop redundant maxItems for power-domain-names
2021-11-09 10:29 [PATCH v2] bindings: media: venus: Drop redundant maxItems for power-domain-names Geert Uytterhoeven
@ 2021-11-12 4:23 ` Rob Herring
0 siblings, 0 replies; 2+ messages in thread
From: Rob Herring @ 2021-11-12 4:23 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Stanimir Varbanov, Rob Herring, linux-arm-msm,
Mauro Carvalho Chehab, linux-kernel, devicetree, Bjorn Andersson,
Andy Gross, linux-media, Dikshita Agarwal
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 <geert+renesas@glider.be>
> Acked-by: Rob Herring <robh@kernel.org>
> ---
> 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!
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-11-12 4:23 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-09 10:29 [PATCH v2] bindings: media: venus: Drop redundant maxItems for power-domain-names Geert Uytterhoeven
2021-11-12 4:23 ` Rob Herring
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).