Re: [PATCH] dt-bindings: media: venus: Add an optional power domain for perf voting

From: Rob Herring
Date: Tue May 12 2020 - 09:55:23 EST


On Tue, 12 May 2020 16:42:46 +0530, Rajendra Nayak wrote:
> Add an optional power domain which when specified can be used for
> setting the performance state of Venus.
>
> Signed-off-by: Rajendra Nayak <rnayak@xxxxxxxxxxxxxx>
> ---
> Documentation/devicetree/bindings/media/qcom,sc7180-venus.yaml | 4 +++-
> Documentation/devicetree/bindings/media/qcom,sdm845-venus-v2.yaml | 4 +++-
> 2 files changed, 6 insertions(+), 2 deletions(-)
>


My bot found errors running 'make dt_binding_check' on your patch:

/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/media/qcom,sdm845-venus-v2.example.dt.yaml: video-codec@aa00000: power-domain-names: ['venus', 'vcodec0', 'vcodec1'] is too short
/builds/robherring/linux-dt-review/Documentation/devicetree/bindings/media/qcom,sc7180-venus.example.dt.yaml: video-codec@aa00000: power-domain-names: ['venus', 'vcodec0'] is too short

See https://patchwork.ozlabs.org/patch/1288381

If you already ran 'make dt_binding_check' and didn't see the above
error(s), then make sure dt-schema is up to date:

pip3 install git+https://github.com/devicetree-org/dt-schema.git@master --upgrade

Please check and re-submit.